Michael Loeffler <zvpunry@xxxxxxxxxx> wrote:
> In the Archive::Tar manpage they use gunzip and uncompress instead of
> zcat or gzcat, so what do you think about the following patch?
>
> diff --git a/contrib/fast-import/import-tars.perl b/contrib/fast-import/import-tars.perl
> index 990c9e7..5585a8b 100755
> --- a/contrib/fast-import/import-tars.perl
> +++ b/contrib/fast-import/import-tars.perl
> @@ -25,11 +25,14 @@ foreach my $tar_file (@ARGV)
> my $tar_name = $1;
>
> if ($tar_name =~ s/\.(tar\.gz|tgz)$//) {
> - open(I, '-|', 'gzcat', $tar_file) or die "Unable to gzcat $tar_file: $!\n";
> + open(I, '-|', 'gunzip', '-c', $tar_file)
> + or die "Unable to gunzip -c $tar_file: $!\n";
> } elsif ($tar_name =~ s/\.(tar\.bz2|tbz2)$//) {
> - open(I, '-|', 'bzcat', $tar_file) or die "Unable to bzcat $tar_file: $!\n";
> + open(I, '-|', 'bunzip2', '-c', $tar_file)
> + or die "Unable to bunzip2 -c $tar_file: $!\n";
> } elsif ($tar_name =~ s/\.tar\.Z$//) {
> - open(I, '-|', 'zcat', $tar_file) or die "Unable to zcat $tar_file: $!\n";
> + open(I, '-|', 'uncompress', '-c', $tar_file)
> + or die "Unable to uncompress -c $tar_file: $!\n";
> } elsif ($tar_name =~ s/\.tar$//) {
> open(I, $tar_file) or die "Unable to open $tar_file: $!\n";
> } else {
Yes, that's what I should have done initially. Thanks. How about
the following message and sbo?
commit 908387056949c0fb8153fbb84f4dbeb6695611e6
Author: Michael Loeffler <zvpunry@xxxxxxxxxx>
Date: Wed Feb 14 17:03:12 2007 +0100
Use gunzip -c over gzcat in import-tars example.
Not everyone has gzcat or bzcat installed on their system, but
gunzip -c and bunzip2 -c perform the same task and are available
if the user has installed gzip support or bzip2 support.
Signed-off-by: Michael Loeffler <zvpunry@xxxxxxxxxx>
Signed-off-by: Shawn O. Pearce <spearce@xxxxxxxxxxx>
--
Shawn.
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html