[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: [PATCH] Allow XZ streaming format in tar.
From:       Denys Vlasenko <vda.linux () googlemail ! com>
Date:       2012-05-24 6:59:53
Message-ID: CAK1hOcMzzcX9d0G6a66rQVKgYQ=D6vTfDXNx+rrms9gPamAgcA () mail ! gmail ! com
[Download RAW message or body]

On Mon, May 21, 2012 at 9:24 PM, Boris Reisig <boris@boris.ca> wrote:
> Here's a patch that will allow tar to uncompress XZ file formats when it
> can't properly detect XZ data formats when the input data comes from the
> standard input.  I have added the (-J) option to force tar to recognize XZ
> file format. Without this patch, it won't work with data streams.

Patch is whitespace-damaged.

>  static const char tar_longopts[] ALIGN1 =
> @@ -924,6 +930,7 @@
>                IF_FEATURE_SEAMLESS_LZMA("a"   )
>                IF_FEATURE_TAR_FROM(     "T:X:")
>                IF_FEATURE_SEAMLESS_GZ(  "z"   )
> +               IF_FEATURE_SEAMLESS_XZ(  "J"   )
>                IF_FEATURE_SEAMLESS_Z(   "Z"   )
>                IF_FEATURE_TAR_NOPRESERVE_TIME("m")
>                , &base_dir // -C dir

You did not add --xz long option, only short one, -J.


> +               if (opt & OPT_XZ)
> +                       USE_FOR_MMU(xformer = unpack_xz_stream;)
> +                       USE_FOR_NOMMU(xformer_prog = "xz";)

Shouldn't this be "unxz"?

-- 
vda
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread]