Re: [PATCH v11 00/22] Convert "git stash" to C builtin
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Nov 26, 2018, 07:38 UTC
- Message-ID
- <xmqq8t1gwano.fsf@gitster-ct.c.googlers.com>
- In-Reply-To
- <xmqqa7lwz8xm.fsf@gitster-ct.c.googlers.com>
Junio C Hamano <gitster@pobox.com> writes:
Show 18 quoted lines
> Thomas Gummerer <t.gummerer@gmail.com> writes: > >> Thanks for your work on this! I have read through the range-diff and >> the new patch of this last round, and this addresses all the comments >> I had on v10 (and some more :)). I consider it >> Reviewed-by: Thomas Gummerer <t.gummerer@gmail.com> > > Thanks. > > One thing that bothers me is that this seems to have been rebased on > 'master', but as long as we are rebasing, the updated series must > also take into account of the sd/stash-wo-user-name topic, i.e. if > we are rebasing it, it should be rebased on top of the result of > > git checkout -B ps/rebase-in-c master > git merge --no-ff sd/stash-wo-user-name > > I think.
https://travis-ci.org/git/git/builds/459619672 would show that this C reimplementation now regresses from the scripted version due to lack of such rebasing (i.e. porting a correction from scripted one).