Mark Benedetto King wrote:
>On Thu, Oct 18, 2001 at 03:24:22PM -0500, Ben Collins-Sussman wrote:
>
>>Mark Benedetto King <bking@answerfriend.com> writes:
>>
>>>Okay, here's a second try.
>>>
>>This looks great. Smaller too. :-)
>>
>
>Amazing what using the correct level of abstraction will do.
>
>>
>>>+ if (list != NULL)
>>>+ {
>>>+ char *walk = list->data;
>>>+ do {
>>>+ char *p = strchr (walk, '\n');
>>>
>>Maybe we should be more flexible here. Can we allow the patterns to
>>be separated by any amount of whitespace?
>>
>>Personally, I'd like to see this someday:
>>
>> % svn proplist dir/
>> svn:ignore : *.so *.o *~
>>
>>...rather than see a bunch of newline-separated patterns.
>>
>
>
>Oh, sure. Of course, that will make patterns-with-spaces tricky,
>but people who put spaces in file names deserve what they get.
>
No! This attitude is simply wrong. Spaces in file names are quite
reasonable, and are in fact often used on systems Subversion runs (will
run) on.
There is absolutely no good reason to forbid whitespaces in patterns. We
went through this discussion when I originally proposed using the
svn:ignore property instead of .svnignore files, and it was actually Ben
who convinced me that we *didn't* need any special magic to store a list
of values into a property, because we already have the -F switch.
--
Brane �ibej <brane_at_xbc.nu> http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:36:45 2006