Skip Menu |
 

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

Report information
The Basics
Id: 60432
Status: open
Priority: 0/
Queue: Module-Install

People
Owner: Nobody in particular
Requestors: djerius [...] cpan.org
Cc: CHORNY [...] cpan.org
AdminCc:

Bug Information
Severity: Critical
Broken in:
  • 0.96
  • 0.97
  • 0.98
  • 0.99
  • 1.00
Fixed in: (no value)



Subject: Bundled module's Makefile.PL ignored if Build.PL is present
Download (untitled) / with headers
text/plain 880b
Back at version 0.95, if a bundled distribution had both a Bundle.PL and an Makefile.PL file, the distribution was correctly built. Currently (since version 0.96) there's an explicit rejection of distributions which contain Bundle.PL. Some modules (Capture::Tiny for example) provide both Bundle.PL and Makefile.PL and work just fine with "perl Makefile.PL" and are thus being incorrectly rejected. My previously working distributions are now failing. I think the current behavior is too restrictive. The crucial test is for the existence of Makefile.PL. Since there's an explicit rejection of distributions without a Makefile.PL, the added rejection of Bundle.PL is superfluous, and as in case of Capture::Tiny, results in inappropriate rejection of distributions. I think it would also be useful if at make dist time such rejections were flagged as errors. Thanks! Diab
Download (untitled) / with headers
text/plain 421b
On Tue Aug 17 14:41:18 2010, DJERIUS wrote: Show quoted text
> Back at version 0.95, if a bundled distribution had both a Bundle.PL and > an Makefile.PL file, the distribution was correctly built. > > Currently (since version 0.96) there's an explicit rejection of > distributions which contain Bundle.PL.
Do you meant the case when Build.PL is written with Module::Install, not Module::Build? -- Alexandr Ciornii, http://chorny.net


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.