Author: adam-guest
Date: 2008-07-19 16:53:18 +0000 (Sat, 19 Jul 2008)
New Revision: 1576
Modified:
trunk/debian/changelog
trunk/scripts/checkbashisms.pl
Log:
checkbashisms: Only flag "local x y" and "local foo=bar" when --posix is
used, as the use of the constructs is likely to become policy compliant
in the near future (see Policy bug #473019)
Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog 2008-07-18 06:01:54 UTC (rev 1575)
+++ trunk/debian/changelog 2008-07-19 16:53:18 UTC (rev 1576)
@@ -1,5 +1,8 @@
devscripts (2.10.35) UNRELEASED; urgency=low
+ * checkbashisms: Only flag "local x y" and "local foo=bar" when --posix is
+ used, as the use of the constructs is likely to become policy compliant
+ in the near future (see Policy bug #473019)
* debcommit: When --diff is used, don't open an editor to confirm commit
messages
* po4a/po/fr.po: Update French manpage translations; thanks Nicolas FRANCOIS
Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl 2008-07-18 06:01:54 UTC (rev 1575)
+++ trunk/scripts/checkbashisms.pl 2008-07-19 16:53:18 UTC (rev 1576)
@@ -480,8 +480,6 @@
$LEADIN . qr'alias\s+-p' => q<alias -p>,
$LEADIN . qr'unalias\s+-a' => q<unalias -a>,
$LEADIN . qr'local\s+-[a-zA-Z]+' => q<local -opt>,
- $LEADIN . qr'local\s+\w+=' => q<local foo=bar>,
- $LEADIN . qr'local\s+\w+\s+\w+' => q<local x y>,
qr'(?:^|\s+)\s*\(?\w*[^\(\w\s]+\S*?\s*\(\)\s*([\{|\(]|\Z)'
=> q<function names should only contain [a-z0-9_]>,
$LEADIN . qr'(push|pop)d(\s|\Z)' => q<(push|pod)d>,
@@ -531,6 +529,8 @@
}
if ($opt_posix) {
$bashisms{$LEADIN . qr'local\s+\w+(\s+\W|\s*[;&|)]|$)'} = q<local foo>;
+ $bashisms{$LEADIN . qr'local\s+\w+='} = q<local foo=bar>;
+ $bashisms{$LEADIN . qr'local\s+\w+\s+\w+'} = q<local x y>;
$bashisms{$LEADIN . qr'((?:test|\[)\s+.+\s-[ao])\s'} = q<test -a/-o>;
}
--
To unsubscribe, send mail to [EMAIL PROTECTED]
