[lintian] 01/02: c/filename-length: Skip on auto-generated pkgs (e.g. dbgsym)
This is an automated email from the git hooks/post-receive script.
nthykier pushed a commit to branch master
in repository lintian.
commit 792b671cb108e18395fbd54a5cf41ab82d6e2433
Author: Niels Thykier <niels@thykier.net>
Date: Sat Sep 16 10:51:08 2017 +0000
c/filename-length: Skip on auto-generated pkgs (e.g. dbgsym)
Signed-off-by: Niels Thykier <niels@thykier.net>
---
checks/filename-length.pm | 5 +++++
debian/changelog | 7 ++++++-
lib/Lintian/Collect/Binary.pm | 11 +++++++++++
3 files changed, 22 insertions(+), 1 deletion(-)
diff --git a/checks/filename-length.pm b/checks/filename-length.pm
index 9d15c01..fb0ca60 100644
--- a/checks/filename-length.pm
+++ b/checks/filename-length.pm
@@ -41,6 +41,11 @@ sub run {
my $len = length($pkg) + length($proc->pkg_version) + 1;
my $extra;
+ # Skip auto-generated packages (dbgsym)
+ return
+ if ($type eq 'binary' or $type eq 'udeb')
+ and $info->is_pkg_class('auto-generated');
+
if ($type eq 'binary' || $type eq 'source'){
# Here we add length .deb / .dsc (in both cases +4)
$len += 4;
diff --git a/debian/changelog b/debian/changelog
index 3f21085..569af92 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -51,7 +51,7 @@ lintian (2.5.53) UNRELEASED; urgency=medium
* checks/debian-readme.{pm,desc}:
+ [CL] Ensure readme-debian-contains-debmake-template also checks
for templates "Automatically generated by debmake".
- * checks/description.desc::
+ * checks/description.desc:
+ [CL] Clarify explanation of description-starts-with-leading-spaces
tag. Thanks to Taylor Kline <taylor.kline@utexas.edu> for the report
and patch. (Closes: #849622)
@@ -62,6 +62,9 @@ lintian (2.5.53) UNRELEASED; urgency=medium
+ [BR] Use Lintian::Data for name/section mapping
+ [CL] Check for packages including "?rev=0&sc=0" in Vcs-Browser.
(Closes: #681713)
+ * checks/filename-length.pm:
+ + [NT] Skip the check on auto-generated binary packages (such as
+ dbgsym packages).
* checks/files.{pm,desc}:
+ [BR] Avoid privacy-breach-generic false positives for legal.xml.
+ [BR] Detect install of node package under /usr/lib/nodejs/[^/]*$
@@ -144,6 +147,8 @@ lintian (2.5.53) UNRELEASED; urgency=medium
* lib/Lintian/Check.pm:
+ [CL] Stop emitting {maintainer,uploader}-address-causes-mail-loops
for @packages.debian.org addresses. (Closes: #871575)
+ * lib/Lintian/Collect/Binary.pm:
+ + [NT] Introduce an "auto-generated" argument for "is_pkg_class".
* lib/Lintian/Data.pm:
+ [CL] Modify Lintian::Data's "all" to always return keys in insertion
order, dropping dependency on libtie-ixhash-perl.
diff --git a/lib/Lintian/Collect/Binary.pm b/lib/Lintian/Collect/Binary.pm
index 6f391c7..62df829 100644
--- a/lib/Lintian/Collect/Binary.pm
+++ b/lib/Lintian/Collect/Binary.pm
@@ -661,6 +661,13 @@ The package is (probably) a package containing debug symbols.
Guessed from the package name.
+=item auto-generated
+
+The package is (probably) a package generated automatically (e.g. a
+dbgsym package)
+
+Guessed from the "Auto-Built-Package" field.
+
=back
Needs-Info requirements for using I<is_pkg_class>: L<Same as field|Lintian::Collect/field ([FIELD[, DEFAULT]])>
@@ -680,6 +687,10 @@ Needs-Info requirements for using I<is_pkg_class>: L<Same as field|Lintian::Coll
return 1 if $self->name =~ m/-dbg(?:sym)?/;
return;
}
+ if ($pkg_class eq 'auto-generated') {
+ return 1 if $self->field('auto-built-package');
+ return;
+ }
return 1 if $desc =~ m/transitional package/;
$desc = lc($desc);
if ($pkg_class eq 'any-meta') {
--
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git
Reply to: