Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 132567 - app-text/gocr-0.4.0: fix compilation with -Wl,--as-needed
Summary: app-text/gocr-0.4.0: fix compilation with -Wl,--as-needed
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Text-Markup Team (OBSOLETE)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: as-needed
  Show dependency tree
 
Reported: 2006-05-07 07:34 UTC by TGL
Modified: 2006-05-08 01:17 UTC (History)
0 users

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

Attachments
files/gocr-0.40-as-needed.patch (gocr-0.40-as-needed.patch,611 bytes, patch)
2006-05-07 07:35 UTC, TGL 		Details | Diff
gocr-0.40-ebuild.patch (gocr-0.40-ebuild.patch,763 bytes, patch)
2006-05-07 07:35 UTC, TGL 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description TGL 2006-05-07 07:34:24 UTC 
This package fails to build with LDFLAGS="-Wl,--as-needed" because of wrong order of the linker arguments, and unresolved symbols in libPgm2asc.a. The attached patch fixes that.

Note that i don't use the "gtk" flag on this package (no GTK+-1.2 here), so i don't know whether the frontend compiles.
Comment 1 TGL 2006-05-07 07:35:03 UTC 
Created attachment 86361 [details, diff]
files/gocr-0.40-as-needed.patch
Comment 2 TGL 2006-05-07 07:35:22 UTC 
Created attachment 86362 [details, diff]
gocr-0.40-ebuild.patch
Comment 3 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-05-07 20:42:30 UTC 
Patch looks fine for me.
Comment 4 Martin Ehmsen (RETIRED) gentoo-dev 2006-05-08 01:17:46 UTC 
Added to CVS in gocr-0.40-r1
Thanks for supplying the patch!

Just a quick remark: remember to use || die in your ebuilds on patches for style.