Author: adam-guest
Date: 2008-04-02 05:56:46 +0000 (Wed, 02 Apr 2008)
New Revision: 1233
Modified:
trunk/debian/changelog
trunk/scripts/checkbashisms.pl
Log:
+ Add checks for ulimit and shopt
Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog 2008-04-01 19:54:58 UTC (rev 1232)
+++ trunk/debian/changelog 2008-04-02 05:56:46 UTC (rev 1233)
@@ -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 and popd
+ + Add checks for pushd, popd, ulimit and shopt
+ 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-01 19:54:58 UTC (rev 1232)
+++ trunk/scripts/checkbashisms.pl 2008-04-02 05:56:46 UTC (rev 1233)
@@ -180,6 +180,8 @@
'(?:^|\s+)\s*\(?\w*[^\(\w\s]+\S*?\s*[^\"]\(\)' => q<function
names should only contain [a-z0-9_]>,
'(?:^|\s+)(push|pod)d\b' => q<(push|pod)d>,
'(?:^|\s+)export\s+-[^p]' => q<export only takes -p as an
option>,
+ '(?:^|\s+)ulimit\b' => q<ulimit>,
+ '(?:^|\s+)shopt\b' => q<shopt>,
);
my %string_bashisms = (
--
To unsubscribe, send mail to [EMAIL PROTECTED]
