The following commit has been merged in the master branch:
commit bc72f45f4c09dc5d3764397d72644de17916559b
Author: Raphael Geissert <[email protected]>
Date: Wed Oct 31 20:35:05 2012 -0600
checkbashisms: check for other forms of brace expansion
Signed-off-by: Raphael Geissert <[email protected]>
Signed-off-by: Benjamin Drung <[email protected]>
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
_______________________________________________
devscripts-devel mailing list
[email protected]
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/devscripts-devel
