Bug 370109

Summary:	sci-libs/dsdp fails because of underlinking
Product:	Gentoo Linux	Reporter:	Diego Elio Pettenò (RETIRED) <flameeyes>
Component:	New packages	Assignee:	Gentoo Science Related Packages <sci>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	Build log

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 ;)