Bug #75342 for Module-ExtractUse: Module::ExtractUse::Grammar version error in Module-ExtractUse-0.24

Mon Feb 27 07:22:41 2012 mkmetw [...] hotmail.com - Ticket created

Subject:	Module::ExtractUse::Grammar version error in Module-ExtractUse-0.24
Date:	Mon, 27 Feb 2012 17:52:27 +0530
To:	<bug-module-extractuse [...] rt.cpan.org>
From:	Manoj Kumar <mkmetw [...] hotmail.com>

Hi, As per META.json / META.yml, the version of Module::ExtractUse::Grammar should be 0, but in the pm file VERSION is not defined. So, every time I run "r" in CPAN, I get this module in "to be updated" list. Please find attached the patch to solve this problem. Regards,

Message body is not shown because sender requested not to inline it.

Thu Mar 22 06:22:02 2012 domm [...] cpan.org - Correspondence added

On Mon Feb 27 07:22:41 2012, mkmetw@hotmail.com wrote: Show quoted text

> As per META.json / META.yml, the version of > Module::ExtractUse::Grammar should be 0, but in the pm file VERSION is > not defined. > > So, every time I run "r" in CPAN, I get this module in "to be updated" > list.

The file Module::ExtractUse::Grammar is autogenerated by Parse::ReqDescent, therefor it's a bit hacky to add the version to the generated code. But as I'm now precompiling the Grammar during Build, I've added some workaround that adds a VERSION (at least for Module::ExtractUse::Grammar). Please check if this works for you. I could also add a VERSION to Parse::RecDescent::Module::ExtractUse::Grammar, but this would be even more hackish.. Thanks for the feedback, domm

Thu Mar 22 06:22:04 2012 The RT System itself - Status changed from 'new' to 'open'

Thu Mar 22 06:22:07 2012 domm [...] cpan.org - Status changed from 'open' to 'resolved'

Fri Mar 23 06:29:38 2012 mkmetw [...] hotmail.com - Correspondence added

Subject:	RE: [rt.cpan.org #75342] Module::ExtractUse::Grammar version error in Module-ExtractUse-0.24
Date:	Fri, 23 Mar 2012 15:59:24 +0530
To:	<bug-module-extractuse [...] rt.cpan.org>
From:	Manoj Kumar <mkmetw [...] hotmail.com>

Hello, I have tested Module-ExtractUse-0.25 and Module-ExtractUse-0.26. Module-ExtractUse-0.25 is OK. Module-ExtractUse-0.26 fails. t/23_universal_require.t (Wstat: 65280 Tests: 28 Failed: 0) Non-zero exit status: 255 Parse errors: Bad plan. You planned 29 tests but ran 28. It seems you have not adjusted number of tests planned after removing Test::NoWarnings. If I decrease number of tests planned by 1, this test passes. Regards, Show quoted text

> Subject: [rt.cpan.org #75342] Module::ExtractUse::Grammar version error in Module-ExtractUse-0.24 > From: bug-Module-ExtractUse@rt.cpan.org > To: mkmetw@hotmail.com > Date: Thu, 22 Mar 2012 06:22:03 -0400 > > <URL: https://rt.cpan.org/Ticket/Display.html?id=75342 > > > On Mon Feb 27 07:22:41 2012, mkmetw@hotmail.com wrote: >

> > As per META.json / META.yml, the version of > > Module::ExtractUse::Grammar should be 0, but in the pm file VERSION is > > not defined. > > > > So, every time I run "r" in CPAN, I get this module in "to be updated" > > list.

> > The file Module::ExtractUse::Grammar is autogenerated by > Parse::ReqDescent, therefor it's a bit hacky to add the version to the > generated code. > > But as I'm now precompiling the Grammar during Build, I've added some > workaround that adds a VERSION (at least for > Module::ExtractUse::Grammar). Please check if this works for you. > I could also add a VERSION to > Parse::RecDescent::Module::ExtractUse::Grammar, but this would be even > more hackish.. > > Thanks for the feedback, > domm > > > > > >

Fri Mar 23 06:29:41 2012 The RT System itself - Status changed from 'resolved' to 'open'

Fri Mar 23 07:45:03 2012 domm [...] cpan.org - Correspondence added

fixed in 0.27, on it's way to cpan (and already on github) thanks!

Fri Mar 23 07:45:04 2012 domm [...] cpan.org - Status changed from 'open' to 'resolved'