Author: adam-guest
Date: 2008-04-03 05:51:24 +0000 (Thu, 03 Apr 2008)
New Revision: 1235
Modified:
trunk/debian/changelog
trunk/scripts/checkbashisms.pl
Log:
Add $SHELLOPTS
Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog 2008-04-02 19:19:35 UTC (rev 1234)
+++ trunk/debian/changelog 2008-04-03 05:51:24 UTC (rev 1235)
@@ -15,7 +15,7 @@
("foo && exec bar")
+ Temporarily disable the check for "complete" as it's too prone to false
positives
- + Add checks for pushd, popd, ulimit and shopt
+ + Add checks for pushd, popd, ulimit, shopt and $SHELLOPTS
+ Remove check for $((var+1)) as dash now supports the syntax (the syntax
is POSIX compliant but was not supported properly by dash)
+ Improve quoted string handling to cope with "'foo'"
Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl 2008-04-02 19:19:35 UTC (rev 1234)
+++ trunk/scripts/checkbashisms.pl 2008-04-03 05:51:24 UTC (rev 1235)
@@ -199,6 +199,7 @@
'\$\{?EUID\}?\b' => q<$EUID should be "id -u">,
'\$\{?SECONDS\}?\b' => q<$SECONDS>,
'\$\{?BASH_[A-Z]+\}?\b' => q<$BASH_SOMETHING>,
+ '\$\{?SHELLOPTS\}?\b' => q<$SHELLOPTS>,
'<<<' => q<\<\<\< here string>,
);
@@ -214,6 +215,7 @@
$string_bashisms{'(?:^|\s+)DIRSTACK='} = q<DIRSTACK=>;
$string_bashisms{'(?:^|\s+)EUID='} = q<EUID=>;
$string_bashisms{'(?:^|\s+)BASH(_[A-Z]+)?='} =
q<BASH(_SOMETHING)=>;
+ $string_bashisms{'(?:^|\s+)SHELLOPTS='} = q<SHELLOPTS=>;
}
my $line = $_;
--
To unsubscribe, send mail to [EMAIL PROTECTED]
