[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[SCM] Debian package checker branch, master, updated. 2.0.0-rc2-2-gd89ad45



The following commit has been merged in the master branch:
commit d89ad45eaf84da97ba23a2477bdf0151d02f685d
Author: Adam D. Barratt <adam@adam-barratt.org.uk>
Date:   Wed Sep 24 19:40:27 2008 +0100

    Detect {,foo} and {foo,} as brace expansions in bashism checks.
    
    Detect brace expansions with either a leading or trailing blank
    component (e.g. "{foo,}"). Thanks to Jonny Lamb for pointing this out.

diff --git a/checks/scripts b/checks/scripts
index 8013212..5e0f8de 100644
--- a/checks/scripts
+++ b/checks/scripts
@@ -586,7 +586,7 @@ while (<SCRIPTS>) {
 		  qr'(test|-o|-a)\s*[^\s]+\s+==\s', # should be 'b = a'
 		  qr'\[\s+[^\]]+\s+==\s',        # should be 'b = a'
 		  qr'\s(\|\&)',		         # pipelining is not POSIX
-		  qr'[^\\\$]\{(?:[^\s\\\}]+?,)+[^\\\}\s]+\}', # brace expansion
+		  qr'[^\\\$]\{(?:[^\s\\\}]*?,)+[^\\\}\s]*\}', # brace expansion
 		  qr'(?:^|\s+)\w+\[\d+\]=',      # bash arrays, H[0]
 		  $LEADIN . qr'read\s+(?:-[a-qs-zA-Z\d-]+)',
 			# read with option other than -r
diff --git a/debian/changelog b/debian/changelog
index 28da2e7..35ec757 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -4,6 +4,11 @@ lintian (2.0.0) UNRELEASED; urgency=low
     + [RA] Allow tildes in the optional version number in the Source
       field.  Thanks, Rene Engelhard.  (Closes: #500063)
 
+  * checks/scripts:
+    + [ADB] Detect brace expansions with either a leading or trailing
+      blank component (e.g. "{foo,}"). Thanks to Jonny Lamb for pointing
+      this out.
+
  -- Russ Allbery <rra@debian.org>  Wed, 24 Sep 2008 11:13:27 -0700
 
 lintian (2.0.0~rc2) experimental; urgency=low

-- 
Debian package checker


Reply to: