[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[PATCH] [KLUDGE] fix ra_loader

From: Scott Lamb <slamb_at_slamb.org>
Date: 2002-08-01 19:07:26 CEST

On Wed, Jul 31, 2002 at 10:04:55PM +0200, Branko ??ibej wrote:
> Scott Lamb wrote:
> >I've got a patch to the specfile to do this, but before I can use it, I
> >need ra_loader to work right. I'll try moving version to configure.in so
> >it can be used both in build-outputs.mk and in ra_loader.c
> ... and not at all on Windows, which doesn't use configure.in. We need a
> fix that can (be made to) work on all platforms.

I don't know how to do that. Is there any chance of this patch being
applied, that just hardcodes the version? I know it's not ideal - it
will break when the version changes. But it's broken now and this would
at least fix it for a while.

(Speaking of Windows, I think the thing never worked there - the other
### comment points out ".so" is assumed. I guess Apache never had this
problem because the name of the module was specified by the user in the
httpd.conf file.)

Scott Lamb
Fix --enable-dso
* subversion/libsvn_ra/ra_loader.c:
  Include hardcoded version 1 in library to load.
Index: subversion/libsvn_ra/ra_loader.c
--- subversion/libsvn_ra/ra_loader.c
+++ subversion/libsvn_ra/ra_loader.c	Thu Aug  1 11:57:41 2002
@@ -89,7 +89,9 @@
     apr_status_t status;
     /* ### fix the .so part */
-    libname = apr_psprintf (pool, "libsvn_ra_%s.so", ra_name);
+    /* ### fix the build system to make version accessible from the source
+       on Unix and Windows */
+    libname = apr_psprintf (pool, "libsvn_ra_%s-1.so", ra_name);
     funcname = apr_psprintf (pool, "svn_ra_%s_init", ra_name);
     /* find/load the specified library */
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Aug 1 19:07:26 2002

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.