| Summary: | dev-libs/nspr-4.8.7: 'nspr-4.8-config.patch' causes 'nspr-config --libs' to contain -R option | ||
|---|---|---|---|
| Product: | Gentoo/Alt | Reporter: | James G. Kim <jgkim> |
| Component: | Prefix Support | Assignee: | Gentoo Prefix <prefix> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | mozilla |
| Priority: | Normal | ||
| Version: | unspecified | ||
| Hardware: | AMD64 | ||
| OS: | OS X | ||
| See Also: | https://bugs.gentoo.org/show_bug.cgi?id=354523 | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Bug Depends on: | |||
| Bug Blocks: | 354523 | ||
hmmm, that sounds like the problem of bug #354523 (In reply to comment #1) > hmmm, that sounds like the problem of bug #354523 Yes, it's the cause behind the bug #354523. this version comes from an overlay, doesn't it? the problem is that the patch unconditionally adds -R in nspr-config, whereas it guards it for Linux in the makefiles Odd, since for Linux one would typically use -rpath=, which works on Solaris too with GNU ld, but -R is there for Solaris, for which it isn't used due to the conditional. (In reply to comment #3) > this version comes from an overlay, doesn't it? No, it comes from the main 'Gentoo Prefix' repo. > the problem is that the patch unconditionally adds -R in nspr-config, whereas > it guards it for Linux in the makefiles > > Odd, since for Linux one would typically use -rpath=, which works on Solaris > too with GNU ld, but -R is there for Solaris, for which it isn't used due to > the conditional. what I don't understand is why they don't just call pkg-config, like they do for commands like libdir. It seems to return the correct value to me. (`pkg-config --libs nspr` that is) ok, fixed in 4.8.9 now |
The last portion of the patch makes 'nspr-config --libs' to have -R option, which causes emerging portages like dev-lang/spidermonkey to fail with 'ld: unknown option: -R'. --- mozilla/nsprpub/config/nspr-config.in.orig 2005-05-11 00:53:41.000000000 +0000 +++ mozilla/nsprpub/config/nspr-config.in 2006-01-15 06:37:58.000000000 +0000 @@ -122,7 +122,7 @@ fi if test "$echo_libs" = "yes"; then - libdirs=-L$libdir + libdirs="-Wl,-R$libdir -L$libdir" if test -n "$lib_plds"; then libdirs="$libdirs -lplds${major_version}" fi