Skip Menu |

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

Report information
The Basics
Id: 40230
Status: resolved
Priority: 0/
Queue: Module-Install
People
Owner: Nobody in particular
Requestors: CHORNY [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in:
  • 0.73
  • 0.77
Fixed in: (no value)

History Show all quoted text
  Tue Oct 21 12:36:03 2008 CHORNY [...] cpan.org - Ticket created
Subject: test_requires and include
Found when installing PPI. test_requires 'File::Remove' => '0.39'; include 'File::Remove'; I have expected that File::Remove would not be required for install, but it is present in generated 'Makefile'. # PREREQ_PM => { File::Remove=>q[0.39], so it was passed to EU::MM as PREREQ_PM. -- Alexandr Ciornii, http://chorny.net
  Tue Oct 21 20:33:08 2008 adamkennedybackup [...] gmail.com - Correspondence added
Subject: Re: [rt.cpan.org #40230] test_requires and include
Date: Wed, 22 Oct 2008 11:32:58 +1100
To: bug-Module-Install [...] rt.cpan.org
From: "Adam Kennedy" <adamkennedybackup [...] gmail.com>
This is correct, I believe. Because it is bundled, and inc is included in the PATH, when MakeMaker receives the PREREQ_PM it will find the File::Remove in inc and everything should be ok. NOTABUG (unless someone shows otherwise) Adam K 2008/10/22 Alexandr Ciornii via RT <bug-Module-Install@rt.cpan.org>: Show quoted text
> Tue Oct 21 12:36:03 2008: Request 40230 was acted upon. > Transaction: Ticket created by CHORNY > Queue: Module-Install > Subject: test_requires and include > Broken in: 0.73, 0.77 > Severity: Normal > Owner: Nobody > Requestors: CHORNY@cpan.org > Status: new > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=40230 > > > > Found when installing PPI. > test_requires 'File::Remove' => '0.39'; > include 'File::Remove'; > I have expected that File::Remove would not be required for install, but > it is present in generated 'Makefile'. > # PREREQ_PM => { File::Remove=>q[0.39], > so it was passed to EU::MM as PREREQ_PM. > > -- > Alexandr Ciornii, http://chorny.net > >
  Tue Oct 21 20:33:10 2008 The RT System itself - Status changed from 'new' to 'open'
  Wed Oct 22 05:13:00 2008 CHORNY [...] cpan.org - Correspondence added
Втр. Окт. 21 20:33:08 2008, adamkennedybackup@gmail.com писал: Show quoted text
> This is correct, I believe. > > Because it is bundled, and inc is included in the PATH, when MakeMaker > receives the PREREQ_PM it will find the File::Remove in inc and > everything should be ok. > > NOTABUG (unless someone shows otherwise)
It is passed to Makefile and then installed by CPAN shell. This makes bundling useless. Also generated ppd file has dependency on them, but it is more a problem of EU::MM, which I hope will be resolved. -- Alexandr Ciornii, http://chorny.net
  Thu Oct 23 01:00:03 2008 adamkennedybackup [...] gmail.com - Correspondence added
Subject: Re: [rt.cpan.org #40230] test_requires and include
Date: Thu, 23 Oct 2008 15:59:50 +1100
To: bug-Module-Install [...] rt.cpan.org
From: "Adam Kennedy" <adamkennedybackup [...] gmail.com>
Actually, in that case it's possibly my fault... I shall investigate. 2008/10/22 Alexandr Ciornii via RT <bug-Module-Install@rt.cpan.org>: Show quoted text
> Queue: Module-Install > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=40230 > > > Втр. Окт. 21 20:33:08 2008, adamkennedybackup@gmail.com писал: >
>> This is correct, I believe. >> >> Because it is bundled, and inc is included in the PATH, when MakeMaker >> receives the PREREQ_PM it will find the File::Remove in inc and >> everything should be ok. >> >> NOTABUG (unless someone shows otherwise)
> > It is passed to Makefile and then installed by CPAN shell. This makes > bundling useless. > > Also generated ppd file has dependency on them, but it is more a problem > of EU::MM, which I hope will be resolved. > > -- > Alexandr Ciornii, http://chorny.net > >
  Fri Apr 09 19:23:40 2010 ishigaki [...] cpan.org - Correspondence added
Hi. Added a remedy in the trunk, and will be fixed in the next release. Thanks. On 2008-10-21 Tue 12:36:03, CHORNY wrote: Show quoted text
> Found when installing PPI. > test_requires 'File::Remove' => '0.39'; > include 'File::Remove'; > I have expected that File::Remove would not be required for install,
but Show quoted text
> it is present in generated 'Makefile'. > # PREREQ_PM => { File::Remove=>q[0.39], > so it was passed to EU::MM as PREREQ_PM. >
  Fri Apr 09 19:23:41 2010 ishigaki [...] cpan.org - Status changed from 'open' to 'patched'
  Sun May 09 22:47:02 2010 ishigaki [...] cpan.org - Correspondence added
Hi. Module::Install 0.96/0.97 with a fix is out. Thanks.
  Sun May 09 22:47:03 2010 The RT System itself - Status changed from 'patched' to 'open'
  Sun May 09 22:47:03 2010 ishigaki [...] cpan.org - Status changed from 'open' to 'resolved'