Subject: Re: [PATCH] update-ref: fail create operation over stdin if ref already exists

From: Michael Haggerty <[email-masked]>
Date: Thu, 03 Apr 2014 15:20:30 +0200

On 04/02/2014 02:57 PM, Brad King wrote:
> On 04/02/2014 04:09 AM, Michael Haggerty wrote:
>> From: Aman Gupta <[email-masked]>
> [snip]
>> @@ -147,6 +147,7 @@ static void parse_cmd_create(const char *next)
>>  	struct ref_update *update;
>>  
>>  	update = update_alloc();
>> +	update->have_old = 1;
> 
> Looks good.
> 
>> +test_expect_success 'stdin -z create ref fails when ref exists' '
> 
> Strictly speaking we should have a non-z mode test too.

This code path is used regardless of whether -z is used, so I don't
think it is necessary to duplicate the test.

Michael

-- 
Michael Haggerty
[email-masked]
http://softwareswirl.blogspot.com/