[SCM] Debian package checker branch, master, updated. 3a0a57ee7b57dd45f9a0409fa0cd97828a7c1d38
The following commit has been merged in the master branch:
commit 3a0a57ee7b57dd45f9a0409fa0cd97828a7c1d38
Author: Adam D. Barratt <adam@adam-barratt.org.uk>
Date: Sun Jul 6 21:41:17 2008 +0100
Fix potential false-positives in the bashism checks for pushd, popd, ulimit, shopt and dirs
diff --git a/checks/scripts b/checks/scripts
index 1861733..fe7ca5f 100644
--- a/checks/scripts
+++ b/checks/scripts
@@ -611,13 +611,13 @@ while (<SCRIPTS>) {
$LEADIN . qr'local\s+\w+=', # local foo=bar
qr'(?:^|\s+)\s*\(?\w*[^\(\w\s]+\S*?\s*\(\)[^\"]?',
# function names should only contain [a-z0-9_]
- $LEADIN . qr'(push|pop)d\b', # (push|pod)d
+ $LEADIN . qr'(push|pop)d(\s|\Z)', # (push|pod)d
$LEADIN . qr'export\s+-[^p]', # export only takes -p as an option
- $LEADIN . qr'ulimit\b',
- $LEADIN . qr'shopt\b',
+ $LEADIN . qr'ulimit(\s|\Z)',
+ $LEADIN . qr'shopt(\s|\Z)',
$LEADIN . qr'type\s',
$LEADIN . qr'time\s',
- $LEADIN . qr'dirs\b',
+ $LEADIN . qr'dirs(\s|\Z)',
qr'(?:^|\s+)[<>]\(.*?\)', # <() process substituion
qr'(?:^|\s+)readonly\s+-[af]', # readonly -[af]
$LEADIN . qr'(sh|\$\{?SHELL\}?) -[rD]', # sh -[rD]
--
Debian package checker
Reply to: