[PATCH] test: fix RPATH in the test suite

Bernhard Reutner-Fischer rep.dot.nop at gmail.com
Mon Feb 2 22:02:40 UTC 2009


On Wed, Jan 28, 2009 at 04:39:35PM +0100, Carmelo AMOROSO wrote:
>-----BEGIN PGP SIGNED MESSAGE-----
>Hash: SHA1
>
>Folks,
>while reviewing test suite, found an issue in setting RPATH
>when TEST_INSTALLED_UCLIBC is not set.
>The attached patch should fix it.

Denys touched this in the meantime, i think?

PThe problem sounds like it's just coming from the testsuite not beeing
too well integrated into the in-tree (one could argue about whether it
should be more tightly coupled or purely freestanding, i.e. from outside
of libc. I'd prefer if it was not too intimately coupled, fwiw).

>- --------
>
>Build tests setting RPATH to ./ instead of $(top_builddir)lib
>to allow tests using their shared libraries to be loaded
>without setting LD_LIBRARY_PATH.
>When TEST_INSTALLED_UCLIBC is set, it doesn't need to set the RPATH,
>otherwise ass to the RPATH the $(top_builddir)lib too.
>UCLIBC_LDSO_ABSPATH is untouched.
>
>Signed-off-by: Carmelo Amoroso <carmelo.amoroso at st.com>
>
>Index: test/Rules.mak
>===================================================================
>- --- test/Rules.mak  (revision 25045)
>+++ test/Rules.mak  (working copy)
>@@ -103,11 +103,13 @@
>    HOST_LDFLAGS    += -static
> endif
>
>- -LDFLAGS += -B$(top_builddir)lib -Wl,-rpath,$(top_builddir)lib
>- -Wl,-rpath-link,$(top_builddir)lib
>+LDFLAGS += -B$(top_builddir)lib -Wl,-rpath,./
>- -Wl,-rpath-link,$(top_builddir)lib
> UCLIBC_LDSO_ABSPATH=$(shell pwd)
> ifdef TEST_INSTALLED_UCLIBC
>- -LDFLAGS += -Wl,-rpath,./
>- -UCLIBC_LDSO_ABSPATH=$(SHARED_LIB_LOADER_PREFIX)
>+   UCLIBC_LDSO_ABSPATH=$(SHARED_LIB_LOADER_PREFIX)
>+else
>+   LDFLAGS += -Wl,-rpath,$(top_builddir)lib
>+   UCLIBC_LDSO_ABSPATH=$(shell pwd)
> endif
>
> ifeq ($(findstring -static,$(LDFLAGS)),)


More information about the uClibc mailing list