Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 178923 - dev-perl/Compress-Raw-Zlib has an internal copy of zlib
Summary: dev-perl/Compress-Raw-Zlib has an internal copy of zlib
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Perl team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-05-17 19:32 UTC by Diego Elio Pettenò (RETIRED)
Modified: 2007-06-06 23:14 UTC (History)
1 user (show)

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


Attachments
Use external zlib in Compress-Raw-Zlib (external-zlib.patch,381 bytes, patch)
2007-05-19 14:39 UTC, İsmail Dönmez
Details | Diff
Ebuild patch (crz-patch.patch,614 bytes, patch)
2007-05-23 21:25 UTC, Diego Elio Pettenò (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Elio Pettenò (RETIRED) gentoo-dev 2007-05-17 19:32:40 UTC
Not a good idea, in general it would be nicer to use the shared copy of zlib already present in the system, to avoid having to load more data (zlib is almost always already present in memory) and to allow fixes in zlib to be shared by its users too.
Comment 1 İsmail Dönmez 2007-05-19 14:39:44 UTC
Created attachment 119709 [details, diff]
Use external zlib in Compress-Raw-Zlib
Comment 2 İsmail Dönmez 2007-05-19 14:40:20 UTC
Attached a patch to use system zlib.
Comment 3 Diego Elio Pettenò (RETIRED) gentoo-dev 2007-05-23 21:25:18 UTC
Created attachment 120134 [details, diff]
Ebuild patch

Thanks to Ismail, I've been able to test the attached patch for the ebuild (Ismail's patch is bigger than the actual file to replace, and as only those variables are present and documented, this should do it).
Comment 4 Michael Cummings (RETIRED) gentoo-dev 2007-06-06 23:14:02 UTC
bumped with a variation of the second patch (except invoking multilib so lib64 would be correctly captured, or lib128, or lib1024....hey, some day...) Thanks!