Re: [PATCH] allow >1 URL schemes per RA plugin

Joe Orton <jorton@redhat.com> writes:

> The ra_dav plugin can support SSL, which means https: scheme URLs, but
> the current ra interface only allows one URL scheme per plugin. It could
> have two seperate _init functions, but that seems a bit clunky. Is this
> change okay?

Joe, this change looks great... and simplifies things as well. We've
known about this shortcoming for a while, so I'm glad you noticed and
fixed it. :-)

Go ahead and commit the change. I should mention that you forgot to
update the ra_local implementation though. Be sure to modify
libsvn_ra_local/ra_plugin.c:svn_ra_local_init() as part of your
change.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:36:32 2006