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

Bug 178923

Summary: dev-perl/Compress-Raw-Zlib has an internal copy of zlib
Product: Gentoo Linux Reporter: Diego Elio Pettenò (RETIRED) <flameeyes>
Component: New packagesAssignee: Gentoo Perl team <perl>
Severity: normal CC: ismail
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Package list:
Runtime testing required: ---
Attachments: Use external zlib in Compress-Raw-Zlib
Ebuild patch

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!