Re: Git bisect run should check for the existence of the script

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Junio C Hamano <gitster@xxxxxxxxx> writes:

> Do you mean a new option must be passed if the end-user expects the
> script to always exist across revisions, or the script is not
> tracked to begin with?  It feels somewhat backwards and the effort
> by the end-user to always type the option is better spent to make
> sure there is no typo on the command line.

I forgot to follow up on this part.

It does not change the conclusion that it needs to be done carefully
if we chose to retroactivelyreserve return code 127 for our own use,
but such a backward incompatible change can easily be worked around
if users relied on the current behaviour that a missing (tracked)
script would mark the revision "bad" without being a fatal error in
"git bisect run".  Instead of

    git bisect run "./tracked-script"

they can just do

    git bisect run "test -f ./tracked-script && ./tracked-script"

and the problem is solved ;-)



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux