Re: [PATCH] bisect: don't use invalid oid as rev when starting
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Sep 24, 2020, 07:51 UTC
- Message-ID
- <nycvar.QRO.7.76.6.2009240950010.5061@tvgsbejvaqbjf.bet>
- In-Reply-To
- <CAP8UFD3mfPgedqGk0ZFp3K3r1gDA3-stbSxoEomRh4T0_qH+tg@mail.gmail.com>
Hi Christian & Junio,
On Thu, 24 Sep 2020, Christian Couder wrote:
Show 26 quoted lines
> On Wed, Sep 23, 2020 at 11:39 PM Junio C Hamano <gitster@pobox.com> wrote: > > > > Johannes Schindelin <Johannes.Schindelin@gmx.de> writes: > > > > > From f673cea53e046774847be918f4023430e56bf6cb Mon Sep 17 00:00:00 2001 > > > From: Christian Couder <christian.couder@gmail.com> > > > Date: Wed, 23 Sep 2020 19:09:15 +0200 > > > Subject: [PATCH] bisect: don't use invalid oid as rev when starting > > > ... > > > diff --git a/builtin/bisect--helper.c b/builtin/bisect--helper.c > > > index 93e855271b9..d11d4c9bbb5 100644 > > > --- a/builtin/bisect--helper.c > > > +++ b/builtin/bisect--helper.c > > > > Unfortunately this does not apply to the broken commit or 'master' > > or anywhere else, it seems (no such blob as 93e855271b9 found at the > > path). > > > > It is better to make it applicable at least to 'master'. Making it > > also apply to 'maint' is optional, I would say, as the bug it fixes > > is not so critical. > > Sorry, I don't know what happened. It seemed to me that my branch was > based on master, but maybe I did something wrong. > > Hopefully the V2 I just sent will be better anyway.
FWIW I was working off of Miriam's `git-bisect-work-part2-v8` branch at https://gitlab.com/mirucam/git.git.
I'm happy with Christian's v2 (with or without the indentation fixes I suggested).
Ciao, Dscho