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

Re: git bisect outputs list of commits in the wrong order

From
Junio C Hamano <gitster@pobox.com>
Date
Jun 10, 2022, 18:33 UTC
Message-ID
<xmqqmteke5cv.fsf@gitster.g>
In-Reply-To
<YqOJQKSH4eTo4cGn@coredump.intra.peff.net>
Jeff King <peff@peff.net> writes:
Show 17 quoted lines
> On Fri, Jun 10, 2022 at 10:28:04AM -0700, Junio C Hamano wrote:
>
>> I looked at an "scripted" version in the ancient past and it seems
>> to have computed by iterating over
>> 
>> 	git rev-list bisect/bad --not $good_revs
>> 
>> which would have given these intermediate ones roughly in the
>> reverse chronological order.  It could be that the behaviour
>> regressed when the scripted version was rewritten in C, but I dunno.
>> 
>> Christian (as an "area" expert for bisect), do you have any
>> comments?
>
> It would probably be nice to show them as --oneline, as well. I'd guess
> that a human reading the subjects of a handful of commits could quickly
> make a good guess as to the actual culprit.
True, too.

The scripted version fed the output from above iteration each to "show-branch" (as "git log --oneline" was probably not so widely used or may not have been available back then), to make them human readable. We should be able to use pretty.c::pretty_print_commit() with CMT_FMT_ONELINE to do that more easily these days.

By the way, welcome back ;-)
Previous: Jeff King
Message 4 of 4 in “git bisect outputs list of commits in the wrong order”
  1. Volker WeißmannJun 10, 2022
  2. Junio C HamanoJun 10, 2022
  3. Jeff KingJun 10, 2022
  4. Junio C HamanoJun 10, 2022

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.