osimons wrote:
> I'm not sure I get the importance of adding this abstraction? I
> haven't been tracking the branch as closely as I should, but is it
> really important? Changing the mount point would invalidate all links
> and more, so it might as well invalidate everything and require a
> resync? Please don't make it too complex for the rest of us... :-)

No, you're right, changing the mount point should almost never be done
anyway, and if you really need to do it, a resync is the least of your
worries.

> The ConfigParser does support case-sensitive option names, but not by
> default:
> from ConfigParser import ConfigParser
> myconfig = ConfigParser()
> myconfig.optionxform = str   # support case-sensitive option names
> myconfig.options('trac')
> ==> [..... , 'MyOption', .....]

That's correct, except that that option is not used in Trac, and I doubt
that we can change it now, as there are many config files out there with
all-lowercase options (as this is what is written when the configuration
is changed).

And instantiating a separate ConfigParser just for getting the
repository keys feels like a hack.

-- Remy

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to