Bug#719788: lintian: downgrade warning for not using ${phppear:description} to wishlist
Package: lintian
Version: 2.5.15
Severity: normal
Tags: patch
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256
Dear Maintainer,
In bug #708551, a warning was introduced for PEAR packages not using
${phppear:description} and ${phppear:summary}. On some PEAR packages,
the provided description are poor (starting with the package name for
example). Therefore, the use of those macros should not be done blindly.
I propose this patch to
1. downgrade to wishlist the warning
2. explain to not blindly rely on PEAR package descriptions
- -- System Information:
Debian Release: jessie/sid
APT prefers unstable
APT policy: (500, 'unstable'), (500, 'stable'), (101, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386
Kernel: Linux 3.10-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=fr_FR.utf8, LC_CTYPE=fr_FR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Versions of packages lintian depends on:
ii binutils 2.23.52.20130727-1
ii bzip2 1.0.6-5
ii diffstat 1.55-3
ii file 1:5.14-2
ii gettext 0.18.3-1
ii hardening-includes 2.3
ii intltool-debian 0.35.0+20060710.1
ii libapt-pkg-perl 0.1.29
ii libarchive-zip-perl 1.30-7
ii libclass-accessor-perl 0.34-1
ii libclone-perl 0.34-1
ii libdpkg-perl 1.17.1
ii libemail-valid-perl 0.190-1
ii libfile-basedir-perl 0.03-1
ii libipc-run-perl 0.92-1
ii liblist-moreutils-perl 0.33-1+b1
ii libparse-debianchangelog-perl 1.2.0-1
ii libtext-levenshtein-perl 0.06~01-2
ii libtimedate-perl 1.2000-1
ii liburi-perl 1.60-1
ii man-db 2.6.5-2
ii patchutils 0.3.2-2
ii perl [libdigest-sha-perl] 5.14.2-21
ii t1utils 1.37-2
Versions of packages lintian recommends:
ii libautodie-perl 2.20-1
ii libperlio-gzip-perl 0.18-1+b2
ii perl-modules [libautodie-perl] 5.14.2-21
Versions of packages lintian suggests:
pn binutils-multiarch <none>
ii dpkg-dev 1.17.1
ii libhtml-parser-perl 3.71-1
ii libtext-template-perl 1.45-2
ii xz-utils 5.1.1alpha+20120614-2
- -- no debconf information
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.14 (GNU/Linux)
iQIcBAEBCAAGBQJSDLN9AAoJEJWkL+g1NSX5F98P/iu1RN7+v8spPGv3qEqwyOvL
lW3oQfL0Z1rbh0/YOkW+NxXR2+s0Rfow2m1sSH1+5Pk8xS6yXHCEaILhiV8+/fDo
/Hnbx3aLprXEMwliy02xRVM695tXKn4oQz4daDPhhsj5jSPKqhl0BhfvDmBGdO0C
F8zghhK/a7/LdgKTZwXXn/mHI76ofXbROUGPgpMuZtQzkn0I3IQSjxp1jRVKq75U
2fd4BxJEPP70SQZeSGVWD+hW77ifx0C5HH+Pqal40nXjbJrjx0MhoPj8YzFBLx1X
WDqvw1WAp4g53kp/OZhJ/YEeAjZwWUw88vUoxts1PbFs+1Rnnfa+pnJGfMWOJzx+
jei5jM8ctJOjZMndClXxh2qMhKPg+oq9PBFxmO+z/3A+uY/eGtxPrikFd4p84sBB
XqIzX+xzQ8wUL1exGc3K6rq6JYsXJbHMRQ8Fqo5vAF0iz0+FjDeTbC0/rGSGYuby
1QKhSQ2oEggYyk2YlEwCHVrXqmaUQ8GoMXKQM7Er7YZJkkezZXkCaNy4rlWOIBiM
P3d6Gome7uXS3bSOn39ftz2QpDlasurZeQbJvTPXQEST+YHkUw3xe1xA+QIP+3k7
RvaIHO/AKjDVFDbyhgVFO/qbgEy9NqX9+yrisreWc3gQlfUmYXKdDrsfnifD8Hfj
yiYhC0IwDqvTnPR61iaD
=cVDe
-----END PGP SIGNATURE-----
>From 9ca0a9d4ec009d5fb41ffda7678cf5de5780bdd6 Mon Sep 17 00:00:00 2001
From: Vincent Bernat <bernat@luffy.cx>
Date: Thu, 15 Aug 2013 11:50:17 +0200
Subject: [PATCH] Downgrade check for ${phppear:description} and
${phppear:summary}
The check is downgraded to wishlist and the description is enhanced to
explain when the provided description can be used. A reference to
Debian packaging best practices is also used.
---
checks/phppear.desc | 7 ++++++-
t/tests/phppear-pear/tags | 4 ++--
2 files changed, 8 insertions(+), 3 deletions(-)
diff --git a/checks/phppear.desc b/checks/phppear.desc
index b63127f..08b61d0 100644
--- a/checks/phppear.desc
+++ b/checks/phppear.desc
@@ -31,10 +31,15 @@ Info: The package is a PEAR package but its control file doesn't have
${phppear:Debian-Depends} in Depends or ${phppear:Debian-Recommends}.
Tag: pear-package-not-using-substvar
-Severity: normal
+Severity: wishlist
Certainty: possible
Info: The package is a PEAR package but its control file doesn't have
${phppear:summary} or ${phppear:description} in its description fields.
+ .
+ Those substitution variables should only be used if the description
+ included in PEAR package is of sufficient quality and respect Debian
+ packaging best practices.
+Ref: http://www.debian.org/doc/manuals/developers-reference/best-pkging-practices.html#bpp-desc-basics
Tag: pear-package-feature-requires-newer-pkg-php-tools
Severity: normal
diff --git a/t/tests/phppear-pear/tags b/t/tests/phppear-pear/tags
index b819015..e01731b 100644
--- a/t/tests/phppear-pear/tags
+++ b/t/tests/phppear-pear/tags
@@ -1,3 +1,5 @@
+I: phppear-pear source: pear-package-not-using-substvar ${phppear:description}
+I: phppear-pear source: pear-package-not-using-substvar ${phppear:summary}
W: phppear-pear source: missing-pkg-php-tools-addon php5
W: phppear-pear source: missing-pkg-php-tools-addon phppear
W: phppear-pear source: missing-pkg-php-tools-buildsystem phppear
@@ -7,7 +9,5 @@ W: phppear-pear source: pear-package-but-missing-dependency Recommends
W: phppear-pear source: pear-package-feature-requires-newer-pkg-php-tools (>= 1.4~) for package2.xml
W: phppear-pear source: pear-package-feature-requires-newer-pkg-php-tools (>= 1.5~) for PECL support
W: phppear-pear source: pear-package-feature-requires-newer-pkg-php-tools (>= 1~) for package name overrides
-W: phppear-pear source: pear-package-not-using-substvar ${phppear:description}
-W: phppear-pear source: pear-package-not-using-substvar ${phppear:summary}
W: phppear-pear source: pecl-package-requires-build-dependency dh-php5
W: phppear-pear source: pecl-package-requires-build-dependency php5-dev
--
1.8.4.rc2
Reply to: