The following commit has been merged in the master branch:
commit 8fc08c2bf8f476c60248459607d9a376772611a2
Author: Raphael Geissert <[email protected]>
Date:   Fri May 14 19:25:37 2010 -0500

    checkbashisms: fix a false positive 'should be >word 2>&1'
    
    Signed-off-by: Raphael Geissert <[email protected]>
    Signed-off-by: James Vega <[email protected]>

diff --git a/scripts/checkbashisms.pl b/scripts/checkbashisms.pl
index 5982c20..ecb17f2 100755
--- a/scripts/checkbashisms.pl
+++ b/scripts/checkbashisms.pl
@@ -499,7 +499,7 @@ sub init_hashes {
        qr'(?<![\$\(])\(\(.*\)\)' =>     q<'((' should be '$(('>,
        qr'(?:^|\s+)(\[|test)\s+-a' =>            q<test with unary -a (should 
be -e)>,
        qr'\&>' =>                     q<should be \>word 2\>&1>,
-       qr'(<\&|>\&)\s*((-|\d+)[^\s;|)`&\\\\]|[^-\d\s]+)' =>
+       qr'(<\&|>\&)\s*((-|\d+)[^\s;|)}`&\\\\]|[^-\d\s]+)' =>
                                       q<should be \>word 2\>&1>,
        $LEADIN . qr'kill\s+-[^sl]\w*' => q<kill -[0-9] or -[A-Z]>,
        $LEADIN . qr'trap\s+["\']?.*["\']?\s+.*[1-9]' => q<trap with signal 
numbers>,

-- 
Git repository for devscripts


-- 
To unsubscribe, send mail to [email protected].

Reply via email to