Re: [PATCH] Allow more than one pattern per line in svn:ignore

mark benedetto king wrote:

> On Thu, Dec 11, 2003 at 12:08:12AM +1300, Edmund Horner wrote:
>
>>* subversion/libsvn_wc/status.c
>> (collect_ignore_patterns): Split the svn:ignore property string
>> on any of "n\r\t\v " (as is used with global-ignores) to
>> allow more allow than one pattern per line.
>
>
> This makes it impossible to have patterns containing spaces. Since
> multiple patterns can simply be placed on multiple lines, the
> loss of functionality was deemed unacceptable. This was discussed
> earlier:
>
> http://subversion.tigris.org/servlets/ReadMsg?list=dev&msgNo=7473

Ah... I thought I'd done a proper search for any history. Apparently
not. :-(

The issue came up last night when someone asked on #svn why svn:ignore
wasn't working properly. They were putting more than one thing on a
single line; which was then being taken to be a single pattern for
filenames with spaces.

Although that's clearly what the Book says will happen, it was
inconsistent with global-ignores. I tok this was an oversight rather
than policy.

If (svn_cstring_split_append) has an extra argument QUOTED, or handles
esacaping with \, or a new function (svn_cstring_split_append_quoted) is
made...

I'm happy to do it but I'll ask for the list's advice first!

Edmund Horner.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Dec 11 00:24:15 2003