Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 299478

Summary: net-im/minbif: fails with forced --as-needed
Product: Gentoo Linux Reporter: Diego Elio Pettenò (RETIRED) <flameeyes>
Component: New packagesAssignee: Cédric Krier <cedk>
Status: RESOLVED FIXED    
Severity: QA CC: esigra, xarthisius
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
URL: http://www.gentoo.org/proj/en/qa/asneeded.xml
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 129413    
Attachments: Build log
adding patch to ebuild
fixing as-needed issues

Description Diego Elio Pettenò (RETIRED) gentoo-dev 2010-01-03 13:15:37 UTC
I'm reporting this bug because the package in summary fails to build when forcing --as-needed on through spec files (check out http://blog.flameeyes.eu/2008/11/14/problems-and-mitigation-strategies-for-as-needed for details).

Please note that this bug _might_ apply to -Wl,--as-needed in LDFLAGS as well; in both cases it should be fixed. Also, if this is due to the package in question not respecting user-defined LDFLAGS, you should get to fix that too.

Check the attached build log.

Thanks,
Diego
Comment 1 Diego Elio Pettenò (RETIRED) gentoo-dev 2010-01-03 13:15:54 UTC
Created attachment 215017 [details]
Build log
Comment 2 Kacper Kowalik (Xarthisius) (RETIRED) gentoo-dev 2010-01-05 20:50:44 UTC
Created attachment 215333 [details, diff]
adding patch to ebuild
Comment 3 Kacper Kowalik (Xarthisius) (RETIRED) gentoo-dev 2010-01-05 20:54:43 UTC
Created attachment 215335 [details, diff]
fixing as-needed issues

This patch is applied blindly, i.e. I wasn't able to reproduce this bug even on fresh stage3 chroot with every package recompiled with as-needed forced through specs.
However, minbif.cpp calls for g_thread_init(), which is defined in libgthread-2.0. Thus this patch _should_ fix issues present in the build.log
Comment 4 avx 2010-02-25 16:42:22 UTC
Patch works, but after ~6 weeks, this still ain't in portage? Please push.
Comment 5 avx 2010-02-25 16:43:35 UTC
I forgot, it build's clean and works on ~amd64, so please KEYWORD it accordingly, or should I file another bug for this?
Comment 6 Cédric Krier gentoo-dev 2010-02-28 14:06:09 UTC
Fix in cvs
Thanks