[buildd-tools-devel] Bug#768091: Bug#768091: schroot: please support alternative compressions for file-based chroots

[Roger Leigh]

On Tue, Nov 04, 2014 at 10:51:12PM +0100, Aurelien Jarno wrote:
> schroot currently supports the gzip and bzip2 compressions for file-based
> chroots. It might be worth to support alternative compressions to let
> the user do a compromise between speed and space, depending for example
> if the chroot is used often or not.
> 
> The patch below adds support for xz, lzop and lz4 compression, which
> should cover most of the standard compression format available today.
> Could you please apply it in one of the next uploads? Thanks.

Thanks for the patch.  It looks fine except for the lz4 case, which
uses GNU tar-specific options and hardcodes the path to lz4 which is
nonportable, and it also lacks an update to the docs in
man/schroot.conf.5.man ("optionally compressed with...").

If you could fix it to use lz4 portably (I'd suggest piping) so it
works with BSD tar, and also fix up the docs, I'll be happy to apply
this.


Thanks,
Roger

-- 
  .''`.  Roger Leigh
 : :' :  Debian GNU/Linux    http://people.debian.org/~rleigh/
 `. `'   schroot and sbuild  http://alioth.debian.org/projects/buildd-tools
   `-    GPG Public Key      F33D 281D 470A B443 6756 147C 07B3 C8BC 4083 E800