Author: adam-guest
Date: 2008-03-15 15:52:36 +0000 (Sat, 15 Mar 2008)
New Revision: 1137
Modified:
trunk/debian/changelog
trunk/scripts/checkbashisms.pl
Log:
+ Add a check for "VAR+=foo" (Closes: #471001)
Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog 2008-03-15 15:40:30 UTC (rev 1136)
+++ trunk/debian/changelog 2008-03-15 15:52:36 UTC (rev 1137)
@@ -40,6 +40,7 @@
but which may contain other useful information - for example, checking
whether $BASH is set before using bashisms or $RANDOM being used as a
local variable rather than the bash built-in variable (Closes: #470999)
+ + Add a check for "VAR+=foo" (Closes: #471001)
* bts:
+ Allow the sendmail command to begin with a ~ (Closes: #469207)
+ Don't treat "userblah" as an alias for "users" in "select"
Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl 2008-03-15 15:40:30 UTC (rev 1136)
+++ trunk/scripts/checkbashisms.pl 2008-03-15 15:52:36 UTC (rev 1137)
@@ -141,6 +141,7 @@
'\s\|\&' => q<pipelining is not POSIX>,
'[^\\\]\{([^\s]+?,)+[^\\\}\s]+\}' =>
q<brace expansion>,
+ '(?:^|\s+)\w+\+=' => q<should be VAR="$VAR foo">,
'(?:^|\s+)\w+\[\d+\]=' => q<bash arrays, H[0]>,
'(?:^|\s+)(read\s*(-[^r])?(?:;|$))' => q<should be read [-r]
variable>,
'\$\(\([A-Za-z]' => q<cnt=$((cnt + 1)) does not work in dash>,
--
To unsubscribe, send mail to [EMAIL PROTECTED]
