On 01/16/2012 10:54 PM, Daniel Shahaf wrote:
> cmpil...@apache.org wrote on Mon, Jan 16, 2012 at 13:12:20 -0000:
>> Modified: subversion/site/publish/docs/community-guide/repro-template.sh
>> URL: 
>> http://svn.apache.org/viewvc/subversion/site/publish/docs/community-guide/repro-template.sh?rev=1231988&r1=1231987&r2=1231988&view=diff
>> ==============================================================================
>> --- subversion/site/publish/docs/community-guide/repro-template.sh (original)
>> +++ subversion/site/publish/docs/community-guide/repro-template.sh Mon Jan 
>> 16 13:12:19 2012
>> @@ -31,7 +31,9 @@ LC_ALL=C; export LC_ALL
>>  # URL=svn://localhost/repos
>>  URL=file:///`pwd`/repos
>>  
>> -rm -rf repos wc import-me
>> +if [ -e repos ]; then rm -rf repos; fi
>> +if [ -e wc ]; then rm -rf wc; fi
>> +if [ -e import-me ]; then rm -rf import-me; fi
> 
> According to the POSIX man page on my system, the when -f is given
> nonexistent targets are silently skipped.
> 
> Therefore, seems to me this hunk is unneeded.

+1 to revert if you feel strongly about it (and I don't get around to it first).

-- 
C. Michael Pilato <cmpil...@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to