Skip Menu |

This queue is for tickets about the CSS CPAN distribution.

Report information
The Basics
Id: 53948
Status: resolved
Priority: 0/
Queue: CSS

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

Bug Information
Severity: Normal
Broken in: 1.08
Fixed in: (no value)



CC: DCONWAY [...] cpan.org
Subject: Parse::RecDescent 1.963 breaks IAMCAL/CSS-1.08.tar.gz
From: paul [...] city-fan.org
On Sat Jan 23 23:41:01 2010, ANDK wrote: Show quoted text
> Just a heads up, I don't know whose bug this is. > > http://analysis.cpantesters.org/reports_by_field?distv=CSS- > 1.08;field=mod%3AParse%3A%3ARecDescent
This is also showing up in Debian and Fedora now: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=569404 https://bugzilla.redhat.com/show_bug.cgi?id=564808 Symptoms: Show quoted text
> make[1]: Entering directory
`/build/user-libcss-perl_1.08-1-amd64-upLvEr/libcss-perl-1.08' Show quoted text
> PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e"
"test_harness(0, 'blib/lib', 'blib/arch')" t/*.t Show quoted text
> t/01_basic.t ........... ok > t/02_parse_lite.t ...... ok > Use of uninitialized value $what in substitution (s///) at
/usr/share/perl5/Parse/RecDescent.pm line 3087. Show quoted text
> Use of uninitialized value $what in substitution (s///) at
/usr/share/perl5/Parse/RecDescent.pm line 3087. Show quoted text
> Use of uninitialized value $what in substitution (s///) at
/usr/share/perl5/Parse/RecDescent.pm line 3087. Show quoted text
> Use of uninitialized value $what in substitution (s///) at
/usr/share/perl5/Parse/RecDescent.pm line 3087. Show quoted text
> Use of uninitialized value $what in substitution (s///) at
/usr/share/perl5/Parse/RecDescent.pm line 3087. Show quoted text
> Use of uninitialized value $what in substitution (s///) at
/usr/share/perl5/Parse/RecDescent.pm line 3087. Show quoted text
> t/03_parse_heavy.t ..... ok > Unknown starting rule (Parse::RecDescent::namespace000001::stylesheet)
called Show quoted text
> at
/build/user-libcss-perl_1.08-1-amd64-upLvEr/libcss-perl-1.08/blib/lib/CSS/Parse/Compiled.pm line 28 Show quoted text
> Compilation failed in require at t/04_parse_compiled.t line 3. > # Looks like you planned 8 tests but ran 4. > # Looks like your test exited with 255 just after 4. > t/04_parse_compiled.t .. > Dubious, test returned 255 (wstat 65280, 0xff00) > Failed 4/8 subtests > t/05_search.t .......... ok > t/06_adaptor_simple.t .. ok > t/07_adaptor_pretty.t .. ok > t/08_adaptor_debug.t ... ok > Failed 1/8 test programs. 0/38 subtests failed.
The uninitialized value warnings are fixed by applying the patch from RT#54457 but I couldn't figure out what was going on with Parse::RecDescent::namespace000001::stylesheet.
CC: ANDK [...] cpan.org, DCONWAY [...] cpan.org
Subject: Re: [rt.cpan.org #53948] Parse::RecDescent 1.963 breaks IAMCAL/CSS-1.08.tar.gz
Date: Wed, 17 Feb 2010 09:38:52 +1100
To: bug-CSS [...] rt.cpan.org
From: Damian Conway <damian [...] conway.org>
On 16 February 2010 21:44, paul@city-fan.org via RT <bug-CSS@rt.cpan.org> wrote: Show quoted text
> I couldn't figure out what was going on with > Parse::RecDescent::namespace000001::stylesheet.
Definitely a P::RD bug. I just uploaded a new version that corrects the problem. Unfortunately, it requires that all precompiled grammars must be recompiled, so ANDK will have to release an update too. Apologies for that. Damian
From: paul [...] city-fan.org
On Tue Feb 16 17:40:21 2010, damian@conway.org wrote: Show quoted text
> On 16 February 2010 21:44, paul@city-fan.org via RT <bug- > CSS@rt.cpan.org> wrote: >
> > I couldn't figure out what was going on with > > Parse::RecDescent::namespace000001::stylesheet.
> > Definitely a P::RD bug. I just uploaded a new version that corrects > the problem. Unfortunately, it requires that all precompiled grammars > must be recompiled, so ANDK will have to release an update too. > Apologies for that.
Thanks for that. I think it's IAMCAL that will need to release the update though. I've managed to work around the issue by recompiling the grammar myself using the attached script and moving the resulting CompiledGrammar.pm on top of the existing one in the CSS/Parse/ directory. Curiously, this approach seems to work with P::RD 1.963 too, i.e. without the fix?
Subject: build-grammar.pl
use Parse::RecDescent; use CSS::Parse::PRDGrammar; $Parse::RecDescent::skip = ''; $::RD_AUTOACTION = 'print "token: ".shift @item; print " : @item\n"'; Parse::RecDescent->Precompile($CSS::Parse::PRDGrammar::GRAMMAR, "CSS::Parse::CompiledGrammar");
From: stolen-from-bitcard [...] l2g.to
Should we expect a new version to correct this bug anytime soon? I am trying to package this module for the MacPorts Project, and right now I think I will have to include this grammar- rebuilding script and incorporate it into the build process as a workaround.
No fix seen. I am now using the above parse-grammar.pl to build packages for openSUSE. Thanks for the script!
From: stercus [...] stercus.com
On Wed Feb 17 06:50:48 2010, paul@city-fan.org wrote: Show quoted text
> On Tue Feb 16 17:40:21 2010, damian@conway.org wrote:
> > On 16 February 2010 21:44, paul@city-fan.org via RT <bug- > > CSS@rt.cpan.org> wrote: > >
> > > I couldn't figure out what was going on with > > > Parse::RecDescent::namespace000001::stylesheet.
> > > > Definitely a P::RD bug. I just uploaded a new version that corrects > > the problem. Unfortunately, it requires that all precompiled grammars > > must be recompiled, so ANDK will have to release an update too. > > Apologies for that.
> > Thanks for that. I think it's IAMCAL that will need to release the > update though. > > I've managed to work around the issue by recompiling the grammar myself > using the attached script and moving the resulting CompiledGrammar.pm on > top of the existing one in the CSS/Parse/ directory. > > Curiously, this approach seems to work with P::RD 1.963 too, i.e. > without the fix?
Thanks for build-grammar.pl, but since it requires CSS::Parse::PRDGrammar to be installed, I guess this will not work for new installs where CSS::Parse::PRDGrammar does not aleady exist since it relies on CSS to install as well? Or did I missing something? I apologize if I did.
1.09 is on its way to CPAN with a recompiled grammar.