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

Bug 247909

Summary: media-video/nvclock: fails with --as-needed
Product: Gentoo Linux Reporter: Diego Elio Pettenò (RETIRED) <flameeyes>
Component: New packagesAssignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it <maintainer-needed>
Status: RESOLVED FIXED    
Severity: normal CC: jer, kanelxake, patrizio.bassi, tetromino
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
nvclock-0.8_beta4-as-needed.patch

Description Diego Elio Pettenò (RETIRED) gentoo-dev 2008-11-21 11:39:12 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).

Check the attached build log.

Thanks,
Diego
Comment 1 Diego Elio Pettenò (RETIRED) gentoo-dev 2008-11-21 11:39:45 UTC
Created attachment 172657 [details]
Build log
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2009-04-02 06:06:16 UTC
Maybe it's failing because it ignores LDFLAGS completely. I might have fixed this through bug #240846. Please check if media-video/nvclock-0.8_beta4-r1 still fails in this respect.
Comment 3 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2009-04-02 22:56:23 UTC
*** Bug 264651 has been marked as a duplicate of this bug. ***
Comment 4 Alexandre Rostovtsev (RETIRED) gentoo-dev 2009-04-03 00:26:56 UTC
Created attachment 187149 [details, diff]
nvclock-0.8_beta4-as-needed.patch

Patch to change link order and fix building with --as-needed.
Comment 5 Xake 2009-04-03 13:18:19 UTC
Since nvclock-0.8_beta4-r2 this is always true since the Makefiles does pick up LDFLAGS.

So could someone please update the summary and/or check in a fix?
Comment 6 Diego Elio Pettenò (RETIRED) gentoo-dev 2009-04-03 13:21:01 UTC
Patch looks good.
Comment 7 Xake 2009-04-03 13:25:26 UTC
Should we even consider go upstream with our changes (i.e. makeing the buildsystem pick up {C,LD}FLAGS and this fix)?
Comment 8 Jeroen Roovers (RETIRED) gentoo-dev 2009-04-03 23:55:59 UTC
Patch applied media-video/nvclock-0.8_beta4-r3.
Comment 9 Jeroen Roovers (RETIRED) gentoo-dev 2009-04-04 00:17:31 UTC
Mail with both patches attached sent upstream.