Skip Menu |

This queue is for tickets about the Parse-RecDescent CPAN distribution.

Report information
The Basics
Id: 74787
Status: resolved
Priority: 0/
Queue: Parse-RecDescent

People
Owner: Nobody in particular
Requestors: POPEL [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 1.967003
Fixed in: (no value)



Subject: Missing dependency on newer ExtUtils::MakeMaker
Installation of Parse::RecDescent fails with older ExtUtils::MakeMaker (I had 6.56). You should add the newest version of ExtUtils::MakeMaker (6.62) to your META.json. The problem was caused by two authors in Makefile.PL. perl Makefile.PL WARNING: AUTHOR takes a string/number not a ARRAY reference. Please inform the author. Checking if your kit is complete... Looks good only nested arrays of non-refs are supported at /opt/lib/perl5/5.12.2/ExtUtils/MakeMaker.pm line 664 As a result no Makefile was created.
Thank you for the bug report. I've added a dependency in BUILD_REQUIRES and CONFIGURE_REQUIRES for ExtUtils::MakeMaker 6.62, and released version 1.967_005. You can view the commit here: https://github.com/jtbraun/Parse-RecDescent/commit/7e8b31aeb1a7c4975dcc11caa7392d8fe764adc3 If this doesn't resolve the issue for you, please let me know.
From: paul [...] city-fan.org
On Wed Feb 08 22:03:39 2012, jtbraun wrote: Show quoted text
> Thank you for the bug report. > > I've added a dependency in BUILD_REQUIRES and CONFIGURE_REQUIRES for > ExtUtils::MakeMaker 6.62, and released version 1.967_005. > > You can view the commit here: > > https://github.com/jtbraun/Parse- > RecDescent/commit/7e8b31aeb1a7c4975dcc11caa7392d8fe764adc3 > > If this doesn't resolve the issue for you, please let me know.
Support for an array of authors was added in ExtUtils::MakeMaker 6.57_02 so it's only necessary to require that version, rather than something as recent as 6.62. You might also want to bump the perl version requirement to 5.8.1; with 5.8.0, the test suite fails: $ make test PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0, 'blib/lib', 'blib/arch')" t/*.t t/00.load.................# Testing Parse::RecDescent 1.967006 ok t/01.basics...............ok t/autotree................ok t/leftop_cap..............ok t/pod.....................ok t/precompile..............ok t/re_capture_return.......ok t/reentry.................ok t/separated_repetition....ok t/skip....................Unknown starting rule (Parse::RecDescent::namespace000001::foo_with_skip) called at t/skip.t line 40 # Looks like you planned 8 tests but only ran 1. # Looks like your test died just after 1. dubious Test returned status 255 (wstat 65280, 0xff00) Scalar found where operator expected at (eval 153) line 1, near "'int' $__val" (Missing operator before $__val?) DIED. FAILED tests 2-8 Failed 7/8 tests, 12.50% okay t/skip_dynamic............ok t/text....................ok Failed 1/12 test scripts, 91.67% okay. 7/94 subtests failed, 92.55% okay. Failed Test Stat Wstat Total Fail Failed List of Failed ------------------------------------------------------------------------------- t/skip.t 255 65280 8 7 87.50% 2-8 make: *** [test_dynamic] Error 29
Show quoted text
> Support for an array of authors was added in ExtUtils::MakeMaker > 6.57_02 > so it's only necessary to require that version, rather than something > as > recent as 6.62.
I'll do so in the next release. Thank you. Show quoted text
> You might also want to bump the perl version requirement to 5.8.1; > with > 5.8.0, the test suite fails:
I believe this is caused by Text::Balanced being out of date. Presumably the version of Text::Balanced in 5.8.1 had some improvements. I'll investigate whether I can just require a newer version of Text::Balanced, which will still allow Parse::RecDescent to install on perl 5.8.0.
I've updated Makefile.PL to require: 'ExtUtils::MakeMaker' => 6.5702 And both Build.PL and Makefile.PL have been updated to require: 'Text::Balanced' => 1.95 which seems to have the necessary support for t/skip.t to pass. Earlier versions may work, but it is the version included in Perl 5.8.1. Users on Perl 5.8.0 should be able to upgrade Text::Balanced and get a working installation. Commit here: https://github.com/jtbraun/Parse-RecDescent/commit/9a56eec9e6a4364a5be93910be3f94120c8ce325 I'm working on a few other bug fixes, and will post a development release to CPAN shortly.