[SCM] Debian package checker branch, master, updated. 2.2.11-3-g832675a
The following commit has been merged in the master branch:
commit 8ca277ada51df4015a151b142912d25b6011bc37
Author: Russ Allbery <rra@debian.org>
Date: Wed Jun 17 15:10:52 2009 -0700
Using debconf is now mandatory
* checks/scripts.desc:
+ [RA] Per Policy 3.8.2, read-in-maintainer-script is now of serious
severity.
diff --git a/checks/scripts.desc b/checks/scripts.desc
index 201e63e..40be48b 100644
--- a/checks/scripts.desc
+++ b/checks/scripts.desc
@@ -309,13 +309,13 @@ Info: Maintainer scripts must not remove device files. This is left to
the system administrator.
Tag: read-in-maintainer-script
-Severity: normal
+Severity: serious
Certainty: certain
Ref: policy 3.9.1
Info: This maintainer script appears to use read to get information from
- the user. Prompting in maintainer scripts should be done by
- communicating through a program such as debconf which conforms to the
- Debian Configuration management specification, version 2 or higher.
+ the user. Prompting in maintainer scripts must be done by communicating
+ through a program such as debconf which conforms to the Debian
+ Configuration management specification, version 2 or higher.
Tag: possible-bashism-in-maintainer-script
Severity: normal
diff --git a/debian/changelog b/debian/changelog
index 6c474f0..cf7458a 100755
--- a/debian/changelog
+++ b/debian/changelog
@@ -12,6 +12,9 @@ lintian (2.2.12) UNRELEASED; urgency=low
rename the missing-comma-between-items tag to the more descriptive
missing-separator-between-items. Based on a patch by Rafael
Laboissiere. (Closes: #533202)
+ * checks/scripts.desc:
+ + [RA] Per Policy 3.8.2, read-in-maintainer-script is now of serious
+ severity.
-- Adam D. Barratt <adam@adam-barratt.org.uk> Mon, 15 Jun 2009 19:21:07 +0100
diff --git a/testset/tags.maintainer-scripts b/testset/tags.maintainer-scripts
index 74726b3..a0c4fac 100644
--- a/testset/tags.maintainer-scripts
+++ b/testset/tags.maintainer-scripts
@@ -25,6 +25,16 @@ E: maintainer-scripts: postrm-contains-additional-updaterc.d-calls /etc/init.d/b
E: maintainer-scripts: postrm-does-not-call-updaterc.d-for-init.d-script /etc/init.d/foo
E: maintainer-scripts: preinst-calls-updaterc.d bar
E: maintainer-scripts: prerm-calls-updaterc.d foo
+E: maintainer-scripts: read-in-maintainer-script postinst:20
+E: maintainer-scripts: read-in-maintainer-script prerm:177
+E: maintainer-scripts: read-in-maintainer-script prerm:179
+E: maintainer-scripts: read-in-maintainer-script prerm:180
+E: maintainer-scripts: read-in-maintainer-script prerm:181
+E: maintainer-scripts: read-in-maintainer-script prerm:62
+E: maintainer-scripts: read-in-maintainer-script prerm:63
+E: maintainer-scripts: read-in-maintainer-script prerm:64
+E: maintainer-scripts: read-in-maintainer-script prerm:65
+E: maintainer-scripts: read-in-maintainer-script prerm:66
E: maintainer-scripts: wrong-debian-qa-group-name QA group <packages@qa.debian.org>
I: maintainer-scripts: no-md5sums-control-file
I: maintainer-scripts: output-of-updaterc.d-not-redirected-to-dev-null bar postrm
@@ -153,15 +163,5 @@ W: maintainer-scripts: postinst-does-not-load-confmodule
W: maintainer-scripts: postinst-should-not-set-usr-doc-link
W: maintainer-scripts: postrm-does-not-purge-debconf
W: maintainer-scripts: postrm-has-useless-call-to-ldconfig
-W: maintainer-scripts: read-in-maintainer-script postinst:20
-W: maintainer-scripts: read-in-maintainer-script prerm:177
-W: maintainer-scripts: read-in-maintainer-script prerm:179
-W: maintainer-scripts: read-in-maintainer-script prerm:180
-W: maintainer-scripts: read-in-maintainer-script prerm:181
-W: maintainer-scripts: read-in-maintainer-script prerm:62
-W: maintainer-scripts: read-in-maintainer-script prerm:63
-W: maintainer-scripts: read-in-maintainer-script prerm:64
-W: maintainer-scripts: read-in-maintainer-script prerm:65
-W: maintainer-scripts: read-in-maintainer-script prerm:66
W: maintainer-scripts: start-stop-daemon-in-maintainer-script postinst:161
W: maintainer-scripts: update-alternatives-remove-called-in-postrm
--
Debian package checker
Reply to: