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

[SCM] Debian package checker branch, master, updated. 2.5.14-57-gbfd87c2



The following commit has been merged in the master branch:
commit cf06cd707fbe1ebe0a8bfacaaea088547a135b13
Author: Mathieu Parent <math.parent@gmail.com>
Date:   Thu May 16 15:27:32 2013 +0200

    phppear: Check for channel.xml
    
    Signed-off-by: Niels Thykier <niels@thykier.net>

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


Reply to: