Author: adam-guest
Date: 2008-03-26 22:07:45 +0000 (Wed, 26 Mar 2008)
New Revision: 1210
Modified:
trunk/debian/changelog
trunk/scripts/checkbashisms.pl
Log:
+ Modify the brace expansion test to allow "${foo},${bar}"
Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog 2008-03-26 21:50:43 UTC (rev 1209)
+++ trunk/debian/changelog 2008-03-26 22:07:45 UTC (rev 1210)
@@ -8,6 +8,7 @@
+ Fix heredoc detection to match '<< \EOT ... EOT' but not
'<< "\EOT" ... EOT'
+ Improve the invalid function name test to not trigger on IFS="()"
+ + Modify the brace expansion test to allow "${foo},${bar}"
* debcheckout:
+ Tighten the "repository owner" regex for darcs.d.o to correctly check
http://darcs.d.o/~foo/bar/baz against user foo
Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl 2008-03-26 21:50:43 UTC (rev 1209)
+++ trunk/scripts/checkbashisms.pl 2008-03-26 22:07:45 UTC (rev 1210)
@@ -147,7 +147,7 @@
'(\[|test|-o|-a)\s*[^\s]+\s+==\s' =>
q<should be 'b = a'>,
'\s\|\&' => q<pipelining is not POSIX>,
- '[^\\\]\{([^\s]+?,)+[^\\\}\s]+\}' =>
+ '[^\\\]\{([^\s\\\}]+?,)+[^\\\}\s]+\}' =>
q<brace expansion>,
'(?:^|\s+)\w+\+=' => q<should be VAR="${VAR}foo">,
'(?:^|\s+)\w+\[\d+\]=' => q<bash arrays, H[0]>,
--
To unsubscribe, send mail to [EMAIL PROTECTED]
