[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1231988 - in /subversion/site/publish/docs/community-guide: repro-template.bat repro-template.sh

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Tue, 17 Jan 2012 08:15:23 -0500

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

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.