Re: [PATCH v8 3/5] bisect: simplify the addition of new bisect terms

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

 



Junio C Hamano <gitster@xxxxxxxxx> writes:

> Matthieu Moy <Matthieu.Moy@xxxxxxx> writes:
>
>> diff --git a/bisect.c b/bisect.c
>> index 2d3dbdc..08be634 100644
>> --- a/bisect.c
>> +++ b/bisect.c
>> @@ -747,7 +747,10 @@ static void handle_bad_merge_base(void)
>>  				"between %s and [%s].\n",
>>  				bad_hex, bad_hex, good_hex);
>>  		} else {
>> -			die("BUG: terms %s/%s not managed", name_bad, name_good);
>> +			fprintf(stderr, "The merge base %s is %s.\n"
>> +				"This means the first commit marked %s is "
>> +				"between %s and [%s].\n",
>> +				bad_hex, name_bad, name_bad, bad_hex, good_hex);

Indeed, I forgot to apply the previous remark. Fixed.

> -		if (!strcmp(name_bad, "bad")) {
> +		if (!strcmp(name_bad, "bad") && !strcmp(name_good, "good") {

Indeed. Applied.

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/
--
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



[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]