Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 276285 - media-libs/imlib2 fails with forced --as-needed and --no-undefined
Summary: media-libs/imlib2 fails with forced --as-needed and --no-undefined
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: SpanKY
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: as-needed
  Show dependency tree
 
Reported: 2009-07-02 20:00 UTC by Christian Ruppert (idl0r)
Modified: 2009-07-03 01:35 UTC (History)
0 users

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


Attachments
as-needed.patch (as-needed.patch,520 bytes, patch)
2009-07-02 20:00 UTC, Christian Ruppert (idl0r)
Details | Diff
imlib2-1.4.2-r1.ebuild.diff (imlib2-1.4.2-r1.ebuild.diff,731 bytes, patch)
2009-07-02 20:01 UTC, Christian Ruppert (idl0r)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Christian Ruppert (idl0r) gentoo-dev 2009-07-02 20:00:15 UTC
media-libs/imlib2 fails with USE="-*" and -Wl,--as-needed -Wl,--no-undefined due to missing -lm for a plugin.
Comment 1 Christian Ruppert (idl0r) gentoo-dev 2009-07-02 20:00:37 UTC
Created attachment 196425 [details, diff]
as-needed.patch
Comment 2 Christian Ruppert (idl0r) gentoo-dev 2009-07-02 20:01:02 UTC
Created attachment 196427 [details, diff]
imlib2-1.4.2-r1.ebuild.diff
Comment 3 SpanKY gentoo-dev 2009-07-03 01:35:57 UTC
as a dev, you really should be aware of proper patch management.  that means creating a patch named "as-needed.patch" is certainly not acceptable.

going beyond that, you should review this document:
http://dev.gentoo.org/~vapier/clean-patches

and to address the original issue, such a trivial change really shouldnt be done to the autotool source files, especially considering it alone needs to run a full autoreconf.  a simple sed can accomplish the same exact thing without forcing significantly more build time overhead.

http://sources.gentoo.org/media-libs/imlib2/imlib2-1.4.2-r1.ebuild?r1=1.6&r2=1.7