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 <[EMAIL PROTECTED]>  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


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to