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

[SCM] Debian package checker branch, master, updated. 2.4.2-35-g25219ae



The following commit has been merged in the master branch:
commit 25219ae58763730acd1e3b5bf7f1bb4fb677319c
Author: Russ Allbery <rra@debian.org>
Date:   Sun Jul 25 19:03:55 2010 -0700

    Improve parsing of package names containing substvars
    
    * lib/Lintian/Relation.pm:
      + [RA] Don't require a package name component after a substvar to
        follow the rules of the beginning of a package name.  Fixes
        stronger-dependency-implies-weaker false positives containing
        substvars.  (Closes: #580494)

diff --git a/debian/changelog b/debian/changelog
index fe941bc..7f39858 100755
--- a/debian/changelog
+++ b/debian/changelog
@@ -57,6 +57,11 @@ lintian (2.4.3) UNRELEASED; urgency=low
   * lib/Lintian/Check.pm:
     + [RA] More correctly handle hyphenated words to avoid triggering
       spelling error false positives.  (Closes: #588896)
+  * lib/Lintian/Relation.pm:
+    + [RA] Don't require a package name component after a substvar to
+      follow the rules of the beginning of a package name.  Fixes
+      stronger-dependency-implies-weaker false positives containing
+      substvars.  (Closes: #580494)
   * lib/Lintian/Schedule.pm:
     + [ADB] Import "warning" from Lintian::Output, really fixing the
       error caused by a non-existent file referenced in a .changes
diff --git a/lib/Lintian/Relation.pm b/lib/Lintian/Relation.pm
index e45f4f7..a96b959 100644
--- a/lib/Lintian/Relation.pm
+++ b/lib/Lintian/Relation.pm
@@ -75,11 +75,16 @@ sub parse_element {
     $element =~ /
         ^\s*                            # skip leading whitespace
         (                               # package name or substvar (1)
+         (?:                            #  start of the name
+          [a-zA-Z0-9]                   #   start of a package name
+          |                             #   or
+          \$\{[a-zA-Z0-9:-]+\}          #   substvar
+         )                              #  end of start of the name
          (?:                            #  substvars may be mixed in
-          [a-zA-Z0-9][a-zA-Z0-9+.-]+    #   package name
+          [a-zA-Z0-9+.-]+               #   package name portion
           |                             #   or
           \$\{[a-zA-Z0-9:-]+\}          #   substvar
-         )+                             #  one or more name or substvar
+         )+                             #  one or more portion or substvar
         )                               # end of package name or substvar
         (?:                             # start of optional version
          \s* \(                         # open parenthesis for version part

-- 
Debian package checker


Reply to: