[SCM] Git repository for devscripts branch, master, updated. v2.12.4-72-gcf18b4b

Benjamin Drung bdrung at debian.org
Sat Nov 10 23:00:35 UTC 2012


The following commit has been merged in the master branch:
commit bc72f45f4c09dc5d3764397d72644de17916559b
Author: Raphael Geissert <geissert at debian.org>
Date:   Wed Oct 31 20:35:05 2012 -0600

    checkbashisms: check for other forms of brace expansion
    
    Signed-off-by: Raphael Geissert <geissert at debian.org>
    Signed-off-by: Benjamin Drung <bdrung at debian.org>

diff --git a/scripts/checkbashisms.pl b/scripts/checkbashisms.pl
index 7ea977d..a61777b 100755
--- a/scripts/checkbashisms.pl
+++ b/scripts/checkbashisms.pl
@@ -569,7 +569,8 @@ sub init_hashes {
 	qr'\[\s+[^\]]+\s+==\s' =>        q<should be 'b = a'>,
 	qr'\s\|\&' =>                    q<pipelining is not POSIX>,
 	qr'[^\\\$]\{([^\s\\\}]*?,)+[^\\\}\s]*\}' => q<brace expansion>,
-	qr'\{\d+\.\.\d+\}' =>          q<brace expansion, should be $(seq a b)>,
+	qr'\{\d+\.\.\d+(?:\.\.\d+)?\}' =>          q<brace expansion, {a..b[..c]}should be $(seq a [c] b)>,
+	qr'(?i)\{[a-z]\.\.[a-z](?:\.\.\d+)?\}' =>          q<brace expansion>,
 	qr'(?:^|\s+)\w+\[\d+\]=' =>      q<bash arrays, H[0]>,
 	$LEADIN . qr'read\s+(?:-[a-qs-zA-Z\d-]+)' => q<read with option other than -r>,
 	$LEADIN . qr'read\s*(?:-\w+\s*)*(?:\".*?\"|[\'].*?[\'])?\s*(?:;|$)'

-- 
Git repository for devscripts



More information about the devscripts-devel mailing list