Mailinglist Archive: opensuse-commit (2092 mails)

< Previous Next >
commit perl-Test-Base
  • From: root@xxxxxxxxxxxxxxx (h_root)
  • Date: Tue, 07 Aug 2007 09:27:58 +0200
  • Message-id: <20070807072758.6E6E2678332@xxxxxxxxxxxxxxx>

Hello community,

here is the log from the commit of package perl-Test-Base
checked in at Tue Aug 7 09:27:58 CEST 2007.

--------
--- perl-Test-Base/perl-Test-Base.changes       2006-10-25 15:34:24.000000000 +0200
+++ /mounts/work_src_done/STABLE/perl-Test-Base/perl-Test-Base.changes  2007-08-07 01:30:38.967049000 +0200
@@ -1,0 +2,6 @@
+Tue Jul 24 15:32:05 CEST 2007 - cwh@xxxxxxx
+
+- version 0.54: Make dependency on Filter::Util::Call explicit in Makefile.PL
+- version 0.53: Changes from miyagawa and crew
+
+-------------------------------------------------------------------
@@ -4,25 +10,15 @@
-version: 0.52
-date:    Mon Jun 19 10:44:53 PDT 2006
-changes:
-- Add use_ok to exports
-
-version: 0.51
-date:    Fri Jun 16 13:05:22 PDT 2006
-changes:
-- Remove build-requires dep of Spiffy for Module::Install::TestBase
-- Add in a patch from the good folk at Socialtext.
-
-version: 0.50
-date:    Mon Jan 30 10:52:52 PST 2006
-changes:
-- No change. 0.49 got borked on the way to CPAN
-
-version: 0.49
-date:    Mon Jan 30 10:52:48 PST 2006
-changes:
-- Added Module::Install::TestBase
-
-version: 0.48
-date:    Sun Jan 29 10:19:46 PST 2006
-changes:
-- Fixed test failures on windows
+- version: 0.52
+  - Add use_ok to exports
+
+- version: 0.51
+  - Remove build-requires dep of Spiffy for Module::Install::TestBase
+  - Add in a patch from the good folk at Socialtext.
+
+- version: 0.50
+  - No change. 0.49 got borked on the way to CPAN
+
+- version: 0.49
+  - Added Module::Install::TestBase
+
+- version: 0.48
+  - Fixed test failures on windows

Old:
----
  Test-Base-0.52.tar.gz

New:
----
  Test-Base-0.54.tar.gz

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ perl-Test-Base.spec ++++++
--- /var/tmp/diff_new_pack.Av5605/_old  2007-08-07 09:27:43.000000000 +0200
+++ /var/tmp/diff_new_pack.Av5605/_new  2007-08-07 09:27:43.000000000 +0200
@@ -1,7 +1,7 @@
 #
-# spec file for package perl-Test-Base (Version 0.52)
+# spec file for package perl-Test-Base (Version 0.54)
 #
-# Copyright (c) 2006 SUSE LINUX Products GmbH, Nuernberg, Germany.
+# Copyright (c) 2007 SUSE LINUX Products GmbH, Nuernberg, Germany.
 # This file and all modifications and additions to the pristine
 # package are under the same license as the package itself.
 #
@@ -14,11 +14,11 @@
 BuildRequires:  perl-Spiffy >= 0.3
 URL:            http://search.cpan.org/~ingy/Test-Base
 License:        Artistic License
-Group:          Development/Languages/Perl
+Group:          Development/Libraries/Perl
 Requires:       perl = %{perl_version} perl-Spiffy >= 0.3
 Autoreqprov:    on
 Summary:        Perl module for easily writing own test framework base classes.
-Version:        0.52
+Version:        0.54
 Release:        1
 Source:         Test-Base-%{version}.tar.gz
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
@@ -26,7 +26,7 @@
 %description
 This module gives you a way to write your own test framework base class
 that is trivial. In fact it is as simple as two lines. It extends
-Test::More and exports all of its functions.
+Test::More and exports all of its functions
 
 
 
@@ -36,18 +36,17 @@
 
 %prep
 %setup -n Test-Base-%{version}
-# ---------------------------------------------------------------------------
 
 %build
 perl Makefile.PL
 make
+
+%check
 make test
-# ---------------------------------------------------------------------------
 
 %install
 rm -rf $RPM_BUILD_ROOT
 make DESTDIR=$RPM_BUILD_ROOT install_vendor
-make test
 %perl_process_packlist
 
 %clean
@@ -71,29 +70,22 @@
 %{perl_vendorarch}/auto/Test/Base/.packlist
 /var/adm/perl-modules/%{name}
 
-%changelog -n perl-Test-Base
+%changelog
+* Tue Jul 24 2007 - cwh@xxxxxxx
+- version 0.54: Make dependency on Filter::Util::Call explicit in Makefile.PL
+- version 0.53: Changes from miyagawa and crew
 * Wed Oct 25 2006 - cwh@xxxxxxx
-  version: 0.52
-  date:    Mon Jun 19 10:44:53 PDT 2006
-  changes:
-- Add use_ok to exports
-  version: 0.51
-  date:    Fri Jun 16 13:05:22 PDT 2006
-  changes:
-- Remove build-requires dep of Spiffy for Module::Install::TestBase
-- Add in a patch from the good folk at Socialtext.
-  version: 0.50
-  date:    Mon Jan 30 10:52:52 PST 2006
-  changes:
-- No change. 0.49 got borked on the way to CPAN
-  version: 0.49
-  date:    Mon Jan 30 10:52:48 PST 2006
-  changes:
-- Added Module::Install::TestBase
-  version: 0.48
-  date:    Sun Jan 29 10:19:46 PST 2006
-  changes:
-- Fixed test failures on windows
+- version: 0.52
+  - Add use_ok to exports
+- version: 0.51
+  - Remove build-requires dep of Spiffy for Module::Install::TestBase
+  - Add in a patch from the good folk at Socialtext.
+- version: 0.50
+  - No change. 0.49 got borked on the way to CPAN
+- version: 0.49
+  - Added Module::Install::TestBase
+- version: 0.48
+  - Fixed test failures on windows
 * Wed Jan 25 2006 - mls@xxxxxxx
 - converted neededforbuild to BuildRequires
 * Fri Jan 20 2006 - cwh@xxxxxxx

++++++ Test-Base-0.52.tar.gz -> Test-Base-0.54.tar.gz ++++++
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/Changes new/Test-Base-0.54/Changes
--- old/Test-Base-0.52/Changes  2006-06-19 19:46:15.000000000 +0200
+++ new/Test-Base-0.54/Changes  2007-06-28 23:23:53.000000000 +0200
@@ -1,4 +1,15 @@
 ---
+version: 0.54
+date:    Wed Nov 29 15:21:02 PST 2006
+changes:
+- Make dependency on Filter::Util::Call explicit in Makefile.PL
+  Thanks to Adriano Ferreira
+---
+version: 0.53
+date:    Wed Nov 29 15:21:02 PST 2006
+changes:
+- Changes from miyagawa and crew
+---
 version: 0.52
 date:    Mon Jun 19 10:44:53 PDT 2006
 changes:
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/inc/Module/Install/Base.pm new/Test-Base-0.54/inc/Module/Install/Base.pm
--- old/Test-Base-0.52/inc/Module/Install/Base.pm       2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/inc/Module/Install/Base.pm       2007-06-28 23:32:47.000000000 +0200
@@ -1,47 +1,52 @@
-#line 1 "inc/Module/Install/Base.pm - /Users/ingy/local/lib/perl5/site_perl/5.8.6/Module/Install/Base.pm"
+#line 1
 package Module::Install::Base;
 
