Debian Bug report logs - #801329
dpkg: scripts/t/mk.t uses system modules to compute reference values

Display info messages

Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Jérémy Bobbio <lunar@debian.org>

To: submit@bugs.debian.org

Subject: dpkg: scripts/t/mk.t uses system modules to compute reference values

Date: Thu, 8 Oct 2015 18:54:59 +0200

[Message part 1 (text/plain, inline)]

Source: dpkg
Version: 1.18.3
Severity: normal
Tags: patch

Hi!

scripts/t/mk.t will shell out to dpkg-architecture.pl and
dpkg-buildpackage.pl. Both scripts are called from the current
development tree. But they will use Perl modules to compute some of
their values. As PERL5LIB is not set, system modules are going to
be used instead of the one from the development tree.

This makes it impossible to have the tests pass when there is a change
in the default dpkg-buildflags values in a new version.

The attached patch sets the PERL5LIB environment variable which fixes
the problem.

-- 
Lunar                                .''`. 
lunar@debian.org                    : :Ⓐ  :  # apt-get install anarchism
                                    `. `'` 
                                      `-   

[0001-Get-reference-values-from-current-modules-in-scripts.patch (text/x-diff, attachment)]

[signature.asc (application/pgp-signature, inline)]

Message #10 received at 801329@bugs.debian.org (full text, mbox, reply):

From: Guillem Jover <guillem@debian.org>

To: Jérémy Bobbio <lunar@debian.org>, 801329@bugs.debian.org

Subject: Re: Bug#801329: dpkg: scripts/t/mk.t uses system modules to compute reference values

Date: Fri, 9 Oct 2015 17:10:06 +0200

[Message part 1 (text/plain, inline)]

Hi!

On Thu, 2015-10-08 at 18:54:59 +0200, Jérémy Bobbio wrote:
> Source: dpkg
> Version: 1.18.3
> Severity: normal
> Tags: patch

> scripts/t/mk.t will shell out to dpkg-architecture.pl and
> dpkg-buildpackage.pl. Both scripts are called from the current
> development tree. But they will use Perl modules to compute some of
> their values. As PERL5LIB is not set, system modules are going to
> be used instead of the one from the development tree.
> 
> This makes it impossible to have the tests pass when there is a change
> in the default dpkg-buildflags values in a new version.
> 
> The attached patch sets the PERL5LIB environment variable which fixes
> the problem.

Ah indeed, this is a regression. I've fixed it globally instead,
please see the attached patch, and let me know if this works for you?

Thanks,
Guillem

[0001-build-Set-PERL5LIB-to-the-local-modules-directory-fo.patch (text/x-diff, attachment)]

Message #13 received at 801329-submitter@bugs.debian.org (full text, mbox, reply):

From: Guillem Jover <guillem@debian.org>

To: 801329-submitter@bugs.debian.org

Subject: Bug#801329 marked as pending

Date: Sun, 18 Oct 2015 03:40:48 +0000

Control: tag 801329 pending

Hi!

Bug #801329 reported by you has been fixed in the Git repository. You can
see the changelog below, and you can check the diff of the fix at:

    https://anonscm.debian.org/cgit/dpkg/dpkg.git/diff/?id=f022170

---
commit f02217095bb294ab1eac879e495ffa23398ff7b6
Author: Guillem Jover <guillem@debian.org>
Date:   Fri Oct 9 16:55:35 2015 +0200

    build: Set PERL5LIB globally for the test suite to the local modules directory
    
    Regression introduced in commit 34b26c7db232b300991df5fb6f52707b381df28f.
    
    Some functional tests are passing through makefile and exec barriers,
    where the lib parameter for TAP::Harness is lost. Reintroduce the
    explicit global setting of PERL5LIB so that it gets inherited for all
    subprocesses.
    
    Closes: #801329
    Reported-by: Jérémy Bobbio <lunar@debian.org>
    Stable-Candidate: 1.17.x

diff --git a/debian/changelog b/debian/changelog
index c7a1c06..52ec913 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -4,6 +4,10 @@ dpkg (1.18.4) UNRELEASED; urgency=low
   * Switch dpkg-scansources and dpkg-scanpackages to use File::Find instead
     of find(1), as the former is more portable with more consistent behavior,
     and always canonicalizes the pathnames. Closes: #800649
+  * Build system:
+    - Set PERL5LIB globally for the test suite to the local modules directory,
+      to avoid using the system modules. Regression introduced in dpkg 1.17.8.
+      Reported by Jérémy Bobbio <lunar@debian.org>. Closes: #801329
   * Documentation:
     - Move description for “target architecture” from the dpkg-architecture(1)
       ‘-A’ option to the TERMS section. Closes: #799046

Message #20 received at 801329-close@bugs.debian.org (full text, mbox, reply):

From: Guillem Jover <guillem@debian.org>

To: 801329-close@bugs.debian.org

Subject: Bug#801329: fixed in dpkg 1.18.4

Date: Fri, 25 Dec 2015 13:34:00 +0000

Source: dpkg
Source-Version: 1.18.4

We believe that the bug you reported is fixed in the latest version of
dpkg, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 801329@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Guillem Jover <guillem@debian.org> (supplier of updated dpkg package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Format: 1.8
Date: Fri, 25 Dec 2015 13:20:26 +0100
Source: dpkg
Binary: libdpkg-dev dpkg dpkg-dev libdpkg-perl dselect
Architecture: source
Version: 1.18.4
Distribution: unstable
Urgency: medium
Maintainer: Dpkg Developers <debian-dpkg@lists.debian.org>
Changed-By: Guillem Jover <guillem@debian.org>
Description:
 dpkg       - Debian package management system
 dpkg-dev   - Debian package development tools
 dselect    - Debian package management front-end
 libdpkg-dev - Debian package management static library
 libdpkg-perl - Dpkg perl modules
Closes: 760248 799046 799432 799875 800513 800649 801156 801329 801958 805872 806315 807156 808912
Changes:
 dpkg (1.18.4) unstable; urgency=medium
 .
   [ Guillem Jover ]
   * Switch dpkg-scansources and dpkg-scanpackages to use File::Find instead
     of find(1), as the former is more portable with more consistent behavior,
     and always canonicalizes the pathnames. Closes: #800649
   * Initialize Config-Version also for packages previously in triggers-pending
     state, otherwise we end up not passing the previously configured version
     to «postinst configure», which might consider this a first install instead
     of an upgrade. Closes: #801156
   * Fix memory leaks in «dpkg --verify» and dpkg infodb format upgrade logic.
   * Merge all update-alternatives action handling into a single if-else-if
     block, to unify the code an allow a future switch into a shared library.
   * Perform any necessary cleanups on normal exit from dpkg-divert --add and
     --remove commands.
   * Make dpkg-architecture warning on non-matching GNU system type compiler
     agnostic.
   * Add ‘.gitreview’ to the default dpkg-source ignore lists.
   * Add support for DPKG_MAINTSCRIPT_DEBUG environment variable to dpkg.
   * Fix dpkg-checkbuilddeps exit code to be 1 instead of a random error value
     on unsatisfied dependencies. Regression introduced in dpkg 1.18.3.
   * Fix an off-by-one write access in dpkg-deb when parsing the old format
     .deb control member size. Thanks to Hanno Böck <hanno@hboeck.de>.
     Fixes CVE-2015-0860.
   * Fix an off-by-one read access in dpkg-deb when parsing ar member names.
     Thanks to Hanno Böck <hanno@hboeck.de>.
   * Add experimental multithreaded xz compression support in libdpkg, which
     requires xz >= 5.2.0.
   * Fix physical file offset comparison in dpkg. Closes: #808912
     Thanks to Yuri Gribov <tetra2005@gmail.com>.
   * Fix usage of dpkg-architecture -s after other action options.
     Reported by Niels Thykier <niels@thykier.net>.
   * Add NIOS2 support to cputable. Thanks to Marek Vasut <marex@denx.de>.
   * On Debian and derivatives enable timeless build flag feature by default.
     Thanks to Paul Wise <pabs@debian.org>. Closes: #805872
   * Perl modules:
     - Add support for Build-Essential field. Closes: #806315
   * Test suite:
     - Improve perl code test coverage.
   * Build system:
     - Set PERL5LIB globally for the test suite to the local modules directory,
       to avoid using the system modules. Regression introduced in dpkg 1.17.8.
       Reported by Jérémy Bobbio <lunar@debian.org>. Closes: #801329
     - Use absolute buildir pathnames in PATH variable for the test suite.
     - Descend into scripts directory when cleaning up code coverage files.
     - Add new configure option --disable-devel-docs to select the kind of docs
       to generate, default for now is development documentation.
     - Try to use AM_GNU_GETTEXT_REQUIRE_VERSION to benefit from the latest
       installed gettext version, while guaranteeing a minimal required version.
   * Packaging:
     - Add missing Build-Depends for restriction formula support.
   * Documentation:
     - Move description for “target architecture” from the dpkg-architecture(1)
       ‘-A’ option to the TERMS section. Closes: #799046
     - Clarify that the md5sum check on «dpkg --verify» is performed on the
       file contents, and failures denote changed content. Closes: #760248
     - Document that dpkg-buildpacakge -nc -S implies -d.
     - Clarify role of Build-Depends in deb-src-control(5).
       Prompted by Johannes Schauer <j.schauer@email.de>.
     - Document supported feature areas.
     - Clarify in dpkg-query(1) when binary:Package gets arch-qualified.
       Closes: #801958
     - Add a subsection separating external from internal environment variables
       in dpkg(1).
 .
   [ Updated programs translations ]
   * Dutch (Frans Spiesschaert). Closes: #800513
   * Japanese (Kenshi Muto). Closes: #799432
   * Turkish (Mert Dirik). Closes: #799875
 .
   [ Updated scripts translations ]
   * German (Helge Kreutzmann).
 .
   [ Updated manpages translations ]
   * German (Helge Kreutzmann, Julian R). Closes: #807156
Checksums-Sha1:
 4f1df693463e7279d4d0362dbb00b6116353a933 2053 dpkg_1.18.4.dsc
 87707de6726d27f2c60fbd95bfaf90f888e2afe8 4377024 dpkg_1.18.4.tar.xz
Checksums-Sha256:
 bcf295adfb7d467220586789f3e7b2de78ded2ea035c9c0ba4e97283d39eeb6f 2053 dpkg_1.18.4.dsc
 fe89243868888ce715bf45861f26264f767d4e4dbd0d6f1a26ce60bbbbf106da 4377024 dpkg_1.18.4.tar.xz
Files:
 32a406117e41c649a0df42289d2a8860 2053 admin required dpkg_1.18.4.dsc
 e95b513c89693f6ec3ab53b6b1c3defd 4377024 admin required dpkg_1.18.4.tar.xz

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQIcBAEBCAAGBQJWfUNzAAoJELlyvz6krlejO5YP/iDDalDQ7TIRBiR9MysEkzcx
M6p9FZ0eSZY0a3sL9UZ8rYJXPmGuZSRCp9ekFucwMY0wEb/qHjy9roDWWDv7ih9b
WPqIPKPv/nuWX7hOqWXAyrkmyeTKS8nhRrf2jBSKSVhm/uGpKiB9Q2i1+7SYSIDf
OPzO+RmDeQuFRiTrAeUrTxz+qyEfbnlWer5ZyEGDNssV6TQQHSwEkYxyhc9jsoE6
xz5AeF2DhV04rM9m9qk5dBPAN8YWTJK7xqE1MC9lQ/UZp35jhryvChbnWbWAC49M
k8xe5AfrDDWclBfSbCeyoISuTNq9m2KLHbNAAhCqPWZFWgT/7rY9QnRxFXNsjPvr
Vr2kfpBy0oR7JEFS4YZvUkVNgDUopgCEEwi5b4wdm6X/NeUHfInfq9DLvyOKeDGa
LY0hazSYNxi4c7ek0TvR1APZEDcN7LVdi3XDc5A3b/y6zcSwhesMCFKro88QDvEO
YwTqznU3gwRPMyAhrINnJIfOdrgLMC8iwqtNDrDqCTJmxGBhTlbRueVXhBcJ9ue9
7i4pdfTx6KVRBVDJNQfp1krITYv6KbqqbqZE4Hui6knXdfamWBkQaCf7uLy4GGv9
RIATMrUBOZK8YNUwIiaaCl+N8X2MybEf7OayWNQsqtkEj7FRX6yR7vNY1rtXprGa
Mfc5QPR0ZmEcbjscpws/
=ZS2K
-----END PGP SIGNATURE-----

Send a report that this bug log contains spam.