Re: [PATCH 1/2] rebase --root: demonstrate a bug while amending rootcommit messages
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Jun 18, 2018, 21:46 UTC
- Message-ID
- <nycvar.QRO.7.76.6.1806182343421.77@tvgsbejvaqbjf.bet>
- In-Reply-To
- <20180618164958.GO11827@zaya.teonanacatl.net>
Hi Todd,
On Mon, 18 Jun 2018, Todd Zullinger wrote:
Show 24 quoted lines
> Phillip Wood wrote: > > On 15/06/18 05:31, Johannes Schindelin via GitGitGadget wrote: > >> > >> From: Todd Zullinger <tmz@pobox.com> > >> > >> +test_expect_failure 'rebase -i --root reword root commit' ' > >> + test_when_finished "test_might_fail git rebase --abort" && > >> + git checkout -b reword-root-branch master && > >> + set_fake_editor && > >> + FAKE_LINES="reword 1 2" FAKE_COMMIT_MESSAGE="A changed" \ > >> + git rebase -i --root && > >> + git show HEAD^ | grep "A changed" > > > > I wonder if it should also check that HEAD^ is the root commit, to make > > sure that the squash-onto commit that's created and then amended has > > been squashed onto. > > Hmm, is that something which other tests don't cover or an > issue that could affect 'rebase -i --root' with reword > differently than other 'rebase -i' commands? > > I admit I'm not well-versed in the rebase -i tests and I > focused only on creating a test which demonstrated the bug I > noticed.
I think we should test this here, to make sure it is tested, and it should be as easy as:
test -z "$(git show -s --format=%p HEAD^)"
Hopefully you beat me to it, otherwise I will try to take care of this tomorrow.
Ciao, Dscho