+$VERSION = '0.64';
+
 # Suspend handler for "redefined" warnings
-BEGIN { my $w = $SIG{__WARN__}; $SIG{__WARN__} = sub { $w } };
+BEGIN {
+       my $w = $SIG{__WARN__};
+       $SIG{__WARN__} = sub { $w };
+}
+
+### This is the ONLY module that shouldn't have strict on
+# use strict;
 
-#line 30
+#line 41
 
 sub new {
     my ($class, %args) = @_;
 
-    foreach my $method (qw(call load)) {
+    foreach my $method ( qw(call load) ) {
         *{"$class\::$method"} = sub {
-            +shift->_top->$method(@_);
+            shift()->_top->$method(@_);
         } unless defined &{"$class\::$method"};
     }
 
-    bless(\%args, $class);
+    bless( \%args, $class );
 }
 
-#line 48
+#line 61
 
 sub AUTOLOAD {
     my $self = shift;
-
     local $@;
     my $autoload = eval { $self->_top->autoload } or return;
     goto &$autoload;
 }
 
-#line 62
+#line 76
 
 sub _top { $_[0]->{_top} }
 
-#line 73
+#line 89
 
 sub admin {
-    my $self = shift;
-    $self->_top->{admin} or Module::Install::Base::FakeAdmin->new;
+    $_[0]->_top->{admin} or Module::Install::Base::FakeAdmin->new;
 }
 
 sub is_admin {
-    my $self = shift;
-    $self->admin->VERSION;
+    $_[0]->admin->VERSION;
 }
 
 sub DESTROY {}
@@ -50,14 +55,16 @@
 
 my $Fake;
 sub new { $Fake ||= bless(\@_, $_[0]) }
+
 sub AUTOLOAD {}
-sub DESTROY {}
 
-1;
+sub DESTROY {}
 
 # Restore warning handler
-BEGIN { $SIG{__WARN__} = $SIG{__WARN__}->() };
+BEGIN {
+       $SIG{__WARN__} = $SIG{__WARN__}->();
+}
 
-__END__
+1;
 
-#line 120
+#line 138
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/inc/Module/Install/Can.pm new/Test-Base-0.54/inc/Module/Install/Can.pm
--- old/Test-Base-0.52/inc/Module/Install/Can.pm        2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/inc/Module/Install/Can.pm        2007-06-28 23:32:47.000000000 +0200
@@ -1,69 +1,82 @@
-#line 1 "inc/Module/Install/Can.pm - /Users/ingy/local/lib/perl5/site_perl/5.8.6/Module/Install/Can.pm"
+#line 1
 package Module::Install::Can;
-use Module::Install::Base; @ISA = qw(Module::Install::Base);
-$VERSION = '0.01';
 
 use strict;
+use Module::Install::Base;
 use Config ();
+### This adds a 5.005 Perl version dependency.
+### This is a bug and will be fixed.
 use File::Spec ();
 use ExtUtils::MakeMaker ();
 
+use vars qw{$VERSION $ISCORE @ISA};
+BEGIN {
+       $VERSION = '0.64';
+       $ISCORE  = 1;
+       @ISA     = qw{Module::Install::Base};
+}
+
 # check if we can load some module
+### Upgrade this to not have to load the module if possible
 sub can_use {
-    my ($self, $mod, $ver) = @_;
-    $mod =~ s{::|\\}{/}g;
-    $mod .= ".pm" unless $mod =~ /\.pm$/i;
-
-    my $pkg = $mod;
-    $pkg =~ s{/}{::}g;
-    $pkg =~ s{\.pm$}{}i;
+       my ($self, $mod, $ver) = @_;
+       $mod =~ s{::|\\}{/}g;
+       $mod .= '.pm' unless $mod =~ /\.pm$/i;
+
+       my $pkg = $mod;
+       $pkg =~ s{/}{::}g;
+       $pkg =~ s{\.pm$}{}i;
 
-    local $@;
-    eval { require $mod; $pkg->VERSION($ver || 0); 1 };
+       local $@;
+       eval { require $mod; $pkg->VERSION($ver || 0); 1 };
 }
 
 # check if we can run some command
 sub can_run {
-    my ($self, $cmd) = @_;
+       my ($self, $cmd) = @_;
 
-    my $_cmd = $cmd;
-    return $_cmd if (-x $_cmd or $_cmd = MM->maybe_command($_cmd));
+       my $_cmd = $cmd;
+       return $_cmd if (-x $_cmd or $_cmd = MM->maybe_command($_cmd));
 
-    for my $dir ((split /$Config::Config{path_sep}/, $ENV{PATH}), '.') {
-        my $abs = File::Spec->catfile($dir, $_[1]);
-        return $abs if (-x $abs or $abs = MM->maybe_command($abs));
-    }
+       for my $dir ((split /$Config::Config{path_sep}/, $ENV{PATH}), '.') {
+               my $abs = File::Spec->catfile($dir, $_[1]);
+               return $abs if (-x $abs or $abs = MM->maybe_command($abs));
+       }
 
-    return;
+       return;
 }
 
+# can we locate a (the) C compiler
 sub can_cc {
-    my $self = shift;
-    my @chunks = split(/ /, $Config::Config{cc}) or return;
+       my $self   = shift;
+       my @chunks = split(/ /, $Config::Config{cc}) or return;
 
-    # $Config{cc} may contain args; try to find out the program part
-    while (@chunks) {
-        return $self->can_run("@chunks") || (pop(@chunks), next);
-    }
+       # $Config{cc} may contain args; try to find out the program part
+       while (@chunks) {
+               return $self->can_run("@chunks") || (pop(@chunks), next);
+       }
 
-    return;
+       return;
 }
 
 # Fix Cygwin bug on maybe_command();
-if ($^O eq 'cygwin') {
-    require ExtUtils::MM_Cygwin;
-    require ExtUtils::MM_Win32;
-    if (!defined(&ExtUtils::MM_Cygwin::maybe_command)) {
-        *ExtUtils::MM_Cygwin::maybe_command = sub {
-            my ($self, $file) = @_;
-            if ($file =~ m{^/cygdrive/}i and ExtUtils::MM_Win32->can('maybe_command')) {
-                ExtUtils::MM_Win32->maybe_command($file);
-            }
-            else {
-                ExtUtils::MM_Unix->maybe_command($file);
-            }
-        }
-    }
+if ( $^O eq 'cygwin' ) {
+       require ExtUtils::MM_Cygwin;
+       require ExtUtils::MM_Win32;
+       if ( ! defined(&ExtUtils::MM_Cygwin::maybe_command) ) {
+               *ExtUtils::MM_Cygwin::maybe_command = sub {
+                       my ($self, $file) = @_;
+                       if ($file =~ m{^/cygdrive/}i and ExtUtils::MM_Win32->can('maybe_command')) {
+                               ExtUtils::MM_Win32->maybe_command($file);
+                       } else {
+                               ExtUtils::MM_Unix->maybe_command($file);
+                       }
+               }
+       }
 }
 
 1;
+
+__END__
+
+#line 157
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/inc/Module/Install/Fetch.pm new/Test-Base-0.54/inc/Module/Install/Fetch.pm
--- old/Test-Base-0.52/inc/Module/Install/Fetch.pm      2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/inc/Module/Install/Fetch.pm      2007-06-28 23:32:47.000000000 +0200
@@ -1,15 +1,22 @@
-#line 1 "inc/Module/Install/Fetch.pm - /Users/ingy/local/lib/perl5/site_perl/5.8.6/Module/Install/Fetch.pm"
+#line 1
 package Module::Install::Fetch;
-use Module::Install::Base; @ISA = qw(Module::Install::Base);
 
-$VERSION = '0.01';
+use strict;
+use Module::Install::Base;
+
+use vars qw{$VERSION $ISCORE @ISA};
+BEGIN {
+       $VERSION = '0.64';
+       $ISCORE  = 1;
+       @ISA     = qw{Module::Install::Base};
+}
 
 sub get_file {
     my ($self, %args) = @_;
     my ($scheme, $host, $path, $file) = 
         $args{url} =~ m|^(\w+)://([^/]+)(.+)/(.+)| or return;
 
-    if ($scheme eq 'http' and !eval { require LWP::Simple; 1 }) {
+    if ( $scheme eq 'http' and ! eval { require LWP::Simple; 1 } ) {
         $args{url} = $args{ftp_url}
             or (warn("LWP support unavailable!\n"), return);
         ($scheme, $host, $path, $file) = 
@@ -53,14 +60,14 @@
             chdir $dir; return;
         }
 
-        my @dialog = split(/\n/, << ".");
+        my @dialog = split(/\n/, <<"END_FTP");
 open $host
 user anonymous anonymous\@example.com
 cd $path
 binary
 get $file $file
 quit
-.
+END_FTP
         foreach (@dialog) { $fh->print("$_\n") }
         $fh->close;
     } }
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/inc/Module/Install/Makefile.pm new/Test-Base-0.54/inc/Module/Install/Makefile.pm
--- old/Test-Base-0.52/inc/Module/Install/Makefile.pm   2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/inc/Module/Install/Makefile.pm   2007-06-28 23:32:47.000000000 +0200
@@ -1,19 +1,37 @@
-#line 1 "inc/Module/Install/Makefile.pm - /Users/ingy/local/lib/perl5/site_perl/5.8.6/Module/Install/Makefile.pm"
+#line 1
 package Module::Install::Makefile;
-use Module::Install::Base; @ISA = qw(Module::Install::Base);
-
-$VERSION = '0.01';
 
 use strict 'vars';
-use vars '$VERSION';
-
+use Module::Install::Base;
 use ExtUtils::MakeMaker ();
 
+use vars qw{$VERSION $ISCORE @ISA};
+BEGIN {
+       $VERSION = '0.64';
+       $ISCORE  = 1;
+       @ISA     = qw{Module::Install::Base};
+}
+
 sub Makefile { $_[0] }
 
-sub prompt { 
+my %seen = ();
+
+sub prompt {
     shift;
-    goto &ExtUtils::MakeMaker::prompt;
+
+    # Infinite loop protection
+    my @c = caller();
+    if ( ++$seen{"$c[1]|$c[2]|$_[0]"} > 3 ) {
+        die "Caught an potential prompt infinite loop ($c[1]|$c[2]|$_[0])";
+    }
+
+    # In automated testing, always use defaults
+    if ( $ENV{AUTOMATED_TESTING} and ! $ENV{PERL_MM_USE_DEFAULT} ) {
+        local $ENV{PERL_MM_USE_DEFAULT} = 1;
+        goto &ExtUtils::MakeMaker::prompt;
+    } else {
+        goto &ExtUtils::MakeMaker::prompt;
+    }
 }
 
 sub makemaker_args {
@@ -23,8 +41,19 @@
     $args;
 }
 
-sub build_subdirs {
+# For mm args that take multiple space-seperated args,
+# append an argument to the current list.
+sub makemaker_append {
     my $self = shift;
+    my $name = shift;
+    my $args = $self->makemaker_args;
+    $args->{name} = defined $args->{$name}
+       ? join( ' ', $args->{name}, @_ )
+       : join( ' ', @_ );
+}
+
+sub build_subdirs {
+    my $self    = shift;
     my $subdirs = $self->makemaker_args->{DIR} ||= [];
     for my $subdir (@_) {
         push @$subdirs, $subdir;
@@ -32,17 +61,26 @@
 }
 
 sub clean_files {
-    my $self = shift;
+    my $self  = shift;
     my $clean = $self->makemaker_args->{clean} ||= {};
     %$clean = (
         %$clean, 
-        FILES => join(" ", grep length, $clean->{FILES}, @_),
+        FILES => join(' ', grep length, $clean->{FILES}, @_),
+    );
+}
+
+sub realclean_files {
+    my $self  = shift;
+    my $realclean = $self->makemaker_args->{realclean} ||= {};
+    %$realclean = (
+        %$realclean, 
+        FILES => join(' ', grep length, $realclean->{FILES}, @_),
     );
 }
 
 sub libs {
     my $self = shift;
-    my $libs = ref $_[0] ? shift : [shift];
+    my $libs = ref $_[0] ? shift : [ shift ];
     $self->makemaker_args( LIBS => $libs );
 }
 
@@ -56,25 +94,26 @@
     die "&Makefile->write() takes no arguments\n" if @_;
 
     my $args = $self->makemaker_args;
-
     $args->{DISTNAME} = $self->name;
-    $args->{NAME} = $self->module_name || $self->name || $self->determine_NAME($args);
-    $args->{VERSION} = $self->version || $self->determine_VERSION($args);
-    $args->{NAME} =~ s/-/::/g;
-
-    $args->{test} = {TESTS => $self->tests} if $self->tests;
-
+    $args->{NAME}     = $self->module_name || $self->name || $self->determine_NAME($args);
+    $args->{VERSION}  = $self->version || $self->determine_VERSION($args);
+    $args->{NAME}     =~ s/-/::/g;
+    if ( $self->tests ) {
+        $args->{test} = { TESTS => $self->tests };
+    }
     if ($] >= 5.005) {
         $args->{ABSTRACT} = $self->abstract;
-        $args->{AUTHOR} = $self->author;
+        $args->{AUTHOR}   = $self->author;
     }
     if ( eval($ExtUtils::MakeMaker::VERSION) >= 6.10 ) {
         $args->{NO_META} = 1;
     }
-    if ( eval($ExtUtils::MakeMaker::VERSION) > 6.17 ) {
-        $args->{SIGN} = 1 if $self->sign;
+    if ( eval($ExtUtils::MakeMaker::VERSION) > 6.17 and $self->sign ) {
+        $args->{SIGN} = 1;
+    }
+    unless ( $self->is_admin ) {
+        delete $args->{SIGN};
     }
-    delete $args->{SIGN} unless $self->is_admin;
 
     # merge both kinds of requires into prereq_pm
     my $prereq = ($args->{PREREQ_PM} ||= {});
@@ -91,49 +130,62 @@
         }
     }
 
-    if (my $perl_version = $self->perl_version) {
+    if ( my $perl_version = $self->perl_version ) {
         eval "use $perl_version; 1"
-            or die "ERROR: perl: Version $] is installed, ".
-                   "but we need version >= $perl_version";
+            or die "ERROR: perl: Version $] is installed, "
+                . "but we need version >= $perl_version";
     }
 
-    my %args = map {($_ => $args->{$_})} grep {defined($args->{$_})} keys %$args;
-
+    my %args = map { ( $_ => $args->{$_} ) } grep {defined($args->{$_})} keys %$args;
     if ($self->admin->preop) {
         $args{dist} = $self->admin->preop;
     }
 
-    ExtUtils::MakeMaker::WriteMakefile(%args);
-
-    $self->fix_up_makefile();
+    my $mm = ExtUtils::MakeMaker::WriteMakefile(%args);
+    $self->fix_up_makefile($mm->{FIRST_MAKEFILE} || 'Makefile');
 }
 
 sub fix_up_makefile {
-    my $self = shift;
-    my $top_class = ref($self->_top) || '';
-    my $top_version = $self->_top->VERSION || '';
+    my $self          = shift;
+    my $makefile_name = shift;
+    my $top_class     = ref($self->_top) || '';
+    my $top_version   = $self->_top->VERSION || '';
 
     my $preamble = $self->preamble 
-       ? "# Preamble by $top_class $top_version\n" . $self->preamble
-       : '';
-    my $postamble = "# Postamble by $top_class $top_version\n" . 
-                    ($self->postamble || '');
+        ? "# Preamble by $top_class $top_version\n"
+            . $self->preamble
+        : '';
+    my $postamble = "# Postamble by $top_class $top_version\n"
+        . ($self->postamble || '');
 
     local *MAKEFILE;
-    open MAKEFILE, '< Makefile' or die $!;
+    open MAKEFILE, "< $makefile_name" or die "fix_up_makefile: Couldn't open $makefile_name: $!";
     my $makefile = do { local $/; <MAKEFILE> };
-    close MAKEFILE;
+    close MAKEFILE or die $!;
 
     $makefile =~ s/\b(test_harness\(\$\(TEST_VERBOSE\), )/$1'inc', /;
     $makefile =~ s/( -I\$\(INST_ARCHLIB\))/ -Iinc$1/g;
     $makefile =~ s/( "-I\$\(INST_LIB\)")/ "-Iinc"$1/g;
+    $makefile =~ s/^(FULLPERL = .*)/$1 "-Iinc"/m;
+    $makefile =~ s/^(PERL = .*)/$1 "-Iinc"/m;
 
-    $makefile =~ s/^(FULLPERL = .*)/$1 -Iinc/m;
-    $makefile =~ s/^(PERL = .*)/$1 -Iinc/m;
+    # Module::Install will never be used to build the Core Perl
+    # Sometimes PERL_LIB and PERL_ARCHLIB get written anyway, which breaks
+    # PREFIX/PERL5LIB, and thus, install_share. Blank them if they exist
+    $makefile =~ s/^PERL_LIB = .+/PERL_LIB =/m;
+    #$makefile =~ s/^PERL_ARCHLIB = .+/PERL_ARCHLIB =/m;
+
+    # Perl 5.005 mentions PERL_LIB explicitly, so we have to remove that as well.
+    $makefile =~ s/("?)-I\$\(PERL_LIB\)\1//g;
+
+    # XXX - This is currently unused; not sure if it breaks other MM-users
+    # $makefile =~ s/^pm_to_blib\s+:\s+/pm_to_blib :: /mg;
+
+    open  MAKEFILE, "> $makefile_name" or die "fix_up_makefile: Couldn't open $makefile_name: $!";
+    print MAKEFILE  "$preamble$makefile$postamble" or die $!;
+    close MAKEFILE  or die $!;
 
-    open MAKEFILE, '> Makefile' or die $!;
-    print MAKEFILE "$preamble$makefile$postamble";
-    close MAKEFILE;
+    1;
 }
 
 sub preamble {
@@ -144,7 +196,6 @@
 
 sub postamble {
     my ($self, $text) = @_;
-
     $self->{postamble} ||= $self->admin->postamble;
     $self->{postamble} .= $text if defined $text;
     $self->{postamble}
@@ -154,4 +205,4 @@
 
 __END__
 
-#line 286
+#line 334
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/inc/Module/Install/Metadata.pm new/Test-Base-0.54/inc/Module/Install/Metadata.pm
--- old/Test-Base-0.52/inc/Module/Install/Metadata.pm   2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/inc/Module/Install/Metadata.pm   2007-06-28 23:32:47.000000000 +0200
@@ -1,13 +1,14 @@
-#line 1 "inc/Module/Install/Metadata.pm - /Users/ingy/local/lib/perl5/site_perl/5.8.6/Module/Install/Metadata.pm"
+#line 1
 package Module::Install::Metadata;
 
 use strict 'vars';
 use Module::Install::Base;
 
-use vars qw($VERSION @ISA);
+use vars qw{$VERSION $ISCORE @ISA};
 BEGIN {
-    $VERSION = '0.06';
-    @ISA     = 'Module::Install::Base';
+       $VERSION = '0.64';
+       $ISCORE  = 1;
+       @ISA     = qw{Module::Install::Base};
 }
 
 my @scalar_keys = qw{
@@ -62,6 +63,16 @@
     return $self;
 }
 
+sub dynamic_config {
+       my $self = shift;
+       unless ( @_ ) {
+               warn "You MUST provide an explicit true/false value to dynamic_config, skipping\n";
+               return $self;
+       }
+       $self->{'values'}{'dynamic_config'} = $_[0] ? 1 : 0;
+       return $self;
+}
+
 sub all_from {
     my ( $self, $file ) = @_;
 
@@ -112,9 +123,9 @@
 
     require Module::Build;
     my $build = Module::Build->new(
-        dist_name    => $self->{name},
-        dist_version => $self->{version},
-        license      => $self->{license},
+        dist_name    => $self->name,
+        dist_version => $self->version,
+        license      => $self->license,
     );
     $self->provides(%{ $build->find_dist_packages || {} });
 }
@@ -130,8 +141,7 @@
         # The user used ->feature like ->features by passing in the second
         # argument as a reference.  Accomodate for that.
         $mods = $_[0];
-    }
-    else {
+    } else {
         $mods = \@_;
     }
 
@@ -154,7 +164,9 @@
     while ( my ( $name, $mods ) = splice( @_, 0, 2 ) ) {
         $self->feature( $name, @$mods );
     }
-    return @{ $self->{values}{features} };
+    return $self->{values}->{features}
+       ? @{ $self->{values}->{features} }
+       : ();
 }
 
 sub no_index {
@@ -226,11 +238,13 @@
         ^
         use \s*
         v?
-        ([\d\.]+)
+        ([\d_\.]+)
         \s* ;
     /ixms
       )
     {
+        my $v = $1;
+        $v =~ s{_}{}g;
         $self->perl_version($1);
     }
     else {
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/inc/Module/Install/Win32.pm new/Test-Base-0.54/inc/Module/Install/Win32.pm
--- old/Test-Base-0.52/inc/Module/Install/Win32.pm      2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/inc/Module/Install/Win32.pm      2007-06-28 23:32:47.000000000 +0200
@@ -1,40 +1,45 @@
-#line 1 "inc/Module/Install/Win32.pm - /Users/ingy/local/lib/perl5/site_perl/5.8.6/Module/Install/Win32.pm"
+#line 1
 package Module::Install::Win32;
-use Module::Install::Base; @ISA = qw(Module::Install::Base);
-
-$VERSION = '0.02';
 
 use strict;
+use Module::Install::Base;
+
+use vars qw{$VERSION $ISCORE @ISA};
+BEGIN {
+       $VERSION = '0.64';
+       $ISCORE  = 1;
+       @ISA     = qw{Module::Install::Base};
+}
 
 # determine if the user needs nmake, and download it if needed
 sub check_nmake {
-    my $self = shift;
-    $self->load('can_run');
-    $self->load('get_file');
-
-    require Config;
-    return unless (
-        $Config::Config{make}                   and
-        $Config::Config{make} =~ /^nmake\b/i    and
-        $^O eq 'MSWin32'                        and
-        !$self->can_run('nmake')
-    );
-
-    print "The required 'nmake' executable not found, fetching it...\n";
-
-    require File::Basename;
-    my $rv = $self->get_file(
-        url         => 'http://download.microsoft.com/download/vc15/Patch/1.52/W95/EN-US/Nmake15.exe',
-        ftp_url     => 'ftp://ftp.microsoft.com/Softlib/MSLFILES/Nmake15.exe',
-        local_dir   => File::Basename::dirname($^X),
-        size        => 51928,
-        run         => 'Nmake15.exe /o > nul',
-        check_for   => 'Nmake.exe',
-        remove      => 1,
-    );
+       my $self = shift;
+       $self->load('can_run');
+       $self->load('get_file');
+       
+       require Config;
+       return unless (
+               $^O eq 'MSWin32'                     and
+               $Config::Config{make}                and
+               $Config::Config{make} =~ /^nmake\b/i and
+               ! $self->can_run('nmake')
+       );
+
+       print "The required 'nmake' executable not found, fetching it...\n";
+
+       require File::Basename;
+       my $rv = $self->get_file(
+               url       => 'http://download.microsoft.com/download/vc15/Patch/1.52/W95/EN-US/Nmake15.exe',
+               ftp_url   => 'ftp://ftp.microsoft.com/Softlib/MSLFILES/Nmake15.exe',
+               local_dir => File::Basename::dirname($^X),
+               size      => 51928,
+               run       => 'Nmake15.exe /o > nul',
+               check_for => 'Nmake.exe',
+               remove    => 1,
+       );
 
-    if (!$rv) {
-        die << '.';
+       if (!$rv) {
+        die <<'END_MESSAGE';
 
 -------------------------------------------------------------------------------
 
@@ -53,11 +58,8 @@
 You may then resume the installation process described in README.
 
 -------------------------------------------------------------------------------
-.
-    }
+END_MESSAGE
+       }
 }
 
 1;
-
-__END__
-
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/inc/Module/Install/WriteAll.pm new/Test-Base-0.54/inc/Module/Install/WriteAll.pm
--- old/Test-Base-0.52/inc/Module/Install/WriteAll.pm   2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/inc/Module/Install/WriteAll.pm   2007-06-28 23:32:47.000000000 +0200
@@ -1,6 +1,15 @@
-#line 1 "inc/Module/Install/WriteAll.pm - /Users/ingy/local/lib/perl5/site_perl/5.8.6/Module/Install/WriteAll.pm"
+#line 1
 package Module::Install::WriteAll;
-use Module::Install::Base; @ISA = qw(Module::Install::Base);
+
+use strict;
+use Module::Install::Base;
+
+use vars qw{$VERSION $ISCORE @ISA};
+BEGIN {
+       $VERSION = '0.64';
+       $ISCORE  = 1;
+       @ISA     = qw{Module::Install::Base};
+}
 
 sub WriteAll {
     my $self = shift;
@@ -12,22 +21,20 @@
         @_
     );
 
-    $self->sign(1) if $args{sign};
-    $self->Meta->write if $args{meta};
+    $self->sign(1)                if $args{sign};
+    $self->Meta->write            if $args{meta};
     $self->admin->WriteAll(%args) if $self->is_admin;
 
-    if ($0 =~ /Build.PL$/i) {
+    if ( $0 =~ /Build.PL$/i ) {
         $self->Build->write;
-    }
-    else {
+    } else {
         $self->check_nmake if $args{check_nmake};
-        $self->makemaker_args( PL_FILES => {} )
-            unless $self->makemaker_args->{'PL_FILES'};
-
+        unless ( $self->makemaker_args->{'PL_FILES'} ) {
+               $self->makemaker_args( PL_FILES => {} );
+        }
         if ($args{inline}) {
             $self->Inline->write;
-        }
-        else {
+        } else {
             $self->Makefile->write;
         }
     }
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/inc/Module/Install.pm new/Test-Base-0.54/inc/Module/Install.pm
--- old/Test-Base-0.52/inc/Module/Install.pm    2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/inc/Module/Install.pm    2007-06-28 23:32:47.000000000 +0200
@@ -1,16 +1,44 @@
-#line 1 "/Users/ingy/src/ingy/Test-Base/inc/Module/Install.pm - /Users/ingy/local/lib/perl5/site_perl/5.8.6/Module/Install.pm"
+#line 1
 package Module::Install;
 
+# For any maintainers:
+# The load order for Module::Install is a bit magic.
+# It goes something like this...
+#
+# IF ( host has Module::Install installed, creating author mode ) {
+#     1. Makefile.PL calls "use inc::Module::Install"
+#     2. $INC{inc/Module/Install.pm} set to installed version of inc::Module::Install
+#     3. The installed version of inc::Module::Install loads
+#     4. inc::Module::Install calls "require Module::Install"
+#     5. The ./inc/ version of Module::Install loads
+# } ELSE {
+#     1. Makefile.PL calls "use inc::Module::Install"
+#     2. $INC{inc/Module/Install.pm} set to ./inc/ version of Module::Install
+#     3. The ./inc/ version of Module::Install loads
+# }
+
 use 5.004;
 use strict 'vars';
+
 use vars qw{$VERSION};
 BEGIN {
-    # Don't forget to update Module::Install::Admin too!
-    $VERSION = '0.54';
-}
-
-# inc::Module::Install must be loaded first
-unless ( $INC{join('/', inc => split(/::/, __PACKAGE__)).'.pm'} ) {
+    # All Module::Install core packages now require synchronised versions.
+    # This will be used to ensure we don't accidentally load old or
+    # different versions of modules.
+    # This is not enforced yet, but will be some time in the next few
+    # releases once we can make sure it won't clash with custom
+    # Module::Install extensions.
+    $VERSION = '0.64';
+}
+
+# Whether or not inc::Module::Install is actually loaded, the
+# $INC{inc/Module/Install.pm} is what will still get set as long as
+# the caller loaded module this in the documented manner.
+# If not set, the caller may NOT have loaded the bundled version, and thus
+# they may not have a MI version that works with the Makefile.PL. This would
+# result in false errors or unexpected behaviour. And we don't want that.
+my $file = join( '/', 'inc', split /::/, __PACKAGE__ ) . '.pm';
+unless ( $INC{$file} ) {
     die <<"END_DIE";
 Please invoke ${\__PACKAGE__} with:
 
@@ -23,27 +51,42 @@
 END_DIE
 }
 
+# If the script that is loading Module::Install is from the future,
+# then make will detect this and cause it to re-run over and over
+# again. This is bad. Rather than taking action to touch it (which
+# is unreliable on some platforms and requires write permissions)
+# for now we should catch this and refuse to run.
+if ( -f $0 and (stat($0))[9] > time ) {
+       die << "END_DIE";
+Your installer $0 has a modification time in the future.
+
+This is known to create infinite loops in make.
+
+Please correct this, then run $0 again.
+
+END_DIE
+}
+
 use Cwd        ();
-use FindBin;
 use File::Find ();
 use File::Path ();
+use FindBin;
 
 *inc::Module::Install::VERSION = *VERSION;
-@inc::Module::Install::ISA     = 'Module::Install';
+@inc::Module::Install::ISA     = __PACKAGE__;
 
 sub autoload {
-    my $self   = shift;
-    my $caller = $self->_caller;
-    my $cwd    = Cwd::cwd();
-    my $sym    = "$caller\::AUTOLOAD";
-
+    my $self = shift;
+    my $who  = $self->_caller;
+    my $cwd  = Cwd::cwd();
+    my $sym  = "${who}::AUTOLOAD";
     $sym->{$cwd} = sub {
         my $pwd = Cwd::cwd();
         if ( my $code = $sym->{$pwd} ) {
             # delegate back to parent dirs
             goto &$code unless $cwd eq $pwd;
         }
-        $$sym =~ /([^:]+)$/ or die "Cannot autoload $caller - $sym";
+        $$sym =~ /([^:]+)$/ or die "Cannot autoload $who - $sym";
         unshift @_, ($self, $1);
         goto &{$self->can('call')} unless uc($1) eq $1;
     };
@@ -52,18 +95,18 @@
 sub import {
     my $class = shift;
     my $self  = $class->new(@_);
+    my $who   = $self->_caller;
 
     unless ( -f $self->{file} ) {
         require "$self->{path}/$self->{dispatch}.pm";
         File::Path::mkpath("$self->{prefix}/$self->{author}");
-        $self->{admin} = 
-          "$self->{name}::$self->{dispatch}"->new(_top => $self);
+        $self->{admin} = "$self->{name}::$self->{dispatch}"->new( _top => $self );
         $self->{admin}->init;
         @_ = ($class, _self => $self);
         goto &{"$self->{name}::import"};
     }
 
-    *{$self->_caller . "::AUTOLOAD"} = $self->autoload;
+    *{"${who}::AUTOLOAD"} = $self->autoload;
     $self->preload;
 
     # Unregister loader and worker packages so subdirs can use them again
@@ -74,11 +117,11 @@
 sub preload {
     my ($self) = @_;
 
-        unless ( $self->{extentions} ) {
-                $self->load_extensions(
-                        "$self->{prefix}/$self->{path}", $self
-                        );
-        }
+    unless ( $self->{extensions} ) {
+        $self->load_extensions(
+            "$self->{prefix}/$self->{path}", $self
+        );
+    }
 
     my @exts = @{$self->{extensions}};
     unless ( @exts ) {
@@ -86,21 +129,21 @@
         @exts = $admin->load_all_extensions;
     }
 
-    my %seen_method;
+    my %seen;
     foreach my $obj ( @exts ) {
         while (my ($method, $glob) = each %{ref($obj) . '::'}) {
-            next unless defined *{$glob}{CODE};
+            next unless $obj->can($method);
             next if $method =~ /^_/;
             next if $method eq uc($method);
-            $seen_method{$method}++;
+            $seen{$method}++;
         }
     }
 
-    my $caller = $self->_caller;
-    foreach my $name (sort keys %seen_method) {
-        *{"${caller}::$name"} = sub {
-            ${"${caller}::AUTOLOAD"} = "${caller}::$name";
-            goto &{"${caller}::AUTOLOAD"};
+    my $who = $self->_caller;
+    foreach my $name ( sort keys %seen ) {
+        *{"${who}::$name"} = sub {
+            ${"${who}::AUTOLOAD"} = "${who}::$name";
+            goto &{"${who}::AUTOLOAD"};
         };
     }
 }
@@ -118,30 +161,26 @@
 
     $args{dispatch} ||= 'Admin';
     $args{prefix}   ||= 'inc';
-    $args{author}   ||= '.author';
+    $args{author}   ||= ($^O eq 'VMS' ? '_author' : '.author');
     $args{bundle}   ||= 'inc/BUNDLES';
     $args{base}     ||= $base_path;
-
     $class =~ s/^\Q$args{prefix}\E:://;
     $args{name}     ||= $class;
     $args{version}  ||= $class->VERSION;
-
-    unless ($args{path}) {
+    unless ( $args{path} ) {
         $args{path}  = $args{name};
         $args{path}  =~ s!::!/!g;
     }
     $args{file}     ||= "$args{base}/$args{prefix}/$args{path}.pm";
 
-    bless(\%args, $class);
+    bless( \%args, $class );
 }
 
 sub call {
-    my $self   = shift;
-    my $method = shift;
-    my $obj    = $self->load($method) or return;
-
-    unshift @_, $obj;
-    goto &{$obj->can($method)};
+       my ($self, $method) = @_;
+       my $obj = $self->load($method) or return;
+        splice(@_, 0, 2, $obj);
+       goto &{$obj->can($method)};
 }
 
 sub load {
@@ -167,13 +206,13 @@
 }
 
 sub load_extensions {
-    my ($self, $path, $top_obj) = @_;
+    my ($self, $path, $top) = @_;
 
-    unshift @INC, $self->{prefix}
-        unless grep { $_ eq $self->{prefix} } @INC;
+    unless ( grep { lc $_ eq lc $self->{prefix} } @INC ) {
+        unshift @INC, $self->{prefix};
+    }
 
-    local @INC = ($path, @INC);
-    foreach my $rv ($self->find_extensions($path)) {
+    foreach my $rv ( $self->find_extensions($path) ) {
         my ($file, $pkg) = @{$rv};
         next if $self->{pathnames}{$pkg};
 
@@ -184,7 +223,7 @@
             next;
         }
         $self->{pathnames}{$pkg} = delete $INC{$file};
-        push @{$self->{extensions}}, &{$new}($pkg, _top => $top_obj );
+        push @{$self->{extensions}}, &{$new}($pkg, _top => $top );
     }
 
     $self->{extensions} ||= [];
@@ -197,10 +236,32 @@
     File::Find::find( sub {
         my $file = $File::Find::name;
         return unless $file =~ m!^\Q$path\E/(.+)\.pm\Z!is;
-        return if $1 eq $self->{dispatch};
+        my $subpath = $1;
+        return if lc($subpath) eq lc($self->{dispatch});
+
+        $file = "$self->{path}/$subpath.pm";
+        my $pkg = "$self->{name}::$subpath";
+        $pkg =~ s!/!::!g;
+
+        # If we have a mixed-case package name, assume case has been preserved
+        # correctly.  Otherwise, root through the file to locate the case-preserved
+        # version of the package name.
+        if ( $subpath eq lc($subpath) || $subpath eq uc($subpath) ) {
+            open PKGFILE, "<$subpath.pm" or die "find_extensions: Can't open $subpath.pm: $!";
+            my $in_pod = 0;
+            while ( <PKGFILE> ) {
+                $in_pod = 1 if /^=\w/;
+                $in_pod = 0 if /^=cut/;
+                next if ($in_pod || /^=cut/);  # skip pod text
+                next if /^\s*#/;               # and comments
+                if ( m/^\s*package\s+($pkg)\s*;/i ) {
+                    $pkg = $1;
+                    last;
+                }
+            }
+            close PKGFILE;
+        }
 
-        $file = "$self->{path}/$1.pm";
-        my $pkg = "$self->{name}::$1"; $pkg =~ s!/!::!g;
         push @found, [ $file, $pkg ];
     }, $path ) if -d $path;
 
@@ -208,15 +269,13 @@
 }
 
 sub _caller {
-    my $depth  = 0;
-    my $caller = caller($depth);
-
-    while ($caller eq __PACKAGE__) {
+    my $depth = 0;
+    my $call  = caller($depth);
+    while ( $call eq __PACKAGE__ ) {
         $depth++;
-        $caller = caller($depth);
+        $call = caller($depth);
     }
-
-    $caller;
+    return $call;
 }
 
 1;
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/lib/Test/Base/Filter.pm new/Test-Base-0.54/lib/Test/Base/Filter.pm
--- old/Test-Base-0.52/lib/Test/Base/Filter.pm  2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/lib/Test/Base/Filter.pm  2007-04-08 07:53:01.000000000 +0200
@@ -213,7 +213,8 @@
 
 sub norm {
     $self->assert_scalar(@_);
-    my $text = shift || '';
+    my $text = shift;
+    $text = '' unless defined $text;
     $text =~ s/\015\012/\n/g;
     $text =~ s/\r/\n/g;
     return $text;
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/lib/Test/Base.pm new/Test-Base-0.54/lib/Test/Base.pm
--- old/Test-Base-0.52/lib/Test/Base.pm 2006-06-19 19:44:43.000000000 +0200
+++ new/Test-Base-0.54/lib/Test/Base.pm 2007-06-28 23:24:12.000000000 +0200
@@ -4,7 +4,7 @@
 use 5.006001;
 use Spiffy 0.30 -Base;
 use Spiffy ':XXX';
-our $VERSION = '0.52';
+our $VERSION = '0.54';
 
 my @test_more_exports;
 BEGIN {
@@ -29,7 +29,7 @@
     filters filters_delay filter_arguments
     run run_compare run_is run_is_deeply run_like run_unlike 
     WWW XXX YYY ZZZ
-    tie_output
+    tie_output no_diag_on_only
 
     find_my_self default_object
 
@@ -50,6 +50,7 @@
 field data_delim =>
       -init => '$self->data_delim_default';
 field _filters_delay => 0;
+field _no_diag_on_only => 0;
 
 field block_delim_default => '===';
 field data_delim_default => '---';
@@ -189,6 +190,11 @@
     $self->_filters_delay(defined $_[0] ? shift : 1);
 }
 
+sub no_diag_on_only() {
+    (my ($self), @_) = find_my_self(@_);
+    $self->_no_diag_on_only(defined $_[0] ? shift : 1);
+}
+
 sub delimiters() {
     (my ($self), @_) = find_my_self(@_);
     $self->check_late;
@@ -393,6 +399,8 @@
     for my $hunk (@_) {
         my $block = $self->_make_block($hunk);
         if (exists $block->{ONLY}) {
+            diag "I found ONLY: maybe you're debugging?"
+                unless $self->_no_diag_on_only;
             return [$block];
         }
         next if exists $block->{SKIP};
@@ -576,11 +584,15 @@
             my $function = "main::$filter";
             no strict 'refs';
             if (defined &$function) {
-                $_ = join '', @value;
+                local $_ = join '', @value;
+                my $old = $_;
                 @value = &$function(@value);
                 if (not(@value) or 
                     @value == 1 and $value[0] =~ /\A(\d+|)\z/
                 ) {
+                    if ($value[0] && $_ eq $old) {
+                        Test::Base::diag("Filters returning numbers are supposed to do munging \$_: your filter '$function' apparently doesn't.");
+                    }
                     @value = ($_);
                 }
             }
@@ -1029,6 +1041,12 @@
 useful when you are concentrating on getting a single test to pass. If
 there is more than one block with ONLY, the first one will be chosen.
 
+Because ONLY is very useful for debugging and sometimes you forgot to
+remove the ONLY flag before commiting to the VCS or uploading to CPAN,
+Test::Base by default gives you a diag message saying I<I found ONLY
+... maybe you're debugging?>. If you don't like it, use
+C<no_diag_on_only>.
+
 A block with a LAST section makes that block the last one in the
 specification. All following blocks will be ignored.
 
@@ -1106,7 +1124,7 @@
 in list context, and the first element of the list in scalar context.
 This is usually "the right thing", but be aware.
 
-head2 The Stock Filters
+=head2 The Stock Filters
 
 Test::Base comes with large set of stock filters. They are in the
 C<Test::Base::Filter> module. See L<Test::Base::Filter> for a listing and
@@ -1133,7 +1151,7 @@
     sub Test::Base::Filter::bar {
         my $self = shift;       # The Test::Base::Filter object
         my $data = shift;
-        my $args = $self->arguments;
+        my $args = $self->current_arguments;
         my $current_block_object = $self->block;
         # transform $data in a barish manner
         return $data;
@@ -1145,6 +1163,18 @@
 Normally you'll probably just use the functional interface, although all
 the builtin filters are methods.
 
+Note that filters defined in the C<main> namespace can look like:
+
+  sub filter9 {
+      s/foo/bar/;
+  }
+
+since Test::Base automatically munges the input string into $_
+variable and checks the return value of the function to see if it
+looks like a number. If you must define a filter that returns just a
+single number, do it in a different namespace as a method. These
+filters don't allow the simplistic $_ munging.
+
 =head1 OO
 
 Test::Base has a nice functional interface for simple usage. Under the
@@ -1247,7 +1277,7 @@
 
 You might be thinking that you do not want to use Test::Base in you
 modules, because it adds an installation dependency. Fear not.
-Module::Build takes care of that.
+Module::Install takes care of that.
 
 Just write a Makefile.PL that looks something like this:
 
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/Makefile.PL new/Test-Base-0.54/Makefile.PL
--- old/Test-Base-0.52/Makefile.PL      2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/Makefile.PL      2007-06-28 23:21:54.000000000 +0200
@@ -6,6 +6,7 @@
 requires    perl => '5.6.1';
 requires    Spiffy => '0.30';
 requires    Test::More => '0.62';
+requires    Filter::Util::Call => '0';
 
 clean_files 't/output';
 
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/META.yml new/Test-Base-0.54/META.yml
--- old/Test-Base-0.52/META.yml 2006-06-19 19:46:40.000000000 +0200
+++ new/Test-Base-0.54/META.yml 2007-06-28 23:32:47.000000000 +0200
@@ -1,16 +1,16 @@
-
+abstract: A Data Driven Testing Framework
+author: "Ingy d\xC3\xB6t Net <ingy@xxxxxxxx>"
+distribution_type: module
+generated_by: Module::Install version 0.64
+license: perl
+name: Test-Base
 no_index: 
   directory: 
     - inc
     - t
-generated_by: Module::Install version 0.54
-distribution_type: module
-version: 0.52
-name: Test-Base
-author: "Ingy d\xC3\xB6t Net <ingy@xxxxxxxx>"
-license: perl
 requires: 
+  Filter::Util::Call: 0
+  Spiffy: 0.30
   Test::More: 0.62
   perl: 5.6.1
-  Spiffy: 0.30
-abstract: A Data Driven Testing Framework
+version: 0.54
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/README new/Test-Base-0.54/README
--- old/Test-Base-0.52/README   2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/README   2007-04-08 07:53:01.000000000 +0200
@@ -367,6 +367,11 @@
     useful when you are concentrating on getting a single test to pass. If
     there is more than one block with ONLY, the first one will be chosen.
 
+    Because ONLY is very useful for debugging and sometimes you forgot to
+    remove the ONLY flag before commiting to the VCS or uploading to CPAN,
+    Test::Base by default gives you a diag message saying *I found ONLY ...
+    maybe you're debugging?*. If you don't like it, use "no_diag_on_only".
+
     A block with a LAST section makes that block the last one in the
     specification. All following blocks will be ignored.
 
@@ -439,8 +444,7 @@
     in list context, and the first element of the list in scalar context.
     This is usually "the right thing", but be aware.
 
-    head2 The Stock Filters
-
+  The Stock Filters
     Test::Base comes with large set of stock filters. They are in the
     "Test::Base::Filter" module. See Test::Base::Filter for a listing and
     description of these filters.
@@ -465,7 +469,7 @@
         sub Test::Base::Filter::bar {
             my $self = shift;       # The Test::Base::Filter object
             my $data = shift;
-            my $args = $self->arguments;
+            my $args = $self->current_arguments;
             my $current_block_object = $self->block;
             # transform $data in a barish manner
             return $data;
@@ -477,6 +481,18 @@
     Normally you'll probably just use the functional interface, although all
     the builtin filters are methods.
 
+    Note that filters defined in the "main" namespace can look like:
+
+      sub filter9 {
+          s/foo/bar/;
+      }
+
+    since Test::Base automatically munges the input string into $_ variable
+    and checks the return value of the function to see if it looks like a
+    number. If you must define a filter that returns just a single number,
+    do it in a different namespace as a method. These filters don't allow
+    the simplistic $_ munging.
+
 OO
     Test::Base has a nice functional interface for simple usage. Under the
     hood everything is object oriented. A default Test::Base object is
@@ -568,7 +584,7 @@
 DISTRIBUTION SUPPORT
     You might be thinking that you do not want to use Test::Base in you
     modules, because it adds an installation dependency. Fear not.
-    Module::Build takes care of that.
+    Module::Install takes care of that.
 
     Just write a Makefile.PL that looks something like this:
 
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/t/diff_is.t new/Test-Base-0.54/t/diff_is.t
--- old/Test-Base-0.52/t/diff_is.t      2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/t/diff_is.t      2007-04-08 07:53:01.000000000 +0200
@@ -18,6 +18,7 @@
     sub smooth_output { 
         s/test-blocks-\d+/test-blocks-321/;
         s/at line \d+\)/at line 000)/;
+        s/in (.*) at line (\d+)/at $1 line $2/; # for Test::Simple 0.65
         s/^\n//gm;
     }
 }
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/t/filter_arguments.t new/Test-Base-0.54/t/filter_arguments.t
--- old/Test-Base-0.52/t/filter_arguments.t     2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/t/filter_arguments.t     2007-04-08 07:53:01.000000000 +0200
@@ -4,11 +4,13 @@
 
 sub foo {
     is filter_arguments, '123,456';
+    return;
 }
 
 sub bar {
     is filter_arguments, '---holy-crow+++';
     is $_, "one\n  two\n";
+    return;
 }
 
 __DATA__
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/t/lazy-filters.t new/Test-Base-0.54/t/lazy-filters.t
--- old/Test-Base-0.52/t/lazy-filters.t 2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/t/lazy-filters.t 2007-04-08 07:53:01.000000000 +0200
@@ -1,5 +1,5 @@
 use Test::Base tests => 2;
-
+no_diag_on_only;
 sub shouldnt_be_run {
     fail "shouldnt_be_run was run";
 }
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/t/main_filters.t new/Test-Base-0.54/t/main_filters.t
--- old/Test-Base-0.52/t/main_filters.t 2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/t/main_filters.t 2007-06-28 23:32:39.000000000 +0200
@@ -1,9 +1,10 @@
-use Test::Base tests => 5;
+use Test::Base tests => 6;
 
 is next_block->xxx, "I lmike mike\n";
 is next_block->xxx, "I like mikey";
 is next_block->xxx, "123\n";
 is next_block->xxx, "I like MIKEY";
+is next_block->xxx, "I like ike\n";
 
 run_is xxx => 'yyy';
 
@@ -24,7 +25,11 @@
 sub mike4 {
     $_ = 'I like MIKEY';
     return;
-};
+}
+
+sub mike5 {
+    return 200;
+}
 
 sub yyy { s/x/y/g }
 
@@ -46,6 +51,10 @@
 I like ike
 
 ===
+--- xxx mike5
+I like ike
+
+===
 --- xxx lines yyy
 xxx xxx
   xxx xxx
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/t/only.t new/Test-Base-0.54/t/only.t
--- old/Test-Base-0.52/t/only.t 2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/t/only.t 2007-04-08 07:53:01.000000000 +0200
@@ -1,5 +1,5 @@
 use Test::Base tests => 3;
-
+no_diag_on_only;
 run { pass };
 
 is scalar(blocks), 1;
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/t/only-with-implicit.t new/Test-Base-0.54/t/only-with-implicit.t
--- old/Test-Base-0.52/t/only-with-implicit.t   2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/t/only-with-implicit.t   2007-04-08 07:53:01.000000000 +0200
@@ -1,5 +1,5 @@
 use Test::Base tests => 1;
-
+no_diag_on_only;
 run_is;
 
 __END__
diff -urN --exclude=CVS --exclude=.cvsignore --exclude=.svn --exclude=.svnignore old/Test-Base-0.52/t/preserve-order.t new/Test-Base-0.54/t/preserve-order.t
--- old/Test-Base-0.52/t/preserve-order.t       2006-04-07 04:25:02.000000000 +0200
+++ new/Test-Base-0.54/t/preserve-order.t       2007-04-08 07:53:01.000000000 +0200
@@ -7,6 +7,7 @@
     my $num = shift;
     chomp $num;
     is $num, ++$count;
+    return;
 }
 
 __END__


++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++



Remember to have fun...

---------------------------------------------------------------------
To unsubscribe, e-mail: opensuse-commit+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: opensuse-commit+help@xxxxxxxxxxxx

< Previous Next >
This Thread
  • No further messages