Re: svn commit: r37369 - in trunk: . build (transform_libtool_scripts.sh)

Arfrever Frehtes Taifersar Arahesis wrote:
> Author: arfrever
> Date: Sun Apr 19 12:32:18 2009
> New Revision: 37369
>
> Log:
> Use LD_PRELOAD to ensure that locally built Subversion libraries are used.
>
> * Makefile.in
> (local-all): Depend on 'transform-libtool-scripts' rule.
> (transform-libtool-scripts): New rule which calls
> 'build/transform_libtool_scripts.sh'.
>
> * build/transform_libtool_scripts.sh: New script which adds definitions of
> LD_PRELOAD to libtool wrapper scripts.

This approach seems rather far into the category of unpleasant hack to
me. I certainly feel that if it's a development-time convenience, it's
too ugly to allowed to occur in release builds.

There's also the problem that it seems to be duplicating a lot of
dependency information that formerly was nicely centralized in
build.conf for the most part.

IIUC the idea here is to affect the loading path for libs loaded via
svn_dso_load ? In which case, why is simply setting an appropriate
LD_LIBRARY_PATH not an acceptable solution?

Max.

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1831334