Skip Menu |
 

This queue is for tickets about the Module-Install CPAN distribution.

Report information
The Basics
Id: 54889
Status: resolved
Priority: 0/
Queue: Module-Install

People
Owner: Nobody in particular
Requestors: miyagawa [...] gmail.com
Cc:
AdminCc:

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



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>
Download (untitled) / with headers
text/plain 443b
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
Download (untitled) / with headers
text/plain 1004b
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. > >
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>
Download (untitled) / with headers
text/plain 1.2k
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
Download (untitled) / with headers
text/plain 203b
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! :)
Download (untitled) / with headers
text/plain 300b
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.
Download (untitled) / with headers
text/plain 408b
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.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.