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

Re: svn commit: r1073325 - /subversion/trunk/contrib/client-side/svn_apply_autoprops.py

From: Stefan Sperling <stsp_at_apache.org>
Date: Tue, 22 Feb 2011 18:54:56 +0100

On Tue, Feb 22, 2011 at 12:18:36PM -0500, C. Michael Pilato wrote:
> On 02/22/2011 08:51 AM, stsp_at_apache.org wrote:
> > Author: stsp
> > Date: Tue Feb 22 13:51:16 2011
> > New Revision: 1073325
>
> [...]
>
> > In some situations users might want to use a different configuration file
> > to configure a specific set of autoprops, so add a new --config option.
>
> <bikeshed>
> Should we use "--config-dir" for consistency with other Subversion tools?
> </bikeshed>
>

The way it's written now it expects a config filename, not a config directory.
I suppose it could be make to look into a specified directory for a
file called 'config' instead. Though that would make things a bit harder
for folks who want to prepare a file like this:

cat > /tmp/myautoprops <<EOF
[auto-props]
*.c = svn:eol-style=native
*.h = svn:keywords=Author Date Id Rev URL;svn:eol-style=native
*.txt = svn:eol-style=native;svn:keywords=Author Date Id Rev URL;
EOF

and then want to use it.
Even though that's not a valid subversion config file (enable-auto-props
is missing), this works with the script.
Received on 2011-02-22 18:55:40 CET

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