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

[SCM] Debian package checker branch, master, updated. 2.3.2-10-g2b5273b



The following commit has been merged in the master branch:
commit 2b5273b2dfcbf919e5da5405931e8b74d24f7c15
Author: Raphael Geissert <atomo64@gmail.com>
Date:   Thu Jan 28 20:34:32 2010 -0600

    Don't warn about depending on the phpapi-* virtual package.
    
    On the rare case where a package may build-depend on a package shipping
    a PHP extension they will very likely already be build-depending on the
    cli SAPI.
    
    * checks/fields{,.desc}:
      + [RG] Don't warn about depending on the phpapi-* virtual package.

diff --git a/checks/fields b/checks/fields
index 29e9f55..6f93052 100644
--- a/checks/fields
+++ b/checks/fields
@@ -538,7 +538,10 @@ if (($type eq "binary") || ($type eq 'udeb')) {
 			tag "virtual-package-depends-without-real-package-depends", "$field: $alternatives[0]->[0]"
 			    if ($VIRTUAL_PACKAGES->known($alternatives[0]->[0])
 				&& ($field eq "depends" || $field eq "pre-depends")
-				&& ($pkg ne 'base-files' || $alternatives[0]->[0] ne 'awk'));
+				&& ($pkg ne 'base-files' || $alternatives[0]->[0] ne 'awk')
+				# ignore phpapi- dependencies as adding an
+				# alternative, real, package breaks its pourpose
+				&& $alternatives[0]->[0] !~ m/^phpapi-/);
 
 			# Check defaults for transitions.  Here, we only care that the first alternative is current.
 			tag "depends-on-old-emacs", "$field: $alternatives[0]->[0]"
diff --git a/debian/changelog b/debian/changelog
index 9ecca0b..fa9e0a5 100755
--- a/debian/changelog
+++ b/debian/changelog
@@ -15,6 +15,8 @@ lintian (2.3.3) UNRELEASED; urgency=low
   * checks/binaries{,.desc}:
     + [RG] Warn about packages shipping PHP extensions but not
       depending on phpapi-*.
+  * checks/fields{,.desc}:
+    + [RG] Don't warn about depending on the phpapi-* virtual package.
   * checks/files{,.desc}:
     + [RG] Warn about packages shipping PHP .ini files using old-style
       (#) comments separators.

-- 
Debian package checker


Reply to: