diff options
author | David Golden <dagolden@cpan.org> | 2010-01-19 21:33:35 -0500 |
---|---|---|
committer | David Golden <dagolden@cpan.org> | 2010-01-19 21:33:35 -0500 |
commit | 7dc9e1b44427ddd72c25f7d40cea4e597b6b55c8 (patch) | |
tree | 3448cec96bad320d4108ac3bd26440befb03b62e | |
parent | 81fc59efbb1a08ca114afd7063c6d7630d321552 (diff) | |
download | perl-7dc9e1b44427ddd72c25f7d40cea4e597b6b55c8.tar.gz |
Update Module::Build to 0.3603
0.3603 - Mon Jan 18 22:28:59 EST 2010
Bug fixes:
- Module::Build::Compat would croak on distibutions that set requires
'perl' to a dotted decimal like '5.6.2'. We now skip that key
since it doesn't go into PREREQ_PM and we numify it properly for
'use 5.006002' in the generated Makefile.PL (RT#53409)
[David Golden, adapted from patch by G. Allen Morris III]
0.3602 - Mon Jan 18 22:09:54 EST 2010
Bug fixes:
- Fix failures in t/properties/needs_compiler.t when $ENV{CC} is set
(RT#53296) [David Golden, adapted from patch by Jens Rehsack]
0.3601 - Mon Dec 21 14:39:33 EST 2009
Bug fixes:
- When the currently running Module::Build is not the same as the one
that created the Build file, there is now a warning rather than a fatal
error. This helps installation of dependency chains where a dependency
might configure_requires a new Module::Build after Build.PL was already
run for an earlier distribution. [David Golden, on advice of Matt Trout]
Other:
- t/bundle_inc.t fails in odd ways. This test of an experimental feature
should not prevent users from installing Module::Build, so this test
now skips unless $ENV{MB_TEST_EXPERIMENTAL} is true
44 files changed, 145 insertions, 96 deletions
diff --git a/Porting/Maintainers.pl b/Porting/Maintainers.pl index a946ddc382..919ccb1a7b 100755 --- a/Porting/Maintainers.pl +++ b/Porting/Maintainers.pl @@ -1004,7 +1004,7 @@ use File::Glob qw(:case); 'Module::Build' => { 'MAINTAINER' => 'kwilliams', - 'DISTRIBUTION' => 'DAGOLDEN/Module-Build-0.36.tar.gz', + 'DISTRIBUTION' => 'DAGOLDEN/Module-Build-0.3603.tar.gz', 'FILES' => q[cpan/Module-Build], 'EXCLUDED' => [ qw{ t/par.t t/signature.t }, qr!^contrib/!, qr!^devtools! ], diff --git a/cpan/Module-Build/Changes b/cpan/Module-Build/Changes index 19f401cf12..ad9195c545 100644 --- a/cpan/Module-Build/Changes +++ b/cpan/Module-Build/Changes @@ -1,5 +1,41 @@ Revision history for Perl extension Module::Build. +0.3603 - Mon Jan 18 22:28:59 EST 2010 + +(Oops, I released the last one before I realized this should have been +fixed along with it.) + + Bug fixes: + + - Module::Build::Compat would croak on distibutions that set requires + 'perl' to a dotted decimal like '5.6.2'. We now skip that key + since it doesn't go into PREREQ_PM and we numify it properly for + 'use 5.006002' in the generated Makefile.PL (RT#53409) + [David Golden, adapted from patch by G. Allen Morris III] + +0.3602 - Mon Jan 18 22:09:54 EST 2010 + + Bug fixes: + + - Fix failures in t/properties/needs_compiler.t when $ENV{CC} is set + (RT#53296) [David Golden, adapted from patch by Jens Rehsack] + +0.3601 - Mon Dec 21 14:39:33 EST 2009 + + Bug fixes: + + - When the currently running Module::Build is not the same as the one + that created the Build file, there is now a warning rather than a fatal + error. This helps installation of dependency chains where a dependency + might configure_requires a new Module::Build after Build.PL was already + run for an earlier distribution. [David Golden, on advice of Matt Trout] + + Other: + + - t/bundle_inc.t fails in odd ways. This test of an experimental feature + should not prevent users from installing Module::Build, so this test + now skips unless $ENV{MB_TEST_EXPERIMENTAL} is true + 0.36 - Sun Dec 20 15:02:38 EST 2009 No changes from 0.35_15 other than the version number. diff --git a/cpan/Module-Build/lib/Module/Build.pm b/cpan/Module-Build/lib/Module/Build.pm index 517a4a624b..356fd42a14 100644 --- a/cpan/Module-Build/lib/Module/Build.pm +++ b/cpan/Module-Build/lib/Module/Build.pm @@ -15,7 +15,7 @@ use Module::Build::Base; use vars qw($VERSION @ISA); @ISA = qw(Module::Build::Base); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; # Okay, this is the brute-force method of finding out what kind of @@ -750,7 +750,8 @@ false to prevent the custom resource file from being loaded. Suppresses the check upon startup that the version of Module::Build we're now running under is the same version that was initially invoked when building the distribution (i.e. when the C<Build.PL> script was -first run). Use with caution. +first run). As of 0.3601, a mismatch results in a warning instead of +a fatal error, so this option effectively just suppresses the warning. =item debug diff --git a/cpan/Module-Build/lib/Module/Build/Base.pm b/cpan/Module-Build/lib/Module/Build/Base.pm index afecaace98..5bd8ec72a5 100644 --- a/cpan/Module-Build/lib/Module/Build/Base.pm +++ b/cpan/Module-Build/lib/Module/Build/Base.pm @@ -4,7 +4,7 @@ package Module::Build::Base; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; BEGIN { require 5.00503 } @@ -103,10 +103,13 @@ sub resume { unless ($self->allow_mb_mismatch) { my $mb_version = $Module::Build::VERSION; - die(" * ERROR: Configuration was initially created with Module::Build version '$self->{properties}{mb_version}',\n". - " but we are now using version '$mb_version'. Please re-run the Build.PL or Makefile.PL script,\n". - " or use --allow_mb_mismatch 1 to skip this version check.\n") - if $mb_version ne $self->{properties}{mb_version}; + if ( $mb_version ne $self->{properties}{mb_version} ) { + $self->log_warn(<<"MISMATCH"); +* WARNING: Configuration was initially created with Module::Build + version '$self->{properties}{mb_version}' but we are now using version '$mb_version'. + If errors occur, you must re-run the Build.PL or Makefile.PL script. +MISMATCH + } } $self->{invoked_action} = $self->{action} ||= 'build'; diff --git a/cpan/Module-Build/lib/Module/Build/Compat.pm b/cpan/Module-Build/lib/Module/Build/Compat.pm index 4130917765..95d695f6f5 100644 --- a/cpan/Module-Build/lib/Module/Build/Compat.pm +++ b/cpan/Module-Build/lib/Module/Build/Compat.pm @@ -2,7 +2,7 @@ package Module::Build::Compat; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; use File::Basename (); use File::Spec; @@ -10,6 +10,7 @@ use IO::File; use Config; use Module::Build; use Module::Build::ModuleInfo; +use Module::Build::Version; use Data::Dumper; my %convert_installdirs = ( @@ -76,6 +77,7 @@ sub _merge_prereq { # validate formats for my $p ( $req, $breq ) { for my $k (keys %$p) { + next if $k eq 'perl'; die "Prereq '$p->{$k}' for '$k' is not supported by Module::Build::Compat\n" unless _simple_prereq($p->{$k}); } @@ -124,7 +126,8 @@ HERE # Makefile.PL my $requires = $build->requires; if ( my $minimum_perl = $requires->{perl} ) { - print {$fh} "require $minimum_perl;\n"; + my $min_ver = Module::Build::Version->new($minimum_perl)->numify; + print {$fh} "require $min_ver;\n"; } # If a *bundled* custom subclass is being used, make sure we add its diff --git a/cpan/Module-Build/lib/Module/Build/Config.pm b/cpan/Module-Build/lib/Module/Build/Config.pm index ab6683154d..997960073f 100644 --- a/cpan/Module-Build/lib/Module/Build/Config.pm +++ b/cpan/Module-Build/lib/Module/Build/Config.pm @@ -2,7 +2,7 @@ package Module::Build::Config; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Config; diff --git a/cpan/Module-Build/lib/Module/Build/Cookbook.pm b/cpan/Module-Build/lib/Module/Build/Cookbook.pm index 8a52f41acb..a68dca540a 100644 --- a/cpan/Module-Build/lib/Module/Build/Cookbook.pm +++ b/cpan/Module-Build/lib/Module/Build/Cookbook.pm @@ -1,7 +1,7 @@ package Module::Build::Cookbook; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; =head1 NAME diff --git a/cpan/Module-Build/lib/Module/Build/Dumper.pm b/cpan/Module-Build/lib/Module/Build/Dumper.pm index 7ab85c0e7d..2a9bad1b63 100644 --- a/cpan/Module-Build/lib/Module/Build/Dumper.pm +++ b/cpan/Module-Build/lib/Module/Build/Dumper.pm @@ -1,7 +1,7 @@ package Module::Build::Dumper; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; # This is just a split-out of a wrapper function to do Data::Dumper # stuff "the right way". See: diff --git a/cpan/Module-Build/lib/Module/Build/ModuleInfo.pm b/cpan/Module-Build/lib/Module/Build/ModuleInfo.pm index 4730d794cc..053ae18d52 100644 --- a/cpan/Module-Build/lib/Module/Build/ModuleInfo.pm +++ b/cpan/Module-Build/lib/Module/Build/ModuleInfo.pm @@ -8,7 +8,7 @@ package Module::Build::ModuleInfo; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use File::Spec; diff --git a/cpan/Module-Build/lib/Module/Build/Notes.pm b/cpan/Module-Build/lib/Module/Build/Notes.pm index 81865bcc92..cf85d4ee3d 100644 --- a/cpan/Module-Build/lib/Module/Build/Notes.pm +++ b/cpan/Module-Build/lib/Module/Build/Notes.pm @@ -4,7 +4,7 @@ package Module::Build::Notes; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Data::Dumper; use IO::File; diff --git a/cpan/Module-Build/lib/Module/Build/PPMMaker.pm b/cpan/Module-Build/lib/Module/Build/PPMMaker.pm index b616a4d663..977b4370d9 100644 --- a/cpan/Module-Build/lib/Module/Build/PPMMaker.pm +++ b/cpan/Module-Build/lib/Module/Build/PPMMaker.pm @@ -5,7 +5,7 @@ use Config; use vars qw($VERSION); use IO::File; -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; # This code is mostly borrowed from ExtUtils::MM_Unix 6.10_03, with a diff --git a/cpan/Module-Build/lib/Module/Build/Platform/Amiga.pm b/cpan/Module-Build/lib/Module/Build/Platform/Amiga.pm index 48919f56db..9356325c9f 100644 --- a/cpan/Module-Build/lib/Module/Build/Platform/Amiga.pm +++ b/cpan/Module-Build/lib/Module/Build/Platform/Amiga.pm @@ -2,7 +2,7 @@ package Module::Build::Platform::Amiga; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Module::Build::Base; diff --git a/cpan/Module-Build/lib/Module/Build/Platform/Default.pm b/cpan/Module-Build/lib/Module/Build/Platform/Default.pm index f807cdd4a8..8d46fc47d5 100644 --- a/cpan/Module-Build/lib/Module/Build/Platform/Default.pm +++ b/cpan/Module-Build/lib/Module/Build/Platform/Default.pm @@ -2,7 +2,7 @@ package Module::Build::Platform::Default; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Module::Build::Base; diff --git a/cpan/Module-Build/lib/Module/Build/Platform/EBCDIC.pm b/cpan/Module-Build/lib/Module/Build/Platform/EBCDIC.pm index 710c10b87e..140e27872a 100644 --- a/cpan/Module-Build/lib/Module/Build/Platform/EBCDIC.pm +++ b/cpan/Module-Build/lib/Module/Build/Platform/EBCDIC.pm @@ -2,7 +2,7 @@ package Module::Build::Platform::EBCDIC; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Module::Build::Base; diff --git a/cpan/Module-Build/lib/Module/Build/Platform/MPEiX.pm b/cpan/Module-Build/lib/Module/Build/Platform/MPEiX.pm index a67a5c9ce6..66bbdc95da 100644 --- a/cpan/Module-Build/lib/Module/Build/Platform/MPEiX.pm +++ b/cpan/Module-Build/lib/Module/Build/Platform/MPEiX.pm @@ -2,7 +2,7 @@ package Module::Build::Platform::MPEiX; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Module::Build::Base; diff --git a/cpan/Module-Build/lib/Module/Build/Platform/MacOS.pm b/cpan/Module-Build/lib/Module/Build/Platform/MacOS.pm index e89754c45b..b80781bd95 100644 --- a/cpan/Module-Build/lib/Module/Build/Platform/MacOS.pm +++ b/cpan/Module-Build/lib/Module/Build/Platform/MacOS.pm @@ -2,7 +2,7 @@ package Module::Build::Platform::MacOS; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Module::Build::Base; use vars qw(@ISA); diff --git a/cpan/Module-Build/lib/Module/Build/Platform/RiscOS.pm b/cpan/Module-Build/lib/Module/Build/Platform/RiscOS.pm index c861d9d7bc..95e3151587 100644 --- a/cpan/Module-Build/lib/Module/Build/Platform/RiscOS.pm +++ b/cpan/Module-Build/lib/Module/Build/Platform/RiscOS.pm @@ -2,7 +2,7 @@ package Module::Build::Platform::RiscOS; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Module::Build::Base; diff --git a/cpan/Module-Build/lib/Module/Build/Platform/Unix.pm b/cpan/Module-Build/lib/Module/Build/Platform/Unix.pm index 2eebc80ac5..0be5fc7df0 100644 --- a/cpan/Module-Build/lib/Module/Build/Platform/Unix.pm +++ b/cpan/Module-Build/lib/Module/Build/Platform/Unix.pm @@ -2,7 +2,7 @@ package Module::Build::Platform::Unix; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Module::Build::Base; diff --git a/cpan/Module-Build/lib/Module/Build/Platform/VMS.pm b/cpan/Module-Build/lib/Module/Build/Platform/VMS.pm index c110a8da9d..f406b7afb6 100644 --- a/cpan/Module-Build/lib/Module/Build/Platform/VMS.pm +++ b/cpan/Module-Build/lib/Module/Build/Platform/VMS.pm @@ -2,7 +2,7 @@ package Module::Build::Platform::VMS; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Module::Build::Base; use Config; diff --git a/cpan/Module-Build/lib/Module/Build/Platform/VOS.pm b/cpan/Module-Build/lib/Module/Build/Platform/VOS.pm index 1ec5fe6d93..001a9507d4 100644 --- a/cpan/Module-Build/lib/Module/Build/Platform/VOS.pm +++ b/cpan/Module-Build/lib/Module/Build/Platform/VOS.pm @@ -2,7 +2,7 @@ package Module::Build::Platform::VOS; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Module::Build::Base; diff --git a/cpan/Module-Build/lib/Module/Build/Platform/Windows.pm b/cpan/Module-Build/lib/Module/Build/Platform/Windows.pm index 38f6662497..1e248a75c5 100644 --- a/cpan/Module-Build/lib/Module/Build/Platform/Windows.pm +++ b/cpan/Module-Build/lib/Module/Build/Platform/Windows.pm @@ -2,7 +2,7 @@ package Module::Build::Platform::Windows; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Config; diff --git a/cpan/Module-Build/lib/Module/Build/Platform/aix.pm b/cpan/Module-Build/lib/Module/Build/Platform/aix.pm index 1b932e0c98..dab064805d 100644 --- a/cpan/Module-Build/lib/Module/Build/Platform/aix.pm +++ b/cpan/Module-Build/lib/Module/Build/Platform/aix.pm @@ -2,7 +2,7 @@ package Module::Build::Platform::aix; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Module::Build::Platform::Unix; diff --git a/cpan/Module-Build/lib/Module/Build/Platform/cygwin.pm b/cpan/Module-Build/lib/Module/Build/Platform/cygwin.pm index 484490fcf0..37c59ad0ce 100644 --- a/cpan/Module-Build/lib/Module/Build/Platform/cygwin.pm +++ b/cpan/Module-Build/lib/Module/Build/Platform/cygwin.pm @@ -2,7 +2,7 @@ package Module::Build::Platform::cygwin; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Module::Build::Platform::Unix; diff --git a/cpan/Module-Build/lib/Module/Build/Platform/darwin.pm b/cpan/Module-Build/lib/Module/Build/Platform/darwin.pm index 4d9986fabb..c2f3126c44 100644 --- a/cpan/Module-Build/lib/Module/Build/Platform/darwin.pm +++ b/cpan/Module-Build/lib/Module/Build/Platform/darwin.pm @@ -2,7 +2,7 @@ package Module::Build::Platform::darwin; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Module::Build::Platform::Unix; diff --git a/cpan/Module-Build/lib/Module/Build/Platform/os2.pm b/cpan/Module-Build/lib/Module/Build/Platform/os2.pm index ae0b67aa13..be164ae980 100644 --- a/cpan/Module-Build/lib/Module/Build/Platform/os2.pm +++ b/cpan/Module-Build/lib/Module/Build/Platform/os2.pm @@ -2,7 +2,7 @@ package Module::Build::Platform::os2; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Module::Build::Platform::Unix; diff --git a/cpan/Module-Build/lib/Module/Build/PodParser.pm b/cpan/Module-Build/lib/Module/Build/PodParser.pm index 7259d8bdac..225450c6b9 100644 --- a/cpan/Module-Build/lib/Module/Build/PodParser.pm +++ b/cpan/Module-Build/lib/Module/Build/PodParser.pm @@ -2,7 +2,7 @@ package Module::Build::PodParser; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use vars qw(@ISA); diff --git a/cpan/Module-Build/lib/inc/latest.pm b/cpan/Module-Build/lib/inc/latest.pm index 9323452256..fc9e2e94bf 100644 --- a/cpan/Module-Build/lib/inc/latest.pm +++ b/cpan/Module-Build/lib/inc/latest.pm @@ -1,7 +1,7 @@ package inc::latest; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use Carp; diff --git a/cpan/Module-Build/lib/inc/latest/private.pm b/cpan/Module-Build/lib/inc/latest/private.pm index c194fbdb97..572ae8a82c 100644 --- a/cpan/Module-Build/lib/inc/latest/private.pm +++ b/cpan/Module-Build/lib/inc/latest/private.pm @@ -1,7 +1,7 @@ package inc::latest::private; use strict; use vars qw($VERSION); -$VERSION = '0.36'; +$VERSION = '0.3603'; $VERSION = eval $VERSION; use File::Spec; diff --git a/cpan/Module-Build/t/basic.t b/cpan/Module-Build/t/basic.t index 74c50b6178..5d76ffaebd 100644 --- a/cpan/Module-Build/t/basic.t +++ b/cpan/Module-Build/t/basic.t @@ -146,7 +146,7 @@ $dist->chdir_in; is $mb->args('dee'), 'goo'; is $mb->destdir, 'yo'; my %runtime = $mb->runtime_params; - is_deeply \%runtime, + is_deeply \%runtime, { verbose => 1, destdir => 'yo', diff --git a/cpan/Module-Build/t/bundle_inc.t b/cpan/Module-Build/t/bundle_inc.t index b6b0a77ff5..e974c26107 100644 --- a/cpan/Module-Build/t/bundle_inc.t +++ b/cpan/Module-Build/t/bundle_inc.t @@ -18,6 +18,9 @@ blib_load('Module::Build::ConfigData'); if ( $ENV{PERL_CORE} ) { plan skip_all => 'bundle_inc tests will never succeed in PERL_CORE'; } +elsif ( ! $ENV{MB_TEST_EXPERIMENTAL} ) { + plan skip_all => '$ENV{MB_TEST_EXPERIMENTAL} is not set'; +} elsif ( ! MBTest::check_EUI() ) { plan skip_all => 'ExtUtils::Installed takes too long on your system'; } diff --git a/cpan/Module-Build/t/compat.t b/cpan/Module-Build/t/compat.t index f84b79b744..0a820a936d 100644 --- a/cpan/Module-Build/t/compat.t +++ b/cpan/Module-Build/t/compat.t @@ -67,7 +67,7 @@ test_makefile_types(); # Test with requires and PL_files my $distname = $dist->name; -$dist->change_build_pl({ +$dist->change_build_pl({ module_name => $distname, license => 'perl', requires => { @@ -104,7 +104,7 @@ test_makefile_types( ###################### -$dist->change_build_pl({ +$dist->change_build_pl({ module_name => $distname, license => 'perl', }); @@ -140,7 +140,7 @@ ok $mb, "Module::Build->new_from_context"; }); foreach my $style ('passthrough', 'small') { create_makefile_pl($style, $foo_builder); - + # Should fail with "can't find Foo/Builder.pm" my $result; my ($stdout, $stderr ) = stdout_stderr_of (sub { @@ -149,7 +149,7 @@ ok $mb, "Module::Build->new_from_context"; ok ! $result, "Makefile.PL failed"; like $stderr, qr{Foo/Builder.pm}, "custom builder wasn't found"; } - + # Now make sure it can actually work. my $bar_builder; stdout_stderr_of( sub { @@ -243,7 +243,7 @@ ok $mb, "Module::Build->new_from_context"; if ExtUtils::Install->VERSION < 1.32; for my $c (@cases) { - my @make_args = @{$c->{args}}; + my @make_args = @{$c->{args}}; if ($is_vms_mms) { # VMS MMK/MMS macros use different syntax. $make_args[0] = '/macro=("' . join('","',@make_args) . '")'; pop @make_args while scalar(@make_args) > 1; @@ -284,7 +284,7 @@ ok $mb, "Module::Build->new_from_context"; my $b2 = Module::Build->current; ok $b2->install_base, "install_base set"; unlike $b2->install_base, qr/^~/, "Tildes should be expanded"; - + stdout_stderr_of( sub { $mb->do_system(@make, 'realclean'); } ); ok ! -e $makefile, "$makefile shouldn't exist"; @@ -341,7 +341,7 @@ sub test_makefile_types { $opts{PL_files} ||= {}; foreach my $type (@makefile_types) { - # Create M::B instance + # Create M::B instance my $mb; stdout_stderr_of( sub { $mb = Module::Build->new_from_context; @@ -355,7 +355,7 @@ sub test_makefile_types { test_makefile_creation($mb); test_makefile_prereq_pm( _merge_prereqs($opts{requires}, $opts{build_requires}) ); test_makefile_pl_files( $opts{PL_files} ) if $type eq 'traditional'; - + my ($output,$success); # Capture output to keep our STDOUT clean $output = stdout_stderr_of( sub { @@ -373,7 +373,7 @@ sub test_makefile_types { }); ok $success, "make test ran without error"; like uc $output, qr{DONE\.|SUCCESS}, "make test output indicated success"; - + $output = stdout_stderr_of( sub { $success = $mb->do_system(@make, 'realclean'); }); @@ -381,7 +381,7 @@ sub test_makefile_types { # Try again with some Makefile.PL arguments test_makefile_creation($mb, [], 'INSTALLDIRS=vendor', 'realclean'); - + # Try again using distclean test_makefile_creation($mb, [], '', 'distclean'); @@ -392,7 +392,7 @@ sub test_makefile_types { sub test_makefile_creation { my ($build, $preargs, $postargs, $cleanup) = @_; - + my ($output, $result); # capture output to avoid polluting our test output $output = stdout_stderr_of( sub { @@ -404,7 +404,7 @@ sub test_makefile_creation { } ok $result, $label; ok -e $makefile, "$makefile exists"; - + if ($cleanup) { # default to 'realclean' unless we recognize the clean method $cleanup = 'realclean' unless $cleanup =~ /^(dist|real)clean$/; @@ -457,7 +457,7 @@ sub test_makefile_pl_requires_perl { } sub find_params_in_makefile { - my $fh = IO::File->new( $makefile, 'r' ) + my $fh = IO::File->new( $makefile, 'r' ) or die "Can't read $makefile: $!"; local($/) = "\n"; diff --git a/cpan/Module-Build/t/destinations.t b/cpan/Module-Build/t/destinations.t index 07247a32ba..2b9aba6ed2 100644 --- a/cpan/Module-Build/t/destinations.t +++ b/cpan/Module-Build/t/destinations.t @@ -242,14 +242,14 @@ $mb->prefix(undef); my %test_config; foreach my $type (keys %$defaults) { my $prefix = shift @prefixes || [qw(foo bar)]; - $test_config{$type} = catdir(File::Spec->rootdir, @$prefix, + $test_config{$type} = catdir(File::Spec->rootdir, @$prefix, @{$defaults->{$type}}); } # Poke at the innards of MB to change the default install locations. my $old = $mb->install_sets->{site}; $mb->install_sets->{site} = \%test_config; - $mb->config(siteprefixexp => catdir(File::Spec->rootdir, + $mb->config(siteprefixexp => catdir(File::Spec->rootdir, 'wierd', 'prefix')); my $prefix = catdir('another', 'prefix'); diff --git a/cpan/Module-Build/t/ext.t b/cpan/Module-Build/t/ext.t index 6101bccd16..3c60a656e7 100644 --- a/cpan/Module-Build/t/ext.t +++ b/cpan/Module-Build/t/ext.t @@ -4,7 +4,7 @@ use strict; use lib 't/lib'; use MBTest; -my @unix_splits = +my @unix_splits = ( { q{one t'wo th'ree f"o\"ur " "five" } => [ 'one', 'two three', 'fo"ur ', 'five' ] }, { q{ foo bar } => [ 'foo', 'bar' ] }, @@ -13,7 +13,7 @@ my @unix_splits = { qq{one\\\ntwo} => [ "one\ntwo" ] }, # TODO ); -my @win_splits = +my @win_splits = ( { 'a" "b\\c" "d' => [ 'a b\c d' ] }, { '"a b\\c d"' => [ 'a b\c d' ] }, diff --git a/cpan/Module-Build/t/files.t b/cpan/Module-Build/t/files.t index cf822fb091..625a473974 100644 --- a/cpan/Module-Build/t/files.t +++ b/cpan/Module-Build/t/files.t @@ -19,19 +19,19 @@ my $mb = Module::Build->new_from_context; { # Make sure copy_if_modified() can handle spaces in filenames - + my @tmp; push @tmp, MBTest->tmpdir for (0 .. 1); - + my $filename = 'file with spaces.txt'; - + my $file = File::Spec->catfile($tmp[0], $filename); my $fh = IO::File->new($file, '>') or die "Can't create $file: $!"; print $fh "Foo\n"; $fh->close; ok -e $file; - - + + my $file2 = $mb->copy_if_modified(from => $file, to_dir => $tmp[1]); ok $file2; ok -e $file2; @@ -41,7 +41,7 @@ my $mb = Module::Build->new_from_context; # Try some dir_contains() combinations my $first = File::Spec->catdir('', 'one', 'two'); my $second = File::Spec->catdir('', 'one', 'two', 'three'); - + ok( Module::Build->dir_contains($first, $second) ); } diff --git a/cpan/Module-Build/t/help.t b/cpan/Module-Build/t/help.t index 2bf34c8d50..0534c92956 100644 --- a/cpan/Module-Build/t/help.t +++ b/cpan/Module-Build/t/help.t @@ -51,7 +51,7 @@ my $mb = Module::Build->subclass( sub ACTION_bar { die "barey" } sub ACTION_baz { die "bazey" } - # guess we can have extra pod later + # guess we can have extra pod later =over @@ -211,7 +211,7 @@ my $mb = Module::Build->subclass( sub ACTION_baz { die "bazey" } sub ACTION_batz { die "batzey" } - # guess we can have extra pod later + # guess we can have extra pod later # Though, I do wonder whether we should allow them to mix... # maybe everything should have to be head2? @@ -243,7 +243,7 @@ my %also = ( bar => "\n=head3 bears\n\nBe careful with bears.\n", baz => "\n=head4 What's a baz\\?\n", ); - + foreach my $action (qw(foo bar baz)) { my $doc = $mb->get_action_docs($action); ok($doc, "got doc for '$action'"); diff --git a/cpan/Module-Build/t/install.t b/cpan/Module-Build/t/install.t index 66cdd5c94a..ffe8afca01 100644 --- a/cpan/Module-Build/t/install.t +++ b/cpan/Module-Build/t/install.t @@ -61,15 +61,15 @@ $mb->add_to_cleanup($destdir); { eval {$mb->dispatch('install', destdir => $destdir)}; is $@, ''; - + my @libdir = strip_volume( $mb->install_destination('lib') ); my $install_to = File::Spec->catfile($destdir, @libdir, $dist->name ) . '.pm'; file_exists($install_to); - + local @INC = (@INC, File::Spec->catdir($destdir, @libdir)); eval "require @{[$dist->name]}"; is $@, ''; - + # Make sure there's a packlist installed my $archdir = $mb->install_destination('arch'); my @dirs = strip_volume($archdir); @@ -106,19 +106,19 @@ $mb->add_to_cleanup($destdir); { # Test the ConfigData stuff - + $mb->config_data(foo => 'bar'); $mb->features(baz => 1); $mb->auto_features(auto_foo => {requires => {'File::Spec' => 0}}); eval {$mb->dispatch('install', destdir => $destdir)}; is $@, ''; - + my @libdir = strip_volume( $mb->install_destination('lib') ); local @INC = (@INC, File::Spec->catdir($destdir, @libdir)); eval "require @{[$dist->name]}::ConfigData"; is $mb->feature('auto_foo'), 1; - + SKIP: { skip $@, 5 if @_; @@ -153,7 +153,7 @@ is $@, ''; my $libdir = File::Spec->catdir('', 'foo', 'lib'); eval {$mb->run_perl_script('Build.PL', [], ['--install_path', "lib=$libdir"])}; is $@, ''; - + my $cmd = 'Build'; $cmd .= ".COM" if $^O eq 'VMS'; eval {$mb->run_perl_script($cmd, [], ['install', '--destdir', $destdir])}; @@ -165,10 +165,10 @@ is $@, ''; eval {$mb->run_perl_script($cmd, [], ['install', '--destdir', $destdir, '--install_base', $basedir])}; is $@, ''; - + $install_to = File::Spec->catfile($destdir, $libdir, $dist->name ) . '.pm'; is -e $install_to, 1, "Look for file at $install_to"; - + eval {$mb->dispatch('realclean')}; is $@, ''; } @@ -214,11 +214,11 @@ Simple Man <simple@example.com> $expect = lc($expect) if $^O eq 'VMS'; is $pods->{$expect}, $expect; - + my $pms = $mb->_find_file_by_type('awefawef', 'lib'); ok $pms; is keys %$pms, 0; - + $pms = $mb->_find_file_by_type('pod', 'awefawef'); ok $pms; is keys %$pms, 0; diff --git a/cpan/Module-Build/t/manifypods.t b/cpan/Module-Build/t/manifypods.t index 5947646d13..aa33ffccd0 100644 --- a/cpan/Module-Build/t/manifypods.t +++ b/cpan/Module-Build/t/manifypods.t @@ -118,7 +118,7 @@ while (my ($from, $v) = each %distro) { ok ! $mb->contains_pod($from), "$from should not contain POD"; next; } - + my $to = File::Spec->catfile('blib', ($from =~ /^[\.\/\[]*lib/ ? 'libdoc' : 'bindoc'), $v); ok $mb->contains_pod($from), "$from should contain POD"; ok -e $to, "Created $to manpage"; diff --git a/cpan/Module-Build/t/metadata2.t b/cpan/Module-Build/t/metadata2.t index 954b6589a0..a0fdd604d1 100644 --- a/cpan/Module-Build/t/metadata2.t +++ b/cpan/Module-Build/t/metadata2.t @@ -42,12 +42,12 @@ SKIP: { my $provides; # Used a bunch of times below -my $pod_text = <<'---'; +my $pod_text = <<'---'; =pod =head1 NAME -Simple - A simple module +Simple - A simple module =head1 AUTHOR @@ -77,11 +77,11 @@ $dist->regen( clean => 1 ); ok( -e "lib/Simple.pm", "Creating Simple.pm" ); my $mb = Module::Build->new_from_context; $mb->do_create_readme; -like( slurp("README"), qr/NAME/, +like( slurp("README"), qr/NAME/, "Generating README from .pm"); -is( $mb->dist_author->[0], 'Simple Simon <simon@simple.sim>', +is( $mb->dist_author->[0], 'Simple Simon <simon@simple.sim>', "Extracting AUTHOR from .pm"); -is( $mb->dist_abstract, "A simple module", +is( $mb->dist_abstract, "A simple module", "Extracting abstract from .pm"); # .pm File with pod in separate file @@ -99,12 +99,12 @@ ok( -e "lib/Simple.pod", "Creating Simple.pod" ); $mb = Module::Build->new_from_context; $mb->do_create_readme; like( slurp("README"), qr/NAME/, "Generating README from .pod"); -is( $mb->dist_author->[0], 'Simple Simon <simon@simple.sim>', +is( $mb->dist_author->[0], 'Simple Simon <simon@simple.sim>', "Extracting AUTHOR from .pod"); -is( $mb->dist_abstract, "A simple module", +is( $mb->dist_abstract, "A simple module", "Extracting abstract from .pod"); -# .pm File with pod and separate pod file +# .pm File with pod and separate pod file # $dist->change_file( 'lib/Simple.pm', <<'---' ); diff --git a/cpan/Module-Build/t/pod_parser.t b/cpan/Module-Build/t/pod_parser.t index 64d4c75348..c47d738166 100644 --- a/cpan/Module-Build/t/pod_parser.t +++ b/cpan/Module-Build/t/pod_parser.t @@ -10,14 +10,14 @@ blib_load('Module::Build::PodParser'); { package IO::StringBased; - + sub TIEHANDLE { my ($class, $string) = @_; return bless { data => [ map "$_\n", split /\n/, $string], }, $class; } - + sub READLINE { shift @{ shift()->{data} }; } @@ -59,7 +59,7 @@ EOF my $pp = Module::Build::PodParser->new(fh => \*FH); ok $pp, 'object created'; - + is_deeply $pp->get_author, [], 'author'; is $pp->get_abstract, 'Perl extension for blah blah blah', 'abstract'; } diff --git a/cpan/Module-Build/t/properties/needs_compiler.t b/cpan/Module-Build/t/properties/needs_compiler.t index f298e82739..f616dfc958 100644 --- a/cpan/Module-Build/t/properties/needs_compiler.t +++ b/cpan/Module-Build/t/properties/needs_compiler.t @@ -106,6 +106,9 @@ is( $mb->build_requires->{'ExtUtils::CBuilder'}, 0.2, # falsify compiler and test error handling #--------------------------------------------------------------------------# +# clear $ENV{CC} so we are sure to fail to find our fake compiler :-) +local $ENV{CC}; + my $err = stderr_of( sub { $mb = $dist->new_from_context( config => { cc => "adfasdfadjdjk" } ) }); diff --git a/cpan/Module-Build/t/script_dist.t b/cpan/Module-Build/t/script_dist.t index 02faca0132..7fd82d9547 100644 --- a/cpan/Module-Build/t/script_dist.t +++ b/cpan/Module-Build/t/script_dist.t @@ -15,7 +15,7 @@ blib_load('Module::Build::ConfigData'); my $dist = DistGen->new(dir => MBTest->tmpdir); $dist->add_file('bin/foo', undent(<<' ---')); #!/usr/bin/perl - + package bin::foo; $VERSION = 0.01; diff --git a/cpan/Module-Build/t/test_type.t b/cpan/Module-Build/t/test_type.t index fe4d599d72..bdbf0cd82b 100644 --- a/cpan/Module-Build/t/test_type.t +++ b/cpan/Module-Build/t/test_type.t @@ -21,7 +21,7 @@ my $dist = DistGen->new( dir => $tmp ); $dist->add_file('t/special_ext.st', <<'---' ); -#!perl +#!perl use Test::More tests => 2; ok(1, 'first test in special_ext'); ok(1, 'second test in special_ext'); @@ -37,7 +37,7 @@ $dist->chdir_in; $::x = 0; my $mb = Module::Build->subclass( code => q# - sub ACTION_testspecial { + sub ACTION_testspecial { $::x++; shift->generic_test(type => 'special'); } diff --git a/cpan/Module-Build/t/test_types.t b/cpan/Module-Build/t/test_types.t index d88e215aa3..bcb58c444a 100644 --- a/cpan/Module-Build/t/test_types.t +++ b/cpan/Module-Build/t/test_types.t @@ -11,20 +11,20 @@ use DistGen; my $dist = DistGen->new()->chdir_in; $dist->add_file('t/special_ext.st', <<'---'); -#!perl +#!perl use Test::More tests => 2; ok(1, 'first test in special_ext'); ok(1, 'second test in special_ext'); --- $dist->add_file('t/another_ext.at', <<'---'); -#!perl +#!perl use Test::More tests => 2; ok(1, 'first test in another_ext'); ok(1, 'second test in another_ext'); --- $dist->add_file('t/foo.txt', <<'---'); -#!perl +#!perl use Test::More tests => 1; ok 0, "don't run this non-test file"; die "don't run this non-test file"; @@ -35,11 +35,11 @@ $dist->regen; my $mb = Module::Build->subclass( code => q# - sub ACTION_testspecial { + sub ACTION_testspecial { shift->generic_test(type => 'special'); } - sub ACTION_testanother { + sub ACTION_testanother { shift->generic_test(type => 'another'); } # @@ -99,7 +99,7 @@ is(scalar(@{[$all_output =~ m/ALL TESTS SUCCESSFUL\./mg]}), 1); $dist->revert; $dist->add_file('t/foo/special.st', <<'---'); -#!perl +#!perl use Test::More tests => 2; ok(1, 'first test in special_ext'); ok(1, 'second test in special_ext'); @@ -113,11 +113,11 @@ $dist->regen; my $mb = Module::Build->subclass( code => q# - sub ACTION_testspecial { + sub ACTION_testspecial { shift->generic_test(type => 'special'); } - sub ACTION_testanother { + sub ACTION_testanother { shift->generic_test(type => 'another'); } # diff --git a/cpan/Module-Build/t/tilde.t b/cpan/Module-Build/t/tilde.t index 692ade0c8a..a5ed79083b 100644 --- a/cpan/Module-Build/t/tilde.t +++ b/cpan/Module-Build/t/tilde.t @@ -55,7 +55,7 @@ SKIP: { is( run_sample( $p => '~/foo' )->$p(), "$home/foo" ); is( run_sample( $p => '~/ foo')->$p(), "$home/ foo" ); - + is( run_sample( $p => '~/fo o')->$p(), "$home/fo o" ); is( run_sample( $p => 'foo~' )->$p(), 'foo~' ); @@ -97,7 +97,7 @@ SKIP: { my @info = eval { getpwuid $> }; skip "No home directory for tilde-expansion tests", 1 if $@; my ($me, $home) = @info[0,7]; - + my $expected = "$home/foo"; if ($^O eq 'VMS') { |