python distutils buildsystem: When checking if a version of python is installed,...

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

diff --git a/Debian/Debhelper/Buildsystem/python_distutils.pm b/Debian/Debhelper/Buildsystem/python_distutils.pm
index 9468e38fe4a2010464ff74b46b23c966db517a2a..81cff3c9fc8fe15689605f800964f585a0725dbd 100644 (file)
--- a/Debian/Debhelper/Buildsystem/python_distutils.pm
+++ b/Debian/Debhelper/Buildsystem/python_distutils.pm
@@ -24,14 +24,14 @@ sub DEFAULT_BUILD_DIRECTORY {
 sub new {
        my $class=shift;
        my $this=$class->SUPER::new(@_);
-       # Out of source tree building is prefered.
+       # Out of source tree building is preferred.
        $this->prefer_out_of_source_building(@_);
        return $this;
 }
 
 sub check_auto_buildable {
        my $this=shift;
-       return -e $this->get_sourcepath("setup.py");
+       return -e $this->get_sourcepath("setup.py") ? 1 : 0;
 }
 
 sub not_our_cfg {
@@ -79,37 +79,117 @@ sub pre_building_step {
        $this->SUPER::pre_building_step($step);
 }
 
+sub dbg_build_needed {
+       my $this=shift;
+       my $act=shift;
+
+       # Return a list of python-dbg package which are listed
+       # in the build-dependencies. This is kinda ugly, but building
+       # dbg extensions without checking if they're supposed to be
+       # built may result in various FTBFS if the package is not
+       # built in a clean chroot.
+
+       my @dbg;
+       open (CONTROL, 'debian/control') ||
+               error("cannot read debian/control: $!\n");
+       foreach my $builddeps (join('', <CONTROL>) =~ 
+                       /^Build-Depends[^:]*:.*\n(?:^[^\w\n].*\n)*/gmi) {
+               while ($builddeps =~ /(python[^, ]*-dbg)/g) {
+                       push @dbg, $1;
+               }
+       }
+
+       close CONTROL;
+       return @dbg;
+
+}
+
 sub setup_py {
        my $this=shift;
        my $act=shift;
+
+       # We need to to run setup.py with the default python last
+       # as distutils/setuptools modifies the shebang lines of scripts.
+       # This ensures that #!/usr/bin/python is installed last and
+       # not pythonX.Y
+       # Take into account that the default Python must not be in
+       # the requested Python versions.
+       # Then, run setup.py with each available python, to build
+       # extensions for each.
+
        my $python_default = `pyversions -d`;
+       if ($? == -1) {
+               error("failed to run pyversions")
+       }
+       my $ecode = $? >> 8;
+       if ($ecode != 0) {
+               error("pyversions -d failed [$ecode]")
+       }
        $python_default =~ s/^\s+//;
        $python_default =~ s/\s+$//;
+       my @python_requested = split ' ', `pyversions -r`;
+       if ($? == -1) {
+               error("failed to run pyversions")
+       }
+       $ecode = $? >> 8;
+       if ($ecode != 0) {
+               error("pyversions -r failed [$ecode]")
+       }
+       if (grep /^\Q$python_default\E/, @python_requested) {
+               @python_requested = (
+                       grep(!/^\Q$python_default\E/, @python_requested),
+                       "python",
+               );
+       }
 
-       # We need to to run setup.py with the default python first
-       # as distutils/setuptools modifies the shebang lines of scripts.
-       # This ensures that #!/usr/bin/python is used and not pythonX.Y
-       $this->doit_in_sourcedir("python", "setup.py", $act, @_);
-       for my $python (grep(!/^$python_default/, (split ' ', `pyversions -r 2>/dev/null`))) {
-               if (-x "/usr/bin/" . $python) {
-                       $this->doit_in_sourcedir($python, "setup.py", $act, @_);
+       my @python_dbg;
+       my @dbg_build_needed = $this->dbg_build_needed();
+       foreach my $python (map { $_."-dbg" } @python_requested) {
+               if (grep /^(python-all-dbg|\Q$python\E)/, @dbg_build_needed) {
+                       push @python_dbg, $python;
+               }
+               elsif (($python eq "python-dbg")
+                      and (grep /^\Q$python_default\E/, @dbg_build_needed)) {
+                       push @python_dbg, $python_default."-dbg";
+               }
+       }
+
+       foreach my $python (@python_dbg, @python_requested) {
+               if (-f "/usr/lib/$python/distutils/__init__.py") {
+                       # To allow backports of debhelper we don't pass
+                       # --install-layout=deb to 'setup.py install` for
+                       # those Python versions where the option is
+                       # ignored by distutils/setuptools.
+                       if ( $act eq "install" and not
+                            ( ($python =~ /^python(?:-dbg)?$/
+                                and $python_default =~ /^python2\.[2345]$/)
+                             or $python =~ /^python2\.[2345](?:-dbg)?$/ )) {
+                               $this->doit_in_sourcedir($python, "setup.py",
+                                               $act, @_, "--install-layout=deb");
+                       }
+                       else {
+                               $this->doit_in_sourcedir($python, "setup.py",
+                                               $act, @_);
+                       }
                }
        }
 }
 
 sub build {
        my $this=shift;
-       $this->setup_py("build", @_);
+       $this->setup_py("build",
+               "--force",
+               @_);
 }
 
 sub install {
        my $this=shift;
        my $destdir=shift;
        $this->setup_py("install",
+               "--force",
                "--root=$destdir",
                "--no-compile",
                "-O0",
-               "--install-layout=deb",
                @_);
 }