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

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

From: Martin Furter <mf_at_rola.ch>
Date: 2003-12-11 00:38:04 CET

On Thu, 11 Dec 2003, Edmund Horner wrote:

> 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...

Such a function would also simplify implementing automatic properties
for directories.

Martin

---------------------------------------------------------------------
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:40:34 2003

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.