Author: adam-guest
Date: 2008-02-29 08:32:17 +0000 (Fri, 29 Feb 2008)
New Revision: 1085

Modified:
   trunk/debian/changelog
   trunk/scripts/checkbashisms.pl
Log:
checkbashsisms: Simplify the >&/&>file check and in the process fix
  another couple of potential false positives.

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog      2008-02-29 06:51:33 UTC (rev 1084)
+++ trunk/debian/changelog      2008-02-29 08:32:17 UTC (rev 1085)
@@ -1,3 +1,10 @@
+devscripts (2.10.19) UNRELEASED; urgency=low
+
+  * checkbashsisms: Simplify the >&/&>file check and in the process fix
+    another couple of potential false positives.
+
+ -- Adam D. Barratt <[EMAIL PROTECTED]>  Fri, 29 Feb 2008 08:31:10 +0000
+
 devscripts (2.10.18) unstable; urgency=low
 
   [ Patrick Schoenfeld ]

Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl      2008-02-29 06:51:33 UTC (rev 1084)
+++ trunk/scripts/checkbashisms.pl      2008-02-29 08:32:17 UTC (rev 1085)
@@ -130,7 +130,7 @@
                '(?<![\$\(])\(\(.*\)\)' =>     q<'((' should be '$(('>,
                '(\[|test)\s+-a' =>            q<test with unary -a (should be 
-e)>,
                '\&>' =>                       q<should be \>word 2\>&1>,
-               '(<\&|>\&)\s*(-[^\s;|]|\d+[^\s;|)]|[^-\d\s;|])' =>
+               '(<\&|>\&)\s*((-|\d+)[^\s;|)`&]|[^-\d])' =>
                                               q<should be \>word 2\>&1>,
                '(?:^|\s+)kill\s+-[^sl]\w*' => q<kill -[0-9] or -[A-Z]>,
                '(?:^|\s+)trap\s+["\']?.*["\']?\s+.*[1-9]' => q<trap with 
signal numbers>,



-- 
To unsubscribe, send mail to [EMAIL PROTECTED]

Reply via email to