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

Re: svn commit: r19901 - trunk

From: Philip Martin <philip_at_codematters.co.uk>
Date: 2006-06-01 22:45:48 CEST

dionisos@tigris.org writes:

> Author: dionisos
> Date: Thu Jun 1 12:12:30 2006
> New Revision: 19901
>
> Modified:
> trunk/build.conf
>
> Log:
> Workaround for build problems on OS X and (my) Debian with libtool 1.5.x
>
> Reorder dependencies, so that uninstalled libs are correctly found
> when running the test framework.
>
> Found by: breser
> dionisos
>
> Patch by: breser

The log message should be include something like:

 * build.conf (svnsync, libsvn_ra_local): Adjust library order.

>
>
> Modified: trunk/build.conf
> URL: http://svn.collab.net/viewvc/svn/trunk/build.conf?pathrev=19901&r1=19900&r2=19901
> ==============================================================================
> --- trunk/build.conf (original)
> +++ trunk/build.conf Thu Jun 1 12:12:30 2006
> @@ -308,7 +308,7 @@
> description = Subversion repository replicator
> type = exe
> path = subversion/svnsync
> -libs = libsvn_ra libsvn_delta libsvn_subr apr
> +libs = libsvn_subr libsvn_delta libsvn_ra apr

There should be a comment in the "code", not the log message, to
explain why this strange library order is needed.

> install = bin
>
>
> @@ -697,9 +697,8 @@
> path = subversion/tests/libsvn_ra_local
> sources = ra-local-test.c
> install = test
> -libs = libsvn_test libsvn_ra_local libsvn_ra
> - libsvn_fs libsvn_delta libsvn_subr
> - aprutil apriconv apr
> +libs = libsvn_subr libsvn_delta libsvn_fs libsvn_test
> + libsvn_ra_local libsvn_ra aprutil apriconv apr

ditto

Is this change portable? Libraries are now being linked in the
"wrong" order, libsvn_subr provides symbols that are used by libraries
that are later in the link line. Has this been tested on any of the
less common platforms like AIX, Solaris, HPUX, etc? Does it work when
the subversion libraries are static? Usually static libraries have to
be specified in the correct order, are you just lucky that there are
enough unresolved symbols when libsvn_subr is first encountered to
pull all the objects from the library?

I'd really like to know *why* this change is necessary. What changes
in the svnsync executable? Is the problem with the link commands that
run during the build, or is it the relink when the svnsync script is
run?

I realise it's frustrating when your build doesn't work, but until you
can explain how this "magic" solves the problem I think it is
premature to commit it. The discussion on the dev list just stopped
without reaching a conclusion :( If you have "before" and "after"
versions of the executable it must be possible to determine what
changes -- on Linux at least, I can't speak for Darwin.

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Jun 1 22:46:28 2006

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.