Re: [Tails-l10n] [dev-needed][fr] Re: Review of the translat…

Delete this message

Reply to this message
Author: intrigeri
Date:  
To: Tails localization discussion
Subject: Re: [Tails-l10n] [dev-needed][fr] Re: Review of the translations made by Mercedes508
elouann wrote (14 Aug 2015 04:08:04 GMT) :
> the branch now merges perfectly with origin/master,


great! merged (resolved a minor conflict), pushed, thanks.

> so hopefully the only problem left is this one:


>> > * resolving the conflicts this way mixes up *in a single commit* the
>> > modifications made during the conflict-merge and *all* the
>> > differences pulled from origin/master. This is not what we want,
>> > as this commit will be a pain to review. Unless I'm missing a secret
>> > git command.
>>
>> I'm curious how you check/see that => please push the result of
>> your merge.


> I pushed in the branch elouann/fr-solve-conflicts.


I've checked:

git diff --stat c005e0f..elouann/fr-solve-conflicts

(c005e0f is the parent commit from upstream's master branch that your
merge from August 12 took)

... and I can't confirm what you wrote earlier. I see only changes
introduced by the other parent commit of this merge.

However, I see no merge commit on that branch from late July, so it
seems that you didn't push the branch that exposed the problem
initially, which doesn't help investigating, and thus I may be
missing something.

> intrigeri <intrigeri@???> :


>> elouann wrote (31 Jul 2015 17:42:30 GMT) :
>> > Here is the situation:
>>
>> > * from the branch elouann/translation:
>> >     git fetch elouann translation
>> >     git checkout -b fr-solve-conflicts elouann/translation

>>
>> > * pull from master
>> >     git pull origin master
>> >   this returns indeed tons of conflicts.


> I tried in vain to separate the conflicting files from the others by
> merging at this step:


> * `git commit` returns "not possible because you have unmerged files."


> * `git commit -a` merges everything in a single commit, even the files
> containing conflicts.


Upon merge conflicts, IIRC Git makes it clear that one needs to first
`git add' files after having resolved conflicts, and then git commit
in the end.

My *guess* is that you're simply misunderstanding the `git status'
output when merging. Indeed, it does expose the changes brought by
both parent commits: the resulting tree *will* contain these
new changes.

Cheers,
--
intrigeri