From: | Samuli Suominen <ssuominen@g.o> | ||
---|---|---|---|
To: | gentoo-dev@l.g.o | ||
Subject: | Re: [gentoo-dev] Re: News item announcing as-needed (glep 42 stuff) | ||
Date: | Tue, 27 Jul 2010 17:40:48 | ||
Message-Id: | 4C4F1A8B.9090209@gentoo.org | ||
In Reply to: | Re: [gentoo-dev] Re: News item announcing as-needed (glep 42 stuff) by "Róbert Čerňanský" |
1 | On 07/27/2010 08:37 PM, Róbert Čerňanský wrote: |
2 | > On Mon, 26 Jul 2010 21:38:05 -0600 |
3 | > Ryan Hill <dirtyepic@g.o> wrote: |
4 | > |
5 | >> it is recommended to use LDFLAGS="${LDFLAGS} <your flags>" instead to |
6 | > |
7 | > Is it not a comma required between ${LDFLAGS} and <your flags>? |
8 | > |
9 | > LDFLAGS="${LDFLAGS},<your flags>" |
10 | > |
11 | > Robert |
12 | |
13 | Don't do that. |
14 | |
15 | Quoting from http://www.gentoo.org/proj/en/qa/asneeded.xml: |
16 | |
17 | "Note: If you use more than one -Wl flag, you have to set |
18 | -Wl,--as-needed separately in LDFLAGS due to libtool deplib reordering[1]." |
19 | |
20 | [1] |
21 | http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/eclass/ELT-patches/as-needed/2.2.6?view=log |
22 | |
23 | Just use: |
24 | |
25 | LDFLAGS="${LDFLAGS} -Wl,--hash-style=gnu" |