Re: [PATCH v2] bisect: improve output when bad commit is found
Jeff King <peff@peff.net> writes:
Show 10 quoted lines
> Of course, that is not a guarantee that nobody scraped stderr, but at
> least it makes me feel better that they're Doing It Wrong. :)
>
> It would be nice if we had some actual data points. I followed the link
> Christian gave to Ingo's old post, but I didn't see the actual script
> there. There is:
>
> https://github.com/grosser/git-autobisect/blob/master/lib/git/autobisect.rb
>
> which does seem to scrape stderr. Bleh.
Heh, anything ending with .rb is a bleh to me ;-)
Show 10 quoted lines
>> So perhaps if we keep
>>
>> <40 char sha1> is the first bad commit
>>
>> and then replace the diff-tree output with "show -s", then the
>> result would be good enough, I would say.
>
> That seems like a reasonable first step, at the very least. I wonder if
> we should also better document the exit code and BISECT_LOG semantics,
> and explicitly tell people not to scrape stderr.
Yeah, that would also be a good first step, whichever comes first.
Junio C Hamano <gitster@pobox.com>