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

lintian: r920 - in trunk: checks debian testset/maintainer-scripts/debian



Author: rra
Date: 2007-07-01 18:54:58 +0200 (Sun, 01 Jul 2007)
New Revision: 920

Modified:
   trunk/checks/scripts
   trunk/debian/changelog
   trunk/testset/maintainer-scripts/debian/postrm
Log:
* checks/scripts:
  + [RA] If rm output is redirected to /dev/null, don't think that rm is
    removing /dev/null.  Thanks, Robert Luberda.  (Closes: #431259)

Modified: trunk/checks/scripts
===================================================================
--- trunk/checks/scripts	2007-07-01 16:54:39 UTC (rev 919)
+++ trunk/checks/scripts	2007-07-01 16:54:58 UTC (rev 920)
@@ -613,7 +613,7 @@
 		if (m/^\s*read(?:\s|\z)/ && !$saw_debconf) {
 		    tag "read-in-maintainer-script", "$file:$.";
 		}
-		if (m,^\s*rm\s+(.*\s)?/dev/,) {
+		if (m,^\s*rm\s+([^>]*\s)?/dev/,) {
 		    tag "maintainer-script-removes-device-files", "$file:$.";
 		}
 		if (m,>\s*(/etc/(?:services|protocols|rpc))(\s|\Z),) {

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog	2007-07-01 16:54:39 UTC (rev 919)
+++ trunk/debian/changelog	2007-07-01 16:54:58 UTC (rev 920)
@@ -1,3 +1,11 @@
+lintian (1.23.33) UNRELEASED; urgency=low
+
+  * checks/scripts:
+    + [RA] If rm output is redirected to /dev/null, don't think that rm is
+      removing /dev/null.  Thanks, Robert Luberda.  (Closes: #431259)
+
+ -- Russ Allbery <rra@debian.org>  Sun, 01 Jul 2007 09:54:45 -0700
+
 lintian (1.23.32) unstable; urgency=low
 
   The "stability of output" release.

Modified: trunk/testset/maintainer-scripts/debian/postrm
===================================================================
--- trunk/testset/maintainer-scripts/debian/postrm	2007-07-01 16:54:39 UTC (rev 919)
+++ trunk/testset/maintainer-scripts/debian/postrm	2007-07-01 16:54:58 UTC (rev 920)
@@ -38,3 +38,6 @@
 # This isn't allowed.
 rm /tmp/foo /dev/device
 rm	/dev/device1
+
+# But this is okay.
+rm /tmp/foo > /dev/null



Reply to: