Bug#344609: marked as done (lintian: better build-depends-indep check)
Your message dated Thu, 19 Jan 2006 07:17:09 -0800
with message-id <E1EzbXB-0000YE-8K@spohr.debian.org>
and subject line Bug#344609: fixed in lintian 1.23.15
has caused the attached Bug report to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what I am
talking about this indicates a serious mail system misconfiguration
somewhere. Please contact me immediately.)
Debian bug tracking system administrator
(administrator, Debian Bugs database)
--------------------------------------
Received: (at submit) by bugs.debian.org; 24 Dec 2005 01:34:25 +0000
>From rra@debian.org Fri Dec 23 17:34:24 2005
Return-path: <rra@debian.org>
Received: from smtp1.stanford.edu ([171.67.16.123])
by spohr.debian.org with esmtp (Exim 4.50)
id 1EpyIi-0002wG-SL
for submit@bugs.debian.org; Fri, 23 Dec 2005 17:34:24 -0800
Received: from windlord.stanford.edu (windlord.Stanford.EDU [171.64.19.147])
by smtp1.Stanford.EDU (8.12.11/8.12.11) with ESMTP id jBO1YNkf000838
for <submit@bugs.debian.org>; Fri, 23 Dec 2005 17:34:23 -0800
Received: by windlord.stanford.edu (Postfix, from userid 1000)
id 590D1E78E2; Fri, 23 Dec 2005 17:34:23 -0800 (PST)
Content-Type: multipart/mixed; boundary="===============1435314799=="
MIME-Version: 1.0
From: Russ Allbery <rra@debian.org>
To: Debian Bug Tracking System <submit@bugs.debian.org>
Subject: lintian: better build-depends-indep check
Message-ID: <20051224013123.20400.7543.reportbug@wanderer.stanford.edu>
X-Mailer: reportbug 3.18
Date: Fri, 23 Dec 2005 17:31:23 -0800
Delivered-To: submit@bugs.debian.org
X-Spam-Checker-Version: SpamAssassin 2.60-bugs.debian.org_2005_01_02
(1.212-2003-09-23-exp) on spohr.debian.org
X-Spam-Level:
X-Spam-Status: No, hits=-8.0 required=4.0 tests=BAYES_00,HAS_PACKAGE
autolearn=no version=2.60-bugs.debian.org_2005_01_02
This is a multi-part MIME message sent by reportbug.
--===============1435314799==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
Package: lintian
Version: 1.23.14
Severity: normal
Tags: patch
While sponsoring a package (cdrbq), I noticed that lintian complaints about
build-depends-without-arch-dep for Build-Depends: dpatch. However, if one
uses the makefile include method for integrating dpatch, dpatch does have
to be installed to run make clean and therefore should be in Build-Depends.
Similar problems exist for cdbs and dbs.
While I was fixing this, I noticed that the regex in the existing check is
insufficiently conservative and:
Build-Depends: debhelper (>= 4.0.0), python-dev (>> 2.3)
will be judged as fine because \(.+?\) will pick up everything to the end
of the line. +? just stops matching at the earliest point it *can* and
not change the success of the overall match; if it has to match farther
in order to make the whole expression match, it will.
This patch pulls the database of things to look for out into a global
variable, makes the check easier to extend down the road, and also adds
a special case for yada packages (allowing yada in Build-Depends if the
debian/packages file exists). Applying this patch will therefore also
close #321135.
I considered also ensuring that the packages discovered were listed in
Build-Depends, but I think that's too likely to turn up false positives
(if, for instance, someone did something sneaky and made the inclusion
of the dpatch makefile fragment conditional). But it wouldn't be too
hard to add if it turns out to be a good idea.
-- System Information:
Debian Release: testing/unstable
APT prefers unstable
APT policy: (500, 'unstable'), (500, 'testing')
Architecture: i386 (i686)
Shell: /bin/sh linked to /bin/bash
Kernel: Linux 2.6.12-1-686
Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968) (ignored: LC_ALL set to C)
Versions of packages lintian depends on:
ii binutils 2.16.1cvs20051214-1 The GNU assembler, linker and bina
ii diffstat 1.41-1 produces graph of changes introduc
ii dpkg-dev 1.13.11 package building tools for Debian
ii file 4.15-2 Determines file type using "magic"
ii gettext 0.14.5-2 GNU Internationalization utilities
ii intltool-debian 0.34.1+20050828 Help i18n of RFC822 compliant conf
ii libparse-debianchang 1.0-1 parse Debian changelogs and output
ii man-db 2.4.3-3 The on-line manual pager
ii perl [libdigest-md5- 5.8.7-10 Larry Wall's Practical Extraction
lintian recommends no packages.
-- no debconf information
--===============1435314799==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="lintian.patch"
diff -ru lintian-1.23.14/checks/fields lintian-new/checks/fields
--- lintian-1.23.14/checks/fields 2005-08-12 16:44:18.000000000 -0700
+++ lintian-new/checks/fields 2005-12-23 17:23:11.000000000 -0800
@@ -29,6 +29,28 @@
use Tags;
use Util;
+# Certain build tools must be listed in Build-Depends even if there are no
+# arch-specific packages because they're required in order to run the clean
+# rule. (See Policy 7.6.) The following is a list of pairs of packages and
+# regular expressions that, if they match anywhere in the debian/rules file,
+# say that this package is allowed in Build-Depends.
+my @global_depends = (
+ [ cdbs => '^include\s+/usr/share/cdbs/' ],
+ [ dbs => '^include\s+/usr/share/dbs/' ],
+ [ debhelper => '^include\s+/usr/share/cdbs/1/rules/debhelper.mk' ],
+ [ dpatch => '^include\s+/usr/share/dpatch/' ]
+);
+
+# Similarly, these pairs of packages and regexes say that if the regex matches
+# in one of clean, build-arch, or binary-arch, this package is allowed in
+# Build-Depends.
+my @rule_depends = (
+ [ debhelper => '^\s+dh_.+' ]
+);
+
+# Note that yada is handled as a special case, based on the existence of
+# debian/packages.
+
sub run {
my $pkg = shift;
@@ -403,23 +425,34 @@
my $build_depends = <BD>;
close BD;
- my $uses_dh = 0;
+ my @allowed;
if (not open (RULES, "debfiles/rules")) {
fail("cannot read debfiles/rules: $!");
} else {
my $target = "none";
local $/ = "\n"; #Read this linewise
while (<RULES>) {
+ for my $rule (@global_depends) {
+ if ($_ =~ /$rule->[1]/) {
+ push (@allowed, $rule->[0]);
+ }
+ }
$target = $1 if (/^(\S+):/);
- if (/^\s+dh_.+/ && grep ($_ eq $target, qw(clean binary-arch build-arch)) or
- m#^include\s+/usr/share/cdbs/1/rules/debhelper.mk#) {
- $uses_dh = "yes";
- last
+ if (grep ($_ eq $target, qw(clean binary-arch build-arch))) {
+ for my $rule (@rule_depends) {
+ if ($_ =~ /$rule->[1]/) {
+ push (@allowed, $rule->[0]);
+ }
+ }
}
}
close RULES;
}
- unless ($build_depends =~ /^\s*debhelper(?:\s+\((.+?)\))?(?:\s+(\[.+?\]))?\s*$/ && $uses_dh){
+ my $packages = join ('|', @allowed);
+ if (-e "debfiles/packages") {
+ $packages .= '|yada';
+ }
+ unless ($build_depends =~ /^(?:\s*(?:$packages)(?:\s+\(([^\)]+?)\))?(?:\s+(\[[^\]]+?\]))?\s*,?)*$/) {
tag "build-depends-without-arch-dep", ""
}
}
--===============1435314799==--
---------------------------------------
Received: (at 344609-close) by bugs.debian.org; 19 Jan 2006 15:21:13 +0000
>From katie@ftp-master.debian.org Thu Jan 19 07:21:13 2006
Return-path: <katie@ftp-master.debian.org>
Received: from katie by spohr.debian.org with local (Exim 4.50)
id 1EzbXB-0000YE-8K; Thu, 19 Jan 2006 07:17:09 -0800
From: Frank Lichtenheld <djpig@debian.org>
To: 344609-close@bugs.debian.org
X-Katie: $Revision: 1.65 $
Subject: Bug#344609: fixed in lintian 1.23.15
Message-Id: <E1EzbXB-0000YE-8K@spohr.debian.org>
Sender: Archive Administrator <katie@ftp-master.debian.org>
Date: Thu, 19 Jan 2006 07:17:09 -0800
X-Spam-Checker-Version: SpamAssassin 2.60-bugs.debian.org_2005_01_02
(1.212-2003-09-23-exp) on spohr.debian.org
X-Spam-Level:
X-Spam-Status: No, hits=-6.0 required=4.0 tests=BAYES_00,HAS_BUG_NUMBER
autolearn=no version=2.60-bugs.debian.org_2005_01_02
X-CrossAssassin-Score: 9
Source: lintian
Source-Version: 1.23.15
We believe that the bug you reported is fixed in the latest version of
lintian, which is due to be installed in the Debian FTP archive:
lintian_1.23.15.dsc
to pool/main/l/lintian/lintian_1.23.15.dsc
lintian_1.23.15.tar.gz
to pool/main/l/lintian/lintian_1.23.15.tar.gz
lintian_1.23.15_all.deb
to pool/main/l/lintian/lintian_1.23.15_all.deb
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 344609@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.
Debian distribution maintenance software
pp.
Frank Lichtenheld <djpig@debian.org> (supplier of updated lintian 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@debian.org)
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Format: 1.7
Date: Thu, 19 Jan 2006 15:13:02 +0100
Source: lintian
Binary: lintian
Architecture: source all
Version: 1.23.15
Distribution: unstable
Urgency: low
Maintainer: Debian Lintian Maintainers <lintian-maint@debian.org>
Changed-By: Frank Lichtenheld <djpig@debian.org>
Description:
lintian - Debian package checker
Closes: 234531 321135 329953 339750 343865 344609 344899 344928 345005 346335 347999 348380
Changes:
lintian (1.23.15) unstable; urgency=low
.
* The "www.no-name-yet.eu" release
.
* debian/changelog:
+ [FL] Fix wrapping of last changelog entry (Closes: #344899)
.
* checks/common_data.pm:
+ [FL] Add xlibs-dev and debmake to %known_obsolete_packages
* checks/debconf:
+ [CW] Show template name for empty-translated-choices,
mismatch-translated-choices, and unknown-field-in-templates.
* checks/debhelper:
+ [FL] Also check for debhelper build-dependency in debhelper-using
cdbs based packages
+ [FL] Indicate where we found which compat version in
declares-possibly-conflicting-debhelper-compat-versions
(Closes: #329953)
+ [FL] Add dh_installtexfonts to list of commands that modfiy
the maintainer scripts
+ [FL] Try a little harder to find the DH_COMPAT setting in debian/rules
(Closes: #348380)
* checks/fields:
+ [FL] Allow version numbers of 0 by correctly checking the output
of _valid_version for definedness and not the boolean value
(Closes: #345005)
+ [FL] Better check for false postives of build-depends-without-arch-dep.
Should fix false positives for cdbs, dbs, dpatch and yada. Based on
a patch by Russ Allbery (Closes: #344609, #321135, #339750)
+ [FL] Also check for build-dependencies on obsolete packages not
only dependencies (Closes: #234531)
* checks/fields.desc:
+ [JvW] Fix typo in description
* checks/init.d:
+ [FL] Only try to validate update-rc.d calls where the name of the
script matches [\w.-]+. That accounts for all today known init
scripts and eliminates false positives like shell variables in the
name (Closes: #343865)
* checks/manpages:
+ [FL] Fix stupid mistake in regex to strip of filenames from roff
output (which was hidden by another stupid mistake in the test
suite). Patch by Nicolas François (Closes: #347999)
* checks/scripts:
+ [FL] Make mknod-in-maintainer-script an error since this is now
a must in policy. Pointed out by Bartosz Fenski (Closes: #344928)
+ [FL] dpkg --print-installation-architecture is deprecated
so delete dpkg-print-architecture-in-maintainer-script
+ [FL] Add parrot to list of valid interpreters (Closes: #346335)
Files:
1cb9288f27a0158fc95a87839f3dea83 772 devel optional lintian_1.23.15.dsc
64e888db4f4a298e3ea1a1c4aab5693e 270918 devel optional lintian_1.23.15.tar.gz
b8d43dbde7a88efb563ccdbdab8c065f 235350 devel optional lintian_1.23.15_all.deb
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (GNU/Linux)
iD8DBQFDz6q8Qbn06FtxPfARAioXAJ4gSTA8GVVkSx/4W/t3rNIH8I+HAACeMHSg
Zs19ZJ9ZD1XukHKu9y/z8IQ=
=Mc6Q
-----END PGP SIGNATURE-----
Reply to: