On Wed, 13 Nov 2019 18:36:43 +0100 Benjamin Drung
<benjamin.dr...@cloud.ionos.com> wrote:
> > > When I developed the patch, I just checked that the tarball was created
> > > and the file size matches, but I didn't check the content.
> >
> > ah indeed. This is of course because mmdebstrap assembles the tarball
> > from two
> > parts and then runs the compressor outside of tar. A correct patch
> > probably
> > look more like this:
> >
> > @@ -161,7 +161,7 @@ sub get_tar_compressor($) {
> > } elsif ($filename =~ /\.lz4$/) {
> > return 'lz4';
> > } elsif ($filename =~ /\.(xz|txz)$/) {
> > - return 'xz';
> > + return ('xz', '--threads=0');
> > } elsif ($filename =~ /\.zst$/) {
> > return 'zstd';
> > }
>
> I have tested this proposed change by dirty patching the two exec
> lines:
>
> exec ($tar_compressor, '--threads=0') or error "[...]";
>
> It works and creates a tarball. The generated tarball is actually working
> (verified by using it).fixed in git: https://gitlab.mister-muffin.de/josch/mmdebstrap/commit/9f2ea61265c36945b1fbbc27fd70099e58df794d
signature.asc
Description: signature
