Author: adam-guest
Date: 2008-03-05 01:05:38 +0000 (Wed, 05 Mar 2008)
New Revision: 1103

Modified:
   trunk/debian/changelog
   trunk/scripts/checkbashisms.pl
Log:
  + Fix a potential false positive on "2>& 1"

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog      2008-03-05 00:29:59 UTC (rev 1102)
+++ trunk/debian/changelog      2008-03-05 01:05:38 UTC (rev 1103)
@@ -19,6 +19,7 @@
     + Identify and ignore multi-line quoted text (Closes: #464641)
     + Reorder a couple of checks so that ". foo $HOSTNAME" is detected as
       source passing arguments rather than misuse of $HOSTNAME
+    + Fix a potential false positive on "2>& 1"
   * bts: Allow the sendmail command to begin with a ~ (Closes: #469207)
   * debcommit: Set $SVKDIFF before calling "svk diff" (Closes: #469238)
 

Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl      2008-03-05 00:29:59 UTC (rev 1102)
+++ trunk/scripts/checkbashisms.pl      2008-03-05 01:05:38 UTC (rev 1103)
@@ -137,7 +137,7 @@
                '(?<![\$\(])\(\(.*\)\)' =>     q<'((' should be '$(('>,
                '(\[|test)\s+-a' =>            q<test with unary -a (should be 
-e)>,
                '\&>' =>                       q<should be \>word 2\>&1>,
-               '(<\&|>\&)\s*((-|\d+)[^\s;|)`&]|[^-\d])' =>
+               '(<\&|>\&)\s*((-|\d+)[^\s;|)`&]|[^-\d\s])' =>
                                               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