git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] bisect: error out when given any good rev that is not an ancestor of the bad rev

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 1, 2008, 01:13 UTC
Message-ID
<7v7ic6tnir.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<200807010220.44657.chriscool@tuxfamily.org>
Christian Couder <chriscool@tuxfamily.org> writes:
> Another option is to introduce a switch to "git bisect start", 
> perhaps --strict, to please people who always want to use good revs that 
> are ancestor of the bad revs, so they get a nice error when this is not the 
> case.

I do not see much niceness value to such an option, though. I think such a switch is going the opposite way from improving usability.

You cannot bisect the history with such a draconian switch when you learn from somebody that 'maint' is Ok but 'master' is broken, and you already know it is a regression somewhere introduced in 'master', because you know you did _not_ fix it (or at least you do not remember fixing it) only for 'maint'.

Previous: Christian Couder
Message 11 of 11 in “bisect: error out when given any good rev that is not an ancestor of the bad rev”
  1. bisect: error out when given any good rev that is not an ancestor of the bad revChristian Couder, Jun 30, 2008
  2. Junio C HamanoJun 30, 2008
  3. Junio C HamanoJun 30, 2008
  4. Christian CouderJun 30, 2008
  5. Junio C HamanoJun 30, 2008
  6. Junio C HamanoJun 30, 2008
  7. Junio C HamanoJun 30, 2008
  8. Christian CouderJun 30, 2008
  9. Junio C HamanoJun 30, 2008
  10. Christian CouderJul 1, 2008
  11. Junio C HamanoJul 1, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.