Pass CPPFLAGS and LDFLAGS to Makefile.PL and Build.PL

[debhelper.git] / Debian / Debhelper / Buildsystem / perl_build.pm

diff --git a/Debian/Debhelper/Buildsystem/perl_build.pm b/Debian/Debhelper/Buildsystem/perl_build.pm
index a43d65dacd7a1cf8fa6d5cbf617c68c4cb92446d..35a23d675b12bd620855206d38ee9eb768ad727a 100644 (file)
--- a/Debian/Debhelper/Buildsystem/perl_build.pm
+++ b/Debian/Debhelper/Buildsystem/perl_build.pm
@@ -1,4 +1,4 @@
-# A buildsystem plugin for handling Perl Build based projects.
+# A build system class for handling Perl Build based projects.
 #
 # Copyright: © 2008-2009 Joey Hess
 #            © 2008-2009 Modestas Vainius
@@ -7,63 +7,71 @@
 package Debian::Debhelper::Buildsystem::perl_build;
 
 use strict;
-use Debian::Debhelper::Dh_Lib;
-use Debian::Debhelper::Dh_Buildsystem_Bases;
-use base 'Debian::Debhelper::Dh_Buildsystem_Basic';
+use Debian::Debhelper::Dh_Lib qw(compat);
+use base 'Debian::Debhelper::Buildsystem';
+use Config;
 
 sub DESCRIPTION {
-       "support for building Perl Build.PL based packages (in-source only)"
+       "Perl Module::Build (Build.PL)"
 }
 
-sub is_buildable {
-       my ($self, $action) = @_;
-       my $ret = (-e "Build.PL");
-       if ($action ne "configure") {
-               $ret &= (-e "Build");
+sub check_auto_buildable {
+       my ($this, $step) = @_;
+
+       # Handles everything
+       my $ret = -e $this->get_sourcepath("Build.PL");
+       if ($step ne "configure") {
+               $ret &&= -e $this->get_sourcepath("Build");
        }
-       return $ret;
+       return $ret ? 1 : 0;
 }
 
-sub invoke_impl {
-       my $self=shift;
-       $ENV{MODULEBUILDRC} = "/dev/null";
-       return $self->SUPER::invoke_impl(@_);
+sub do_perl {
+       my $this=shift;
+       $this->doit_in_sourcedir("perl", @_);
 }
 
 sub new {
-       my $cls=shift;
-       my $self= $cls->SUPER::new(@_);
-       $self->enforce_in_source_building();
-       return $self;
+       my $class=shift;
+       my $this= $class->SUPER::new(@_);
+       $this->enforce_in_source_building();
+       return $this;
 }
 
-sub configure_impl {
-       my $self=shift;
-       # XXX JEH I think the below comment is inherited from elsewhere;
-       # doesn't really make sense now.
-       $ENV{PERL_MM_USE_DEFAULT}=1; # Module::Build can also use this.
-       doit("perl", "Build.PL", "installdirs=vendor", @_);
+sub configure {
+       my $this=shift;
+       my @flags;
+       $ENV{PERL_MM_USE_DEFAULT}=1;
+       if ($ENV{CFLAGS} && ! compat(8)) {
+               push @flags, "config=optimize=$ENV{CFLAGS} $ENV{CPPFLAGS}";
+       }
+       if ($ENV{LDFLAGS} && ! compat(8)) {
+               push @flags, "config=ld=$Config{ld} $ENV{LDFLAGS}";
+       }
+       $this->do_perl("Build.PL", "installdirs=vendor", @flags, @_);
 }
 
-sub build_impl {
-       my $self=shift;
-       doit("perl", "Build", @_);
+sub build {
+       my $this=shift;
+       $this->do_perl("Build", @_);
 }
 
-sub test_impl {
-       my $self=shift;
-       doit(qw/perl Build test/, @_);
+sub test {
+       my $this=shift;
+       $this->do_perl("Build", "test", @_);
 }
 
-sub install_impl {
-       my $self=shift;
+sub install {
+       my $this=shift;
        my $destdir=shift;
-       doit("perl", "Build", "install", "destdir=$destdir", "create_packlist=0", @_);
+       $this->do_perl("Build", "install", "destdir=$destdir", "create_packlist=0", @_);
 }
 
-sub clean_impl {
-       my $self=shift;
-       doit("perl", "Build", "--allow_mb_mismatch", 1, "distclean", @_);
+sub clean {
+       my $this=shift;
+       if (-e $this->get_sourcepath("Build")) {
+               $this->do_perl("Build", "--allow_mb_mismatch", 1, "distclean", @_);
+       }
 }
 
-1;
+1