Author: adam-guest
Date: 2008-03-16 15:54:53 +0000 (Sun, 16 Mar 2008)
New Revision: 1150
Modified:
trunk/debian/changelog
trunk/scripts/checkbashisms.pl
Log:
+ Also detect <<< inside a quoted string (Closes: #471059)
Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog 2008-03-16 14:08:51 UTC (rev 1149)
+++ trunk/debian/changelog 2008-03-16 15:54:53 UTC (rev 1150)
@@ -47,6 +47,7 @@
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)
+ + Also detect <<< inside a quoted string (Closes: #471059)
* 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-16 14:08:51 UTC (rev 1149)
+++ trunk/scripts/checkbashisms.pl 2008-03-16 15:54:53 UTC (rev 1150)
@@ -156,7 +156,6 @@
'(?:^|\s+)kill\s+-[^sl]\w*' => q<kill -[0-9] or -[A-Z]>,
'(?:^|\s+)trap\s+["\']?.*["\']?\s+.*[1-9]' => q<trap with
signal numbers>,
'\[\[(?!:)' => q<alternative test command ([[ foo ]] should be
[ foo ])>,
- '<<<' => q<\<\<\< here string>,
'/dev/(tcp|udp)' => q</dev/(tcp|udp)>,
'(?:^|\s+)suspend\s' => q<suspend>,
'(?:^|\s+)caller\s' => q<caller>,
@@ -189,6 +188,7 @@
'\$\{?EUID\}?\b' => q<$EUID should be "id -u">,
'\$\{?SECONDS\}?\b' => q<$SECONDS>,
'\$\{?BASH_[A-Z]+\}?\b' => q<$BASH_SOMETHING>,
+ '<<<' => q<\<\<\< here string>,
);
if ($opt_echo) {
--
To unsubscribe, send mail to [EMAIL PROTECTED]
