Use $(rtld-prefix) more consistently.
authorJoseph Myers <joseph@codesourcery.com>
Wed, 18 Jun 2014 10:35:29 +0000 (10:35 +0000)
committerJoseph Myers <joseph@codesourcery.com>
Wed, 18 Jun 2014 10:35:29 +0000 (10:35 +0000)
The glibc makefiles have a standard variable, $(rtld-prefix), to run
the dynamic linker with a default --library-path option; this is used
as the basis of lots of other variables for running programs compiled
with the newly built library.

A few places however use $(elf-objpfx)ld.so or
$(elf-objpfx)${rtld-installed-name} directly, with such a
--library-path option.  This patch makes such places use
$(rtld-prefix) instead.  I'm not aware of any significance in these
cases to the choice of ld.so or ${rtld-installed-name} when running
the dynamic linker, or to whether $(patsubst
%,:%,$(sysdep-library-path)) is included in the library-path as it is
in $(rtld-prefix) and just one of the places being changed.

Tested x86_64.

* elf/Makefile ($(objpfx)tst-unused-dep.out): Use $(rtld-prefix).
* iconvdata/Makefile ($(inst_gconvdir)/gconv-modules)
[$(cross-compiling) = no]: Likewise.
* sysdeps/s390/s390-64/Makefile ($(inst_gconvdir)/gconv-modules)
[$(cross-compiling) = no]: Likewise.

localedata/ChangeLog:
* Makefile (LOCALEDEF): Use $(rtld-prefix).

ChangeLog
elf/Makefile
iconvdata/Makefile
localedata/ChangeLog
localedata/Makefile
sysdeps/s390/s390-64/Makefile

index 2c554fa..2e452b4 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2014-06-18  Joseph Myers  <joseph@codesourcery.com>
+
+       * elf/Makefile ($(objpfx)tst-unused-dep.out): Use $(rtld-prefix).
+       * iconvdata/Makefile ($(inst_gconvdir)/gconv-modules)
+       [$(cross-compiling) = no]: Likewise.
+       * sysdeps/s390/s390-64/Makefile ($(inst_gconvdir)/gconv-modules)
+       [$(cross-compiling) = no]: Likewise.
+
 2014-16-17  Rajalakshmi Srinivasaraghavan <raji@linux.vnet.ibm.com>
 
        [BZ #17031]
index a2f5ce5..6602ed5 100644 (file)
@@ -1155,8 +1155,7 @@ $(objpfx)tst-unused-dep.out: $(objpfx)testobj1.so
        LD_TRACE_LOADED_OBJECTS=1 \
        LD_DEBUG=unused \
        LD_PRELOAD= \
-       $(elf-objpfx)${rtld-installed-name} \
-         --library-path $(rpath-link)$(patsubst %,:%,$(sysdep-library-path)) \
+       $(rtld-prefix) \
          $< > $@; \
        $(evaluate-test)
 
index 424e8ee..0a410a1 100644 (file)
@@ -252,7 +252,7 @@ ifeq (no,$(cross-compiling))
 # if this libc has more gconv modules than the previously installed one.
        if test -f "$(inst_gconvdir)/gconv-modules.cache"; then \
           LC_ALL=C \
-          $(elf-objpfx)ld.so --library-path $(rpath-link) \
+          $(rtld-prefix) \
           $(common-objpfx)iconv/iconvconfig \
             $(addprefix --prefix=,$(install_root)); \
        fi
index 68aab86..e5d0ae3 100644 (file)
@@ -1,3 +1,7 @@
+2014-06-18  Joseph Myers  <joseph@codesourcery.com>
+
+       * Makefile (LOCALEDEF): Use $(rtld-prefix).
+
 2014-06-09  Joseph Myers  <joseph@codesourcery.com>
 
        * Makefile (test-srcs): Remove tst-mbswcs1, tst-mbswcs2,
index 76eeedf..023f443 100644 (file)
@@ -216,7 +216,7 @@ INSTALL-SUPPORTED-LOCALES=$(addprefix install-, $(SUPPORTED-LOCALES))
 
 # Sometimes the whole collection of locale files should be installed.
 LOCALEDEF=I18NPATH=. GCONV_PATH=$(common-objpfx)iconvdata LC_ALL=C \
-$(elf-objpfx)ld.so --library-path $(rpath-link) $(common-objpfx)locale/localedef
+$(rtld-prefix) $(common-objpfx)locale/localedef
 install-locales: $(INSTALL-SUPPORTED-LOCALES)
 
 install-locales-dir:
index 4e69e90..fe5030e 100644 (file)
@@ -77,7 +77,7 @@ ifeq (no,$(cross-compiling))
 # if this libc has more gconv modules than the previously installed one.
        if test -f "$(inst_gconvdir)/gconv-modules.cache"; then \
           LC_ALL=C \
-          $(elf-objpfx)ld.so --library-path $(rpath-link) \
+          $(rtld-prefix) \
           $(common-objpfx)iconv/iconvconfig \
             $(addprefix --prefix=,$(install_root)); \
        fi