Re: svn commit: rev 5119 - in trunk/subversion: svnadmin include libsvn_fs tests tests/libsvn_fs tests/clients/cmdline/svntest libsvn_repos

Hey,
Forgive me for being blunt, but I don't like this config solution at all.
We are going to add a config option for every little thing for every fs
type? .... Bah!

My solution, which I'm in the process of getting back up-to-date, is
more flexible.
I use the svn_config_t structure which is loaded by svnadmin using
svn_config_read.
It gets passed opaquely from avnadmin all the way to the fs initializer
in a FS implementation.
Only the administrator and the affected code portions need to know about
specific options.
I also have extensions (as part of the first SQL solution) to the
svn_config_t methods. My SQL statement support code makes extensive use
of config files.

Please consider a different route:-(

If you want to see in better detail what I'm talking about, please go to
http://www.cdrguys.com/subversion/fspatch.4.gz
and search for scn_config_t. This was just the start. I was trying to
keep my patch size down:-)
I'm pretty sure I posted about this before but I'll go check.

Thanks,
gat

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Feb 26 22:54:12 2003