The following commit has been merged in the master branch:
commit cb64fa75862416b3cc359ba58a1fda34e988d4b1
Author: Raphael Geissert <[email protected]>
Date:   Sun Oct 21 00:46:57 2012 -0500

    checkbashisms: detect traps for DEBUG, ERRNO, or RETURN
    
    Signed-off-by: Benjamin Drung <[email protected]>

diff --git a/scripts/checkbashisms.pl b/scripts/checkbashisms.pl
index 861ec6f..ecd64f1 100755
--- a/scripts/checkbashisms.pl
+++ b/scripts/checkbashisms.pl
@@ -595,6 +595,7 @@ sub init_hashes {
 #      $LEADIN . qr'jobs\s+-[^lp]\s' =>  q<'jobs' with option other than -l or 
-p>,
        $LEADIN . qr'command\s+-[^p]\s' =>  q<'command' with option other than 
-p>,
        $LEADIN . qr'setvar\s' =>  q<setvar 'foo' 'bar' should be eval 
'foo="'"$bar"'"'>,
+       $LEADIN . qr'trap\s+["\']?.*["\']?\s+.*(?:ERR|DEBUG|RETURN)' => q<trap 
with ERR|DEBUG|RETURN>,
     );
 
     %string_bashisms = (

-- 
Git repository for devscripts

_______________________________________________
devscripts-devel mailing list
[email protected]
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/devscripts-devel

Reply via email to