|Summary:||dev-perl/Compress-Raw-Zlib has an internal copy of zlib|
|Product:||Gentoo Linux||Reporter:||Diego Elio Pettenò (RETIRED) <flameeyes>|
|Component:||New packages||Assignee:||Gentoo Perl team <perl>|
|Package list:||Runtime testing required:||---|
Use external zlib in Compress-Raw-Zlib
Description Diego Elio Pettenò (RETIRED) 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) 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) 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!