Joe Orton <jor...@redhat.com> writes: > I thought about that, but the SVN buildsystem is building & installing > them like libraries not like DSOs (which should be linked with libtool's > -module argument and not installed directly in $libdir),
I suspect that is accidental. The RA/FS option --enable-dso, which became --enable-runtime-module-search, was added a long time ago and the names of the libraries/modules do not change when switching between shared library and DSO. When the auth DSOs were added they just followed the same naming scheme as the RA/FS DSOs. > and there are > symbols exposed in the headers like svn_auth_gnome_keyring_version(). I'm not sure why those functions are public, we should probably deprecate them and make them private functions. I don't think a 3rd party can do anything useful with the public gnome/kwallet API. Even a 3rd party implementing a custom auth provider would not use the gnome/kwallet part of the svn_auth_ API. > So it looks a bit weird if they are really not intended to be used as > libraries? Your patch is an improvement so I would be happy to see it on trunk/1.10. Going further and removing some .pc files is probably also suitable for 1.10. Changing the DSO names and installation location is probably not suitable for 1.10. -- Philip
