Philipp von Weitershausen <philipp@weitershausen.de> writes:
> cmpilato@collab.net wrote:
> > As well it should. I added the svn_auth.h header to the bindings
> > interface files a week or so ago, and used a script *very* similar to
> > Philipp's to verify that I could actually use the auth baton.
>
> Talking about making changes to the binding interface files: has
> anyone reviewed my patch yet? I would *really* like to discuss this
> and see it in some form of another in the trunk so I can start
> supplying more code that is based on these changes.
I've look at it briefly. I was a little disappointed that you
combined your -noproxy stuff with a change that removed the remapping
of svn_fs_FUNCTION_NAME to fs.FUNCTION_NAME (instead of
fs.svn_fs_FUNCTION_NAME). Not sure I like that, though I agree that
consistency is best.
My suggestion is to take this one task at a time -- do the -noproxy
stuff first, and meanwhile we as a community should decide which way
to go with the naming thing. (While I like the shorter names, I think
it'll come back to bite us as long as we continue to combine multiple
svn interfaces into a single Python module -- take the 'util' module
for example).
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon May 12 19:16:47 2003