Re: stg 0.14.3 breakage on push after moving hunk

From: Karl Hasselström <kha@treskal.com>
Date: Sep 25, 2008, 08:37 UTC
Message-ID: <20080925083747.GC27632@diana.vm.bytemark.co.uk>
In-Reply-To: <20080925075732.GA17458@nan92-1-81-57-214-146.fbx.proxad.net>

On 2008-09-25 09:57:32 +0200, Yann Dirson wrote:

> Thanks, it does fix the exception - I feel a bit ashamed of not
> having looked at the code myself ;)

:-)

Show 17 quoted lines

> Nevertheless, that error was only hiding another more annoying
> problem:
>
> $ ../stgit/stg push
> Checking for changes in the working directory ... done
> Pushing patch "factorize" ...
>   Error: File "t/t4030-diff-rename-factorize.sh" added in both, permissions conflict
>   Error: The merge failed during "push".  
>          Use "refresh" after fixing the conflicts or revert the operation with "push --undo".
>   stg push: GIT index merging failed (possible conflicts)
> $ md5sum t/t4030-diff-rename-factorize.sh*
> cc313acd2824036556128b3e0879dd07  t/t4030-diff-rename-factorize.sh
> cc313acd2824036556128b3e0879dd07  t/t4030-diff-rename-factorize.sh.current
> cc313acd2824036556128b3e0879dd07  t/t4030-diff-rename-factorize.sh.patched
>
> AFAICT, adding the same file in 2 branches used not to produce a
> conflict at all. Is that intended ?

"permissions conflict" sounds like it was added with the exec bit in one branch, and without in the other. Is that right? And

  1. Does it work if there's no permissions conflict?

  2. Did it work before if there was a permissions conflict?

Just a guess, but it's entirely plausible that this case never worked, and you're the first to hit it.

-- 
Karl Hasselström, kha@treskal.com
      www.treskal.com/kalle