Re: [dpkg] 85/130: test: Add an initial functional test suite for dpkg-deb and dpkg-split

[Guillem Jover <guillem@debian.org>]

Hi!

On Sun, 2016-10-30 at 18:15:09 +0100, Sven Joachim wrote:
> On 2016-10-30 11:19 +0100, Sven Joachim wrote:
> > On 2016-10-30 05:46 +0000, Guillem Jover wrote:
> >> +AT_CHECK([
> >> +# Test non-uniform data.tar/control.tar member compression
> >> +ar rc pkg-mixed-comp.deb debian-binary control.tar.xz data.tar.gz
> >> +ar t pkg-mixed-comp.deb
> >> +# XXX: Ideally we would have no need to strip trailing spaces.
> >> +dpkg-deb -I pkg-mixed-comp.deb | sed -e 's/ *$//'
> >> +dpkg-deb -c pkg-mixed-comp.deb
> >> +], [], [debian-binary
> >> +control.tar.xz
> >> +data.tar.gz
> >> + new debian package, version 2.0.
> >> + size 682 bytes: control archive=336 bytes.
> >> +     193 bytes,     7 lines      control
> >> + Package: pkg-deb-format
> >> + Version: 0.0-1
> >> + Section: test
> >> + Priority: extra
> >> + Maintainer: Dpkg Developers <debian-dpkg@lists.debian.org>
> >> + Architecture: all
> >> + Description: test package - deb format support
> >> +drwxr-xr-x root/root         0 1970-01-01 00:00 ./
> >> +-rw-r--r-- root/root         5 1970-01-01 00:00 ./file-templ
> >> +])
> >
> > This test fails for me, because the size of pkg-mixed-comp.deb is 672
> > bytes, not 682.
> 
> Found out the reason, it is caused by having GZIP=-9 set in the
> environment.  Probably the test should care to unset this environment
> variable.

I added code to reset all compressor environment variables that could
affect those processes. Hope I've not missed any!

Thanks,
Guillem