debhelper modular buildsystems (try 3).

* New feature - when listing buildsystems, list their status too
  (auto/specified).
* Dh_Buildsystem_Basic.pm renamed to Dh_Buildsystem.pm
* Addressed a few issues expressed in the comments, answered a few
  comments.
* Cache DEB_BUILD_GNU_TYPE value. Performance hit is noticable when
  listing build systems.
* is_auto_buildable() renamed to check_auto_buildable() (again).
  Since there is is_buildable() now, I didn't want to use is_ for
  that method.

Signed-off-by: Modestas Vainius <modestas@vainius.eu>

1 files changed, 4 insertions, 2 deletions

diff --git a/Debian/Debhelper/Buildsystem/perl_build.pm b/Debian/Debhelper/Buildsystem/perl_build.pm
index 550f2a5b..63655304 100644
--- a/Debian/Debhelper/Buildsystem/perl_build.pm
+++ b/Debian/Debhelper/Buildsystem/perl_build.pm
@@ -8,13 +8,13 @@ package Debian::Debhelper::Buildsystem::perl_build;
 
 use strict;
 use Debian::Debhelper::Dh_Lib;
-use base 'Debian::Debhelper::Dh_Buildsystem_Basic';
+use base 'Debian::Debhelper::Dh_Buildsystem';
 
 sub DESCRIPTION {
 	"support for building Perl Build.PL based packages (in-source only)"
 }
 
-sub is_auto_buildable {
+sub check_auto_buildable {
 	my ($self, $action) = @_;
 
 	# Handles everything
@@ -25,6 +25,8 @@ sub is_auto_buildable {
 	# succeed, silently do nothing. Perhaps it would be better, then,
 	# to omit the test below. Then, it would try to run ./Build
 	# which doesn't exist, which should result in a semi-useful error.
+	# XXX MDX Agreed. But it would not be fully backwards compatible
+	#         (see comment in autotools.mk why). Your call.
 	if ($action ne "configure") {
 		$ret &&= -e "Build";
 	}