[svn:Module-Build] r12664 - in Module-Build/trunk: . lib/Module lib/Module/Build/Platform

[email protected]
Newsgroups perl.module.build.checkins
Message-ID <[email protected]>
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*
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.