Re: [PATCH v2 0/3] rebase -i: mark commits that begin empty in todo editor
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Apr 11, 2020, 21:14 UTC
- Message-ID
- <xmqqr1wthgib.fsf@gitster.c.googlers.com>
- In-Reply-To
- <CABPp-BEFKEr0hA8V4ZqEEGBCkYRbzrgNx2kyPCTvjSh3Oj25bQ@mail.gmail.com>
Elijah Newren <newren@gmail.com> writes:
Show 26 quoted lines
> On Fri, Apr 10, 2020 at 3:30 PM Junio C Hamano <gitster@pobox.com> wrote: >> >> Elijah Newren <newren@gmail.com> writes: >> >> > On Fri, Apr 10, 2020 at 2:29 PM Junio C Hamano <gitster@pobox.com> wrote: >> >> >> >> Junio C Hamano <gitster@pobox.com> writes: >> >> >> >> > So I am OK to rebase the other topic to v2.26.0; would that help? I >> >> > already saw there was some entanglement with the other topic in one >> >> > of the patches in this series, so... >> > ... >> > Thanks, but wouldn't I want to base his patch on top of mine rather >> > than vice versa? (Since mine might be labelled bugfix, and his is a >> > new feature?) >> >> If you are willing to rebase the --[no-]keep-empty so that it does >> not depend on v2.26, that may give us a better result. I just got >> an impression that you somehow wanted to base your changes on the >> newer codebase, but if you want to base both on the older codebase, >> that is fine by me, too. > > Rebasing my stuff on the older codebase wouldn't make sense; the > relevant code would be riddled with conflicts. Rather, I was thinking > of rebasing Jonathan's changes and building his series on top of mine > (and then touching up the docs so the flags reference each other).
That's OK as well. I didn't know if you wanted the "fix" to be only for 2.26.x and forward, or wanted to be applicable to 2.25.x series (which the other topic that is a new feature happened to also work with).
Thanks.