Christian Couder <chriscool@xxxxxxxxxxxxx> writes:
>> > bisect_replay () {
>> >
>> > + test "$#" -eq 1 || die "No logfile given"
>> >
>> > test -r "$1" || die "cannot read $1 for replaying"
>> > bisect_reset
>> > while read git bisect command rev
>>
>> While at it perhaps you could do something like:
>>
>> bisect_replay () {
>> + test "$#" -lt 1 || die "No logfile given"
>> + test "$#" -gt 1 || die "Too many argument. Please give only one logfile."
>> test -r "$1" || die "cannot read $1 for replaying"
>> bisect_reset
>> while read git bisect command rev
>
> I mean:
>
> bisect_replay () {
> + test "$#" -lt 1 && die "No logfile given"
> + test "$#" -gt 1 && die "Too many argument. Please give only one logfile."
> test -r "$1" || die "cannot read $1 for replaying"
> bisect_reset
> while read git bisect command rev
That suggests that the original patch is buggy as well, in that it says
"No logfile given" when it gets two.
How about checking with 1 and saying "Give me only one logfile" and be
done with it?
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html