Summary: | =dev-lang/perl-5.22.3_rc4 fails tests with >=sys-libs/zlib-1.2.9 | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Kent Fredric (IRC: kent\n) (RETIRED) <kentnl> |
Component: | Current packages | Assignee: | Gentoo Perl team <perl> |
Status: | RESOLVED FIXED | ||
Severity: | major | CC: | base-system |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | https://rt.cpan.org/Public/Bug/Display.html?id=119762 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
build.log.xz
build.log.xz |
Description
Kent Fredric (IRC: kent\n) (RETIRED)
2017-01-18 13:46:44 UTC
Created attachment 460550 [details]
build.log.xz
Odd. I added this log when I filed the bug, but it was over 1MB, and so bugzilla rejected it and didn't even bother to tell me.
So here it is xzcompressed.
Created attachment 460552 [details]
build.log.xz
Somehow that got corrupted during upload. I'll give up if this fails.
Looks like the breaking behaviour came in 1.2.9, so its possible its a yet-to-be-identified regression in zlib. Or it fixed a subtle problem in sizes of things, which perl was subtly dependent upon in the tests. Things like this seem like they could be relevant: - Assure that gzoffset() is correct when appending But now I've narrowed down where it is, I just have to work out why I can't reproduce it under perlbrew. From the upstream bug ($URL): --- On Sun Jan 08 12:07:10 2017, https://vcunat.mojeid.cz/#53tSKthUzV wrote: Hide quoted text > On Sun Jan 08 08:45:54 2017, PMQS wrote: > > zlib-1.2.10 changed the API for the deflateParams function. [...] > > So, I assume that means the current version won't (fully) work with > zlib-1.2.10... Correct. The function not working (deflateParams) is one I doubt many (if any) folk rely on. The fix is proving problematic, so may just temporarily remove support for deflateParams if I don't get it sorted soon. --- This falls into the "Eeeeek.... !" category... :/ I've just made a patch that disables the failing tests as an interim solution to avoid discouraging people running tests. https://github.com/gentoo-perl/perl-patchset/commit/1733e4e40d7a628b31d5b082f635a95763bfe1ad Will put it on the next 5.24.1 bump Its not great, but I wasn't able to find anything that touched these features. Fixed in 5.22.3 and 5.24.1-r1 by backporting XS code from latest Compress-Raw-Zlib commit f7f55098218de72848fed8ab450b5d895efc02eb Author: Kent Fredric <kentnl@gentoo.org> Date: Wed Feb 22 05:56:13 2017 +1300 dev-lang/perl,virtual/perl-Module-CoreList: Bump to 5.22.3 Final - Fix bug with patching now-missing CPANPLUS code #608966 - Update cross-perl to 1.1.4 - Fix cross-compile bugs - Fix GCC6 + lto bugs #583532 - Backport fixes from Compress-Raw-Zlib 2.074 for zlib 1.2.9+ #606374 Bug: https://bugs.gentoo.org/608966 Bug: https://bugs.gentoo.org/583532 Bug: https://bugs.gentoo.org/606374 Package-Manager: Portage-2.3.3, Repoman-2.3.1 dev-lang/perl/perl-5.22.3.ebuild | 550 +++++++++++++++++++++++++++++++++++++++ 1 file changed, 550 insertions(+) commit e03cb8b9c2fee8ea8fbc7668e9b9b01d9fca059a Author: Kent Fredric <kentnl@gentoo.org> Date: Tue Feb 21 15:14:57 2017 +1300 dev-lang/perl: Update to patchelevel 3 - Fix related to bug #608966 under crossdev where "touch" on a path would fail because the parent path did not exist - Fix patching failure under crossdev due to SHA1Sums changing - Upgrade perl-cross to 1.1.4 - Fix makefile ordering issue when compiling unicore/ parts before Config.pm has been compiled - Ensure miniperl is built with the right target arch for bootstrap - Upgrade base.pm patchset to latest for testing - Fix compilation with GCC6 and -flto #583532 - Properly fix compilation against newer zlibs #606374 by backporting changes to ZLib.xs from Compress-Raw-Zlib 2.074 Bug: https://bugs.gentoo.org/608966 Bug: https://bugs.gentoo.org/583532 Bug: https://bugs.gentoo.org/606374 Package-Manager: Portage-2.3.3, Repoman-2.3.1 dev-lang/perl/perl-5.24.1-r1.ebuild | 559 ++++++++++++++++++++++++++++++++++++ 1 file changed, 559 insertions(+) |