The following commit has been merged in the master branch: commit c4b87525b17351ee215a0df5f731d7d1f5a1e388 Author: Adam D. Barratt <[EMAIL PROTECTED]> Date: Fri Jul 4 21:06:35 2008 +0100
Add "then" to the list of expressions which can precede a bashism diff --git a/checks/scripts b/checks/scripts index 0fe348b..1861733 100644 --- a/checks/scripts +++ b/checks/scripts @@ -543,7 +543,7 @@ while (<SCRIPTS>) { if ($cat_string eq "" and $checkbashisms and !$within_another_shell) { my $found = 0; my $match = ''; - my $LEADIN = qr'(?:(^|[`&;(|{])\s*|(if|do|while)\s+)'; + my $LEADIN = qr'(?:(^|[`&;(|{])\s*|(if|then|do|while)\s+)'; my @bashism_single_quote_regexs = ( $LEADIN . qr'echo\s+(?:-[^e]+\s+)?([\'])[^\']*(\\[\\abcEfnrtv0])+.*?[\']', # unsafe echo with backslashes -- Debian package checker -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]