The following commit has been merged in the master branch:
commit cf06cd707fbe1ebe0a8bfacaaea088547a135b13
Author: Mathieu Parent <[email protected]>
Date:   Thu May 16 15:27:32 2013 +0200

    phppear: Check for channel.xml
    
    Signed-off-by: Niels Thykier <[email protected]>

diff --git a/checks/phppear.desc b/checks/phppear.desc
index db27cac..f99552f 100644
--- a/checks/phppear.desc
+++ b/checks/phppear.desc
@@ -15,3 +15,12 @@ Info: The package contains an package.xml or package2.xml 
file but doesn't
  pkg-php-tools is the recommended tool for building PEAR and PECL packages. For
  more information, install it and read the included README.Debian.
 
+Tag: pear-channel-without-pkg-php-tools-builddep
+Severity: normal
+Certainty: possible
+Info: The package contains an channel.xml file but doesn't build-depend on
+ pkg-php-tools.
+ .
+ pkg-php-tools is the recommended tool for building PEAR and PECL packages. For
+ more information, install it and read the included README.Debian.
+
diff --git a/checks/phppear.pm b/checks/phppear.pm
index bbef200..205f18e 100644
--- a/checks/phppear.pm
+++ b/checks/phppear.pm
@@ -32,13 +32,20 @@ sub run {
     # PEAR or PECL package
     my $package_xml = $info->index('package.xml');
     my $package2_xml = $info->index('package2.xml');
+    my $bdepends = $info->relation('build-depends');
     if (defined($package_xml) || defined($package2_xml)) {
         # Checking source builddep
-        my $bdepends = $info->relation('build-depends');
         if (!$bdepends->implies('pkg-php-tools')) {
             tag 'pear-package-without-pkg-php-tools-builddep';
         }
     }
+    # PEAR channel
+    my $channel_xml = $info->index('channel.xml');
+    if (defined($channel_xml)) {
+        if (!$bdepends->implies('pkg-php-tools')) {
+            tag 'pear-channel-without-pkg-php-tools-builddep';
+        }
+    }
     return;
 }
 

-- 
Debian package checker


-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
Archive: http://lists.debian.org/[email protected]

Reply via email to