This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [committed, PATCH] PR ld/21923: Replace --rpath with -Wl,--rpath
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Alan Modra <amodra at gmail dot com>
- Cc: Binutils <binutils at sourceware dot org>
- Date: Tue, 8 Aug 2017 16:36:21 -0700
- Subject: Re: [committed, PATCH] PR ld/21923: Replace --rpath with -Wl,--rpath
- Authentication-results: sourceware.org; auth=none
- References: <20170808211705.GA3961@gmail.com> <20170808223922.GE16312@bubble.grove.modra.org>
On Tue, Aug 8, 2017 at 3:39 PM, Alan Modra <amodra@gmail.com> wrote:
> On Tue, Aug 08, 2017 at 02:17:05PM -0700, H.J. Lu wrote:
>> --- a/ld/configure.ac
>> +++ b/ld/configure.ac
>> @@ -470,7 +470,7 @@ EMULATION_LIBPATH=$all_libpath
>> AC_SUBST(EMULATION_LIBPATH)
>>
>> if test x${enable_static} = xno; then
>> - TESTBFDLIB="--rpath ../bfd/.libs ../bfd/.libs/libbfd.so"
>> + TESTBFDLIB="-Wl,--rpath ../bfd/.libs ../bfd/.libs/libbfd.so"
>> else
>> TESTBFDLIB="../bfd/.libs/libbfd.a"
>> fi
>
> Does that actually work? I thought you needed to write -Wl options
> without the spaces..
>
> TESTBFDLIB="-Wl,--rpath,../bfd/.libs,../bfd/.libs/libbfd.so"
>
Yes, a `,' is needed. For some reason, it passed the tests without it.
--
H.J.