On 09/22/2010 09:07 AM, prabhugnanasundar wrote:
> 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
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
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