Possible bug with git-rebase and post-rewrite hook

2015-03-05 Thread Francis Moreau
Hi,

I have a suspect case which happens when rebasing a branch.

I'm using the post-rewrite hook and during certain circumstance, my hook
is called by git-rebase with wrong parameters:  argument is
not related to  one.

This actually happens when git-rebase stops and reports:

  The previous cherry-pick is now empty, possibly due to conflict
  resolution.
  If you wish to commit it anyway, use:

  git commit --allow-empty

  Otherwise, please use 'git reset'
  rebase in progress; onto 2889531
  You are currently rebasing branch 'foo' on '2889531'.


If I decide to skip it by doing 'git rebase --continue' then I get the
issue when my hook is run.

BTW, I find that the message is not really clear since it tells me to
run 'git-reset' in the case I want to ignore this empty commit. I'm not
sure that 'git-reset' is really helping in that case (my index is clean)
but 'git rebase --continue' would have been more appropriate IMHO.

If I do 'git commit --allow-empty' then there's no issue when the hook
is run.

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Possible bug with git-rebase and post-rewrite hook

2015-03-05 Thread Francis Moreau
oops, I forgot to say that this happen with git 2.3.1.

On 03/05/2015 02:55 PM, Francis Moreau wrote:
> Hi,
> 
> I have a suspect case which happens when rebasing a branch.
> 
> I'm using the post-rewrite hook and during certain circumstance, my hook
> is called by git-rebase with wrong parameters:  argument is
> not related to  one.
> 
> This actually happens when git-rebase stops and reports:
> 
>   The previous cherry-pick is now empty, possibly due to conflict
>   resolution.
>   If you wish to commit it anyway, use:
> 
>   git commit --allow-empty
> 
>   Otherwise, please use 'git reset'
>   rebase in progress; onto 2889531
>   You are currently rebasing branch 'foo' on '2889531'.
> 
> 
> If I decide to skip it by doing 'git rebase --continue' then I get the
> issue when my hook is run.
> 
> BTW, I find that the message is not really clear since it tells me to
> run 'git-reset' in the case I want to ignore this empty commit. I'm not
> sure that 'git-reset' is really helping in that case (my index is clean)
> but 'git rebase --continue' would have been more appropriate IMHO.
> 
> If I do 'git commit --allow-empty' then there's no issue when the hook
> is run.
> 
> Thanks.
> 

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html