[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[SCM] Debian package checker branch, master, updated. 2.2.12-16-g6d6d780



The following commit has been merged in the master branch:
commit 6d6d7803303773f7508eb4fdb04a1644c7604794
Author: Russ Allbery <rra@debian.org>
Date:   Tue Jun 23 09:46:31 2009 -0700

    Lower severity of read-in-maintainer-script
    
    * checks/scripts.desc:
      + [RA] Lower certainty of read-in-maintainer-script to possible and
        mention false positives.  Thanks, Raphaël Hertzog.  (Closes: #534276)

diff --git a/checks/scripts.desc b/checks/scripts.desc
index c4d94b3..68687d8 100644
--- a/checks/scripts.desc
+++ b/checks/scripts.desc
@@ -310,12 +310,17 @@ Info: Maintainer scripts must not remove device files.  This is left to
 
 Tag: read-in-maintainer-script
 Severity: serious
-Certainty: certain
+Certainty: possible
 Ref: policy 3.9.1 
 Info: This maintainer script appears to use read to get information from
  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.
+ .
+ This check can have false positives if read is used in a block with a
+ redirection, in a function run in a pipe, or in other ways where
+ standard input is provided in inobvious ways.  If this is the case, please
+ add an override for this tag.
 
 Tag: possible-bashism-in-maintainer-script
 Severity: normal
diff --git a/debian/changelog b/debian/changelog
index 78ca5d1..6a01919 100755
--- a/debian/changelog
+++ b/debian/changelog
@@ -24,6 +24,9 @@ lintian (2.2.13) UNRELEASED; urgency=low
   * checks/patch-systems:
     + [RA] Don't include the package name as extra data in tags that are
       only issued for source packages.  Patch from Raphael Geissert.
+  * checks/scripts.desc:
+    + [RA] Lower certainty of read-in-maintainer-script to possible and
+      mention false positives.  Thanks, Raphaël Hertzog.  (Closes: #534276)
 
   * collection/strings:
     + [ADB] Handle the fact that, when Lintian is run as root, the output of

-- 
Debian package checker


Reply to: