Skip Menu |

This queue is for tickets about the CGI-Application-Dispatch CPAN distribution.

Report information
The Basics
Id: 37859
Status: resolved
Priority: 0/
Queue: CGI-Application-Dispatch

People
Owner: Nobody in particular
Requestors: lubo.rintel [...] gooddata.com
Cc:
AdminCc:

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



Subject: Build.PL does not include Module::Build
Here's what Build.pl says: my $build_pkg = eval "require Apache::TestMB" ? 'Apache::TestMB' : 'Module::Build'; This will actually cause Module::Build not to be included in case there's no Apache::TestMB, since the eval will never yield false value (as the require is syntactically correct). The correct way to do this would probably be: eval "require Apache::TestMB"; my $build_pkg = $@ ? 'Module::Build' : 'Apache::TestMB';
On Tue Jul 22 16:06:46 2008, lkundrak wrote: Show quoted text
> Here's what Build.pl says: > > my $build_pkg = eval "require Apache::TestMB" ? 'Apache::TestMB' : > 'Module::Build'; > > This will actually cause Module::Build not to be included in case > there's no Apache::TestMB, since the eval will never yield false value > (as the require is syntactically correct). The correct way to do this > would probably be: > > eval "require Apache::TestMB"; > my $build_pkg = $@ ? 'Module::Build' : 'Apache::TestMB';
Thanks for the suggestion. Your idea will be included in 2.13.
This was fixed in the the 2.13 release.