Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 370109 - sci-libs/dsdp fails because of underlinking
Summary: sci-libs/dsdp fails because of underlinking
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Science Related Packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-05 12:01 UTC by Diego Elio Pettenò (RETIRED)
Modified: 2011-06-05 12:40 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
Build log (dsdp-5.8:20110605-005520.log,19.15 KB, text/plain)
2011-06-05 12:01 UTC, Diego Elio Pettenò (RETIRED)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Elio Pettenò (RETIRED) gentoo-dev 2011-06-05 12:01:28 UTC
Created attachment 275893 [details]
Build log

Hello there!

You're getting this bug because the package in Summary failed to build in my tinderbox using the gold link editor from binutils. Before closing the bug as INVALID let me explain why this is still important!

The gold link editor does not support underlinking of shared objects, which is something I have described in my blog post:

http://blog.flameeyes.eu/2010/11/26/it-s-not-all-gold-that-shines-why-underlinking-is-a-bad-thing

Even the basic link editor (ld.bfd) has an option to support this but it is a heck to enable and get passed, so linking with gold is simply quicker.

Fixing underlinking provides optimized --as-needed builds (because the "softer" version no longer need to recover libraries that are underlinked), so it is a Good Thing To Do.

Thank you very much for the attention!
Comment 1 Justin Lecher (RETIRED) gentoo-dev 2011-06-05 12:21:23 UTC
Although it is probably the wrong place to fix it here. It should be fixed in the pc file of blas-reference.


+  05 Jun 2011; Justin Lecher <jlec@gentoo.org> dsdp-5.8.ebuild,
+  +files/dsdp-5.8-gold.patch:
+  Make it gold resistant, #370109
+
Comment 2 Diego Elio Pettenò (RETIRED) gentoo-dev 2011-06-05 12:30:11 UTC
Nah, it is right to fix it here: translation units from dsdp itself are using openmp/libm functions.
Comment 3 Justin Lecher (RETIRED) gentoo-dev 2011-06-05 12:37:57 UTC
I am using atlas as blas implementation and there the pc file defines -lm in the linker line. Thats the reason why I mentioned that.
Comment 4 Diego Elio Pettenò (RETIRED) gentoo-dev 2011-06-05 12:40:10 UTC
Relying on that wouldn't really work well, it's a mistake on their part I guess. If they properly used the Libs.private field, -lm wouldn't be there, and you'd have the same issue ;)