Author: dagolden
Date: Wed Apr  1 04:53:56 2009
New Revision: 12664

Modified:
   Module-Build/trunk/Build.PL
   Module-Build/trunk/lib/Module/Build.pm
   Module-Build/trunk/lib/Module/Build/Platform/Unix.pm

Log:
Archive::Tar is now the default everywhere.  Avoids problems with
unix-like systems with incompatible tars, like AIX and Sun's tar.

"All your tars are belong to Perl."


Modified: Module-Build/trunk/Build.PL
==============================================================================
--- Module-Build/trunk/Build.PL (original)
+++ Module-Build/trunk/Build.PL Wed Apr  1 04:53:56 2009
@@ -27,6 +27,7 @@
   },
   requires    => {
     'perl'                  => '5.006001',
+    'Archive::Tar'          => '1.08',
     'Data::Dumper'          => 0,
     'File::Basename'        => 0,
     'File::Compare'         => 0,
@@ -45,7 +46,6 @@
     'Test::Harness'         => 0,
   },
   recommends => {
-    'Archive::Tar'       => '1.08',
     'ExtUtils::CBuilder' => 0.15,
     'ExtUtils::Install'  => 0.30,
     'ExtUtils::ParseXS'  => 1.02,

Modified: Module-Build/trunk/lib/Module/Build.pm
==============================================================================
--- Module-Build/trunk/lib/Module/Build.pm      (original)
+++ Module-Build/trunk/lib/Module/Build.pm      Wed Apr  1 04:53:56 2009
@@ -293,10 +293,9 @@
 tarball of the files listed in F<MANIFEST> and compress the tarball using
 GZIP compression.
 
-By default, this action will use the external C<tar> and C<gzip>
-executables on Unix-like platforms, and the C<Archive::Tar> module
-elsewhere.  However, you can force it to use whatever executable you
-want by supplying an explicit C<tar> (and optional C<gzip>) parameter:
+By default, this action will use the C<Archive::Tar> module. However, you can
+force it to use binary "tar" and "gzip" executables by supplying an explicit 
+C<tar> (and optional C<gzip>) parameter:
 
   ./Build dist --tar C:\path\to\tar.exe --gzip C:\path\to\zip.exe
 

Modified: Module-Build/trunk/lib/Module/Build/Platform/Unix.pm
==============================================================================
--- Module-Build/trunk/lib/Module/Build/Platform/Unix.pm        (original)
+++ Module-Build/trunk/lib/Module/Build/Platform/Unix.pm        Wed Apr  1 
04:53:56 2009
@@ -9,13 +9,6 @@
 use vars qw(@ISA);
 @ISA = qw(Module::Build::Base);
 
-sub make_tarball {
-  my $self = shift;
-  $self->{args}{tar}  ||= ['tar'];
-  $self->{args}{gzip} ||= ['gzip'];
-  $self->SUPER::make_tarball(@_);
-}
-
 sub is_executable {
   # We consider the owner bit to be authoritative on a file, because
   # -x will always return true if the user is root and *any*

Reply via email to