On 01/16/2012 10:54 PM, Daniel Shahaf wrote:
> cmpilato_at_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_at_collab.net>
CollabNet <> www.collab.net <> Distributed Development On Demand
Received on 2012-01-17 14:15:58 CET