subversion-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "C. Michael Pilato" <cmpil...@collab.net>
Subject Re: svn commit: r1231988 - in /subversion/site/publish/docs/community-guide: repro-template.bat repro-template.sh
Date Tue, 17 Jan 2012 13:15:23 GMT
On 01/16/2012 10:54 PM, Daniel Shahaf wrote:
> cmpilato@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 <cmpilato@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand


Mime
View raw message