Bug #67647 for Marpa: Forced to use Marpa::Internal

Wed Apr 20 23:31:07 2011 RSAVAGE [...] cpan.org - Ticket created

Subject:

Forced to use Marpa::Internal

Hi jeffrey ron@zigzag:~/perl.modules/Graph-Marpa$ perl -Ilib scripts/parse.pl Undefined subroutine &Marpa::exception called at /home/ron/perl5/perlbrew/perls/perl- 5.12.2/lib/site_perl/5.12.2/Marpa/Recognizer.pm line 885. I fixed this by adding use Marpa::Internal; beside my use of: use Marpa::Grammar; use Marpa::Recognizer; but I don't really think end users should have to. Cheers!

Wed Apr 20 23:34:49 2011 jkegl [...] cpan.org - Taken

Wed Apr 20 23:35:30 2011 jkegl [...] cpan.org - Correspondence added

Are you loading Marpa.pm? Always load Marpa.pm. It takes care of prerequisites. You should not load Marpa::Recognizer and Marpa::Grammar directly.

Wed Apr 20 23:35:30 2011 The RT System itself - Status changed from 'new' to 'open'

Wed Apr 20 23:38:41 2011 jkegl [...] cpan.org - Correspondence added

On Wed Apr 20 23:35:30 2011, JKEGL wrote: Show quoted text

> Are you loading Marpa.pm? > > Always load Marpa.pm. It takes care of prerequisites. You should not > load Marpa::Recognizer and Marpa::Grammar directly.

I note that the synopsis does NOT make this clear. So if this is indeed the problem, it's a documentation problem.

Wed Apr 20 23:39:05 2011 jkegl [...] cpan.org - Severity Critical changed to Normal

Wed Apr 20 23:41:28 2011 ron [...] savage.net.au - Correspondence added

Subject:	Re: [rt.cpan.org #67647] Forced to use Marpa::Internal
Date:	Thu, 21 Apr 2011 13:40:39 +1000
To:	bug-Marpa [...] rt.cpan.org
From:	Ron Savage <ron [...] savage.net.au>

Hi Jeffrey On Wed, 2011-04-20 at 23:35 -0400, Jeffrey Kegler via RT wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=67647 > > > Are you loading Marpa.pm? > > Always load Marpa.pm. It takes care of prerequisites. You should not > load Marpa::Recognizer and Marpa::Grammar directly.

No, I was loading each module I used. Just loading Marpa made it work, e.g I couldn't call value() on the recce obj. I suggest you patch the code in the synopsis of Marpa::Marpa to say that. It'll certainly remove some confusion... -- Ron Savage http://savage.net.au/ Ph: 0421 920 622

Wed Apr 20 23:44:48 2011 jkegl [...] cpan.org - Correspondence added

I'm accepting this as a documentation bug. I am closing the other as a duplicate.

Fri Apr 22 20:07:43 2011 jkegl [...] cpan.org - Correspondence added

The new Marpa.pod has a "use Marpa" line in both synopses. This fix is in Marpa-0.203_001. I am leaving this ticket open for a whle, pending cpantesters feedback. Note: A corresponding fix is in Marpa::XS 0.003_001.

Fri Apr 22 20:07:44 2011 jkegl [...] cpan.org - Status changed from 'open' to 'stalled'

Fri Apr 22 20:08:17 2011 jkegl [...] cpan.org - Fixed in 0.203_001 added

Tue Apr 26 13:43:28 2011 jkegl [...] cpan.org - Correspondence added

I typically wait for 3 days of cpantesters results on the release candidate before doing the official release. But 3 days have elapsed, and there are no cpantesters results for Marpa-0.203_001 yet, which is unusual. At some point I'll have to release, cpantesters or no cpantesters, but for the moment I think I'll just wait some more. On Fri Apr 22 20:07:43 2011, JKEGL wrote: Show quoted text

> The new Marpa.pod has a "use Marpa" line in both synopses. > > This fix is in Marpa-0.203_001. I am leaving this ticket open for a > whle, pending cpantesters feedback. > > Note: A corresponding fix is in Marpa::XS 0.003_001.

Tue Apr 26 13:43:29 2011 The RT System itself - Status changed from 'stalled' to 'open'

Tue Apr 26 18:11:17 2011 ron [...] savage.net.au - Correspondence added

Subject:	Re: [rt.cpan.org #67647] Forced to use Marpa::Internal
Date:	Wed, 27 Apr 2011 08:11:10 +1000
To:	bug-Marpa [...] rt.cpan.org
From:	Ron Savage <ron [...] savage.net.au>

Hi jeffrey On Tue, 2011-04-26 at 13:43 -0400, Jeffrey Kegler via RT wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=67647 > > > I typically wait for 3 days of cpantesters results on the release > candidate before doing the official release. But 3 days have elapsed, > and there are no cpantesters results for Marpa-0.203_001 yet, which is > unusual.

Over Easter, people could well be preoccupied... Show quoted text

> At some point I'll have to release, cpantesters or no cpantesters, but > for the moment I think I'll just wait some more.

Yep. Show quoted text

> On Fri Apr 22 20:07:43 2011, JKEGL wrote:

> > The new Marpa.pod has a "use Marpa" line in both synopses. > > > > This fix is in Marpa-0.203_001. I am leaving this ticket open for a > > whle, pending cpantesters feedback. > > > > Note: A corresponding fix is in Marpa::XS 0.003_001.

> > >

-- Ron Savage http://savage.net.au/ Ph: 0421 920 622

Wed May 04 00:02:40 2011 jkegl [...] cpan.org - Correspondence added

I've uploaded Marpa 0.204000 with this fix. Once it gets a few days of cpantesters results, if all looks well, I will close this ticket. On Tue Apr 26 18:11:17 2011, ron@savage.net.au wrote: Show quoted text

> Hi jeffrey > > On Tue, 2011-04-26 at 13:43 -0400, Jeffrey Kegler via RT wrote:

> > <URL: https://rt.cpan.org/Ticket/Display.html?id=67647 > > > > > I typically wait for 3 days of cpantesters results on the release > > candidate before doing the official release. But 3 days have elapsed, > > and there are no cpantesters results for Marpa-0.203_001 yet, which is > > unusual.

> > Over Easter, people could well be preoccupied... >

> > At some point I'll have to release, cpantesters or no cpantesters, but > > for the moment I think I'll just wait some more.

> > Yep. >

> > On Fri Apr 22 20:07:43 2011, JKEGL wrote:

> > > The new Marpa.pod has a "use Marpa" line in both synopses. > > > > > > This fix is in Marpa-0.203_001. I am leaving this ticket open for a > > > whle, pending cpantesters feedback. > > > > > > Note: A corresponding fix is in Marpa::XS 0.003_001.

> > > > > >

>

Wed May 04 00:03:02 2011 jkegl [...] cpan.org - Fixed in 0.204000 added

Wed May 04 00:03:02 2011 jkegl [...] cpan.org - Fixed in 0.203_001 deleted

Sat May 28 11:51:26 2011 jkegl [...] cpan.org - Correspondence added

cpantesters looks OK. The few reported FAILs seem to have been purged from the database. I believe they were problems with one test setup.

Sat May 28 11:51:26 2011 jkegl [...] cpan.org - Status changed from 'open' to 'resolved'