Bug #54889 for Module-Install: creates bogus PREREQ

Mon Feb 22 15:37:10 2010 miyagawa [...] gmail.com - Ticket created

Subject:	creates bogus PREREQ_PM with feature() groups
Date:	Mon, 22 Feb 2010 12:35:56 -0800
To:	bug-Module-Install [...] rt.cpan.org
From:	Tatsuhiko Miyagawa <miyagawa [...] gmail.com>

http://cpansearch.perl.org/src/XSAWYERX/Task-Dancer-0.06/Makefile.PL Run this Makefile.PL with PERL5_CPAN_IS_RUNNING=1 and look at the Makefile: It has line like this: # PREREQ_PM => { Text::HAML templating support=>q[0], Dancer::Template::Tenjin=>q[0], ExtUtils::MakeMaker=>q[6.42], which is not parsable. Seems Module::Install is passing the feature group name not each distro to EUMM and causing this bug. -- Tatsuhiko Miyagawa

Mon Feb 22 22:00:06 2010 ishigaki [...] cpan.org - Correspondence added

No, this isn't a Module::Install's bug. See its Makefile.PL. This patch would fix the issue (I'll forward this to the original dist). --- a/Makefile.PL +++ b/Makefile.PL @@ -15,7 +15,7 @@ feature 'Template::Tiny templating support', feature 'Tenjin templating support', -default => 0, - 'Dancer::Template::Tenjin' => 0, + 'Dancer::Template::Tenjin' => 0; feature 'Text::HAML templating support', -default => 0, On 2010-02-22 Mon 15:37:10, miyagawa@gmail.com wrote: Show quoted text

> http://cpansearch.perl.org/src/XSAWYERX/Task-Dancer-0.06/Makefile.PL > > Run this Makefile.PL with PERL5_CPAN_IS_RUNNING=1 and look at the

Makefile: Show quoted text

> > It has line like this: > # PREREQ_PM => { Text::HAML templating support=>q[0], > Dancer::Template::Tenjin=>q[0], ExtUtils::MakeMaker=>q[6.42], > > which is not parsable. > > Seems Module::Install is passing the feature group name not each > distro to EUMM and causing this bug. > >

Mon Feb 22 22:00:12 2010 The RT System itself - Status changed from 'new' to 'open'

Mon Feb 22 22:02:21 2010 miyagawa [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #54889] creates bogus PREREQ_PM with feature() groups
Date:	Mon, 22 Feb 2010 19:01:26 -0800
To:	bug-Module-Install [...] rt.cpan.org
From:	Tatsuhiko Miyagawa <miyagawa [...] gmail.com>

Wooow. Good catch, and sorry for the blame ;p On Mon, Feb 22, 2010 at 7:00 PM, Kenichi Ishigaki via RT <bug-Module-Install@rt.cpan.org> wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=54889 > > > No, this isn't a Module::Install's bug. See its Makefile.PL. This patch > would fix the issue (I'll forward this to the original dist). > > --- a/Makefile.PL > +++ b/Makefile.PL > @@ -15,7 +15,7 @@ feature 'Template::Tiny templating support', > > feature 'Tenjin templating support', > -default => 0, > - 'Dancer::Template::Tenjin' => 0, > + 'Dancer::Template::Tenjin' => 0; > > feature 'Text::HAML templating support', > -default => 0, > > > On 2010-02-22 Mon 15:37:10, miyagawa@gmail.com wrote:

>> http://cpansearch.perl.org/src/XSAWYERX/Task-Dancer-0.06/Makefile.PL >> >> Run this Makefile.PL with PERL5_CPAN_IS_RUNNING=1 and look at the

> Makefile:

>> >> It has line like this: >> # PREREQ_PM => { Text::HAML templating support=>q[0], >> Dancer::Template::Tenjin=>q[0], ExtUtils::MakeMaker=>q[6.42], >> >> which is not parsable. >> >> Seems Module::Install is passing the feature group name not each >> distro to EUMM and causing this bug. >> >>

> > > >

-- Tatsuhiko Miyagawa

Wed Feb 24 08:22:03 2010 ishigaki [...] cpan.org - Correspondence added

Closed this ticket. ref. https://rt.cpan.org/Ticket/Display.html?id=54899

Wed Feb 24 08:22:09 2010 ishigaki [...] cpan.org - Status changed from 'open' to 'resolved'

Sun Mar 07 04:15:17 2010 XSAWYERX [...] cpan.org - Correspondence added

On Wed Feb 24 08:22:03 2010, ISHIGAKI wrote: Show quoted text

> Closed this ticket. > > ref. https://rt.cpan.org/Ticket/Display.html?id=54899

Fixed in version 0.08, currently being uploaded to CPAN. Thank you both! :)

Sun Mar 07 04:15:24 2010 The RT System itself - Status changed from 'resolved' to 'open'

Sun Mar 07 04:16:17 2010 XSAWYERX [...] cpan.org - Correspondence added

On Sun Mar 07 04:15:17 2010, xsawyerx wrote: Show quoted text

> On Wed Feb 24 08:22:03 2010, ISHIGAKI wrote:

> > Closed this ticket. > > > > ref. https://rt.cpan.org/Ticket/Display.html?id=54899

> > Fixed in version 0.08, currently being uploaded to CPAN. > > Thank you both! :)

Eek.. opened it again by accident.

Sun Mar 07 07:17:01 2010 ishigaki [...] cpan.org - Correspondence added

No problem. Thank you for the update ;) On 2010-3-07 Sun 04:16:17, xsawyerx wrote: Show quoted text

> On Sun Mar 07 04:15:17 2010, xsawyerx wrote:

> > On Wed Feb 24 08:22:03 2010, ISHIGAKI wrote:

> > > Closed this ticket. > > > > > > ref. https://rt.cpan.org/Ticket/Display.html?id=54899

> > > > Fixed in version 0.08, currently being uploaded to CPAN. > > > > Thank you both! :)

> > Eek.. opened it again by accident.

Sun Mar 07 07:17:05 2010 ishigaki [...] cpan.org - Status changed from 'open' to 'resolved'

Bug #54889 for Module-Install: creates bogus PREREQ_PM with feature() groups