Author: adam-guest
Date: 2008-06-24 17:37:30 +0000 (Tue, 24 Jun 2008)
New Revision: 1522
Modified:
trunk/debian/changelog
trunk/scripts/checkbashisms.pl
Log:
Add test -a / -o to --posix
Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog 2008-06-23 18:15:16 UTC (rev 1521)
+++ trunk/debian/changelog 2008-06-24 17:37:30 UTC (rev 1522)
@@ -10,8 +10,8 @@
+ Quote test expressions using qr to allow perl to optionally compile
or otherwise optimise them. Thanks to Frank Lichtenheld for the idea.
+ Add a --posix option which flags constructs not supported by POSIX but
- permitted by Debian Policy (test / [ using -a or -o support is currently
- missing pending a reliable check being written)
+ permitted by Debian Policy (echo -n, local (with no options or
+ assignments) and test / [ using -a or -o )
* Debbugs.pm: Move initialisation of the SOAP object in to a common routine
and set the underlying transport object to honour proxies from the
environment (Closes: #487592)
Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl 2008-06-23 18:15:16 UTC (rev 1521)
+++ trunk/scripts/checkbashisms.pl 2008-06-24 17:37:30 UTC (rev 1522)
@@ -499,8 +499,7 @@
}
if ($opt_posix) {
$bashisms{$LEADIN . qr'local\s+\w+\s+'} = q<local foo>;
- # Should include test / [ -a and -o here but we need a test
- # which isn't FP and/or FN prone
+ $bashisms{$LEADIN . qr'((?:test|\[)\s+.+\s-[ao])\s'} = q<test -a/-o>;
}
if ($makefile) {
--
To unsubscribe, send mail to [EMAIL PROTECTED]
