Author: adam-guest
Date: 2008-04-01 19:54:58 +0000 (Tue, 01 Apr 2008)
New Revision: 1232
Modified:
trunk/debian/changelog
trunk/scripts/checkbashisms.pl
Log:
+ Add a test for export being passed options other than -p
(Closes: #473700)
Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog 2008-04-01 17:30:36 UTC (rev 1231)
+++ trunk/debian/changelog 2008-04-01 19:54:58 UTC (rev 1232)
@@ -19,6 +19,8 @@
+ 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'"
+ + Add a test for export being passed options other than -p
+ (Closes: #473700)
* dcmd: Apply patch from Romain to correctly match the x11 section
* debcheckout:
+ Tighten the "repository owner" regex for darcs.d.o to correctly check
Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl 2008-04-01 17:30:36 UTC (rev 1231)
+++ trunk/scripts/checkbashisms.pl 2008-04-01 19:54:58 UTC (rev 1232)
@@ -179,6 +179,7 @@
'(?:^|\s+)local\s+\w+=' => q<local foo=bar>,
'(?:^|\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>,
);
my %string_bashisms = (
--
To unsubscribe, send mail to [EMAIL PROTECTED]
