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

Re: Build configuration records

From: Max Bowsher <maxb_at_ukf.net>
Date: 2004-09-24 11:37:22 CEST

Edvard Majakari wrote:
> "Max Bowsher" <maxb@ukf.net> writes:
>
>> That would be me.
>
> My apologies for the laziness :)
>
> [...]
>> It removes the files just fine for me, so please don't say never.
>> I'd be interested to hear if you discover whatever weirdness is causing
>> it
>> not to work properly for you.
>
> I'm sorry. I didn't mean to say it doesn't work for you (or with others) -
> that it never works for me. I thought I'd look at it more carefully now,
> and realized it really handles DRYRUN correctly. Then I echoed what rm
> gets in $i
>
> if [ -z "$DRYRUN" ]; then
> rm -vrf "$i"
> fi
>
> and realized there's extra whitespace in front of every $i. The reason was
> then quickly found: sed only removed the first two whitespaces. Replacing
>
> for i in `${SVN-svn} status --no-ignore "$@" | sed -n 's/^[I\?] //p'`
>
> with
>
> for i in `${SVN-svn} status --no-ignore "$@" | sed -n 's/^[I\?] *//p'`
>
> made the script work for me. Thanks for the script!

You're welcome!

Your modification above will cause the script to break on files whose names
begin with spaces - unlikely, but possible. The correct number of spaces is
6. That is what I have in my copy of the script. Apparently my mailer ate
the extra spaces when I sent it.

Max.

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org
Received on Fri Sep 24 11:38:13 2004

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