Skip Menu |

This queue is for tickets about the Archive-Zip CPAN distribution.

Report information
The Basics
Id: 52696
Status: open
Priority: 0/
Queue: Archive-Zip

People
Owner: Nobody in particular
Requestors: salvatore.bonaccorso [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 1.31_04



Subject: t/99_pmv.t test failure with new Perl::MinimumVersion 1.22
Date: Sat, 12 Dec 2009 09:44:21 +0100
To: bug-Archive-Zip [...] rt.cpan.org, 560632 [...] bugs.debian.org
From: Salvatore Bonaccorso <salvatore.bonaccorso [...] gmail.com>
Hi First of all, I agree that this is a failure of build when activating author tests. We in Debian Perl Group try to activate any test which is possible to quarantee best quality. During such a rebuild of package for Archive::Zip it fails to build. There was an update in Perl::MinimumVersion: 1.22 Wed 25 Nov 2009 - Negative length in 'splice' requires perl 5.5 (Alexandr Ciornii) - 'mkdir' with one argument requires perl 5.6 (Alexandr Ciornii) - 'substr' will not be checked if it is a method name (Alexandr Ciornii) - better tests and fix for 3-arg 'open' (Alexandr Ciornii) 1.21 Tue 24 Nov 2009 - Add 'bytes' to 5.6 pragma (Alexandr Ciornii) - Add support for "version numbers" like "v1.1.1" (Alexandr Ciornii) - Reimplement _three_argument_open, more tests (Alexandr Ciornii) Now check for 3-arg open is working. - Check 4-arg substr that requires perl 5.5 (Alexandr Ciornii) - Postfix foreach requires perl 5.004.05 (Alexandr Ciornii) - Scalar::Util::weaken requires perl 5.006 (Alexandr Ciornii) - P::MV itself found imcompatibility in 02_main.t with 5.5 (Alexandr Ciornii) Now Archive::Zip fails with: Show quoted text
> make[2]: Entering directory `/build/user-libarchive-zip-perl_1.30-1-amd64-53Luyj/libarchive-zip-perl-1.30' > PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0, 'blib/lib', 'blib/arch')" t/*.t > t/01_compile.t ................ ok > t/02_main.t ................... ok > t/03_ex.t ..................... ok > t/04_readmember.t ............. ok > t/05_tree.t ................... ok > t/06_update.t ................. ok > t/07_filenames_of_0.t ......... ok > t/08_readmember_record_sep.t .. ok > t/09_output_record_sep.t ...... ok > t/10_chmod.t .................. ok > t/11_explorer.t ............... ok > t/12_bug_47223.t .............. skipped: Only required on Win32. > t/97_meta.t ................... ok > t/98_pod.t .................... ok > > # Failed test 't/10_chmod.t' > # at /usr/share/perl5/Test/MinimumVersion.pm line 150. > # t/10_chmod.t requires 5.006 due to syntax > # version markers for 5.006: > # - _three_argument_open > > # Failed test 't/09_output_record_sep.t' > # at /usr/share/perl5/Test/MinimumVersion.pm line 150. > # t/09_output_record_sep.t requires 5.006 due to syntax > # version markers for 5.006: > # - _three_argument_open > # Looks like you failed 2 tests of 29. > Failed 1/15 test programs. 2/256 subtests failed. > t/99_pmv.t .................... > Dubious, test returned 2 (wstat 512, 0x200) > Failed 2/29 subtests > > Test Summary Report > ------------------- > t/99_pmv.t (Wstat: 512 Tests: 29 Failed: 2) > Failed tests: 18, 27 > Non-zero exit status: 2 > Files=15, Tests=256, 13 wallclock secs ( 0.08 usr 0.03 sys + 13.00 cusr 0.32 csys = 13.43 CPU) > Result: FAIL > make[2]: *** [test_dynamic] Error 2
The full Bugreport is located at http://bugs.debian.org/560632 The problem is that in the tests there are used syntax not matching the required version in META.yml. Many thanks for considering looking into it, Bests Salvatore
Download signature.asc
application/pgp-signature 835b

Message body not shown because it is not plain text.

Hi Salvatore, Thank you for reporting the issue. The minimum version of perl required has been set to 5.004 and three-argument open has been converted to two- argument open. The changes are available in the repository at: http://svn.ali.as/cpan/trunk/Archive-Zip/. Regards, Alan Haggai Alavi. -- The difference makes the difference
Subject: Re: [rt.cpan.org #52696] t/99_pmv.t test failure with new Perl::MinimumVersion 1.22
Date: Sat, 23 Jan 2010 11:56:39 +0100
To: Alan Haggai Alavi via RT <bug-Archive-Zip [...] rt.cpan.org>
From: Salvatore Bonaccorso <salvatore.bonaccorso [...] gmail.com>
Hi Alan On Fri, Jan 22, 2010 at 09:28:11PM -0500, Alan Haggai Alavi via RT wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=52696 > > > Hi Salvatore, > > Thank you for reporting the issue. The minimum version of perl required > has been set to 5.004 and three-argument open has been converted to two- > argument open. The changes are available in the repository at: > http://svn.ali.as/cpan/trunk/Archive-Zip/.
Many thanks for your update and pointing to the svn repository! Bests regards Salvatore
Download signature.asc
application/pgp-signature 835b

Message body not shown because it is not plain text.