On Thu, May 20, 2010 at 09:19:07PM -0500, Raphael Geissert wrote:
> Attached mbox includes patches for the corresponding bug reports among other 
> changes (bug fixes, new checks.)

Thanks for the patches.  I've applied all of them, with one slight
change to the one below.

diff --git a/scripts/checkbashisms.pl b/scripts/checkbashisms.pl
index 02698af..1eeda6a 100755
--- a/scripts/checkbashisms.pl
+++ b/scripts/checkbashisms.pl
@@ -198,7 +198,7 @@ foreach my $filename (@ARGV) {
                $_ = $1 if $1;
            } 
 
-           last if m%^(export )?SHELL\s*:?=\s*(/bin/)?bash\s*%;
+           last if m%^\s*(override|export)?\s*SHELL\s*:?=\s*(/bin/)?bash\s*%;

Require a space after override/export -- (override |export )

-- 
James
GPG Key: 1024D/61326D40 2003-09-02 James Vega <james...@debian.org>

Attachment: signature.asc
Description: Digital signature

Reply via email to