Re: [PATCH] fix to svnperms.py to allow regex's that start with '['

Branko Čibej wrote:
>
> I don't like that. I'd really prefer it if svnperms.py started using
> ConfigParser for parsing the config file. There's nothing in the current
> config file that would prevent that, but if you add this syntax, the
> game's over.
>
> I'm not saying you should make this change yourself, only that any
> syntax change should maintain ConfigParser compatiblity.

Ok, that's a good point.

> Since you already have magic section names, i.e., [section groups], you
> could define inheritance in a similar way. For example:
>
> [parent]
> ...
> [section extends parent]
> [section]
> ...

Well, the problem there is that it breaks backwards compatibility.
"section extends parent" was previously a valid section name, and this
change will make it silently behave differently. I suppose its very
unlikely that someone would name a section that without intending
inheritance, though. Any other thoughts?

- Marc

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Aug 18 20:51:25 2005