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

Re: [PATCH] Enhancement to svnperms.py to read sections via regex

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Fri, 24 Sep 2010 11:23:42 -0400

On 09/22/2010 09:07 AM, prabhugnanasundar wrote:
> Hi,
>
> The svnperms.py script reads the svnperms.conf file, which needs the
> section rules for all the repos uniquely. This was quite tough when we
> have n number if repos. I wished that svnperms.py reads regex matches
> and applies the configuration rules and came up with this patch.
> Mid-way of my coding work, I came to know that -s switch would let us
> specify the section name explicitly, but not the regex thing.
> I modified the code to search for a matching section via regex, which
> might be really helpfull for certain cases.
> I have attached the log file and the patch file with this mail for
> review.

I'm not familiar with this script, but won't your change break compatibility
with existing users of the script? Imagine that someone is using the script
today with 'svnperms.py -s perf', but their configuration file also has a
section called "[superfun]" (something that "perf" would match, albeit
incompletely. Will the new script, invoked the old way, read the wrong
configury?

Perhaps you should tie this change of interpretation to a new command-line
flag, perhaps a capitalized "s" (-S).

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on 2010-09-24 17:24:23 CEST

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