diff options
Diffstat (limited to 'Debian/Debhelper/Buildsystem/makefile.pm')
-rw-r--r-- | Debian/Debhelper/Buildsystem/makefile.pm | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/Debian/Debhelper/Buildsystem/makefile.pm b/Debian/Debhelper/Buildsystem/makefile.pm index 7ffb048e..36081ecc 100644 --- a/Debian/Debhelper/Buildsystem/makefile.pm +++ b/Debian/Debhelper/Buildsystem/makefile.pm @@ -10,11 +10,6 @@ use strict; use Debian::Debhelper::Dh_Lib; use base 'Debian::Debhelper::Dh_Buildsystem_Basic'; -# XXX JEH setting this env var is dodgy, -# probably better to test if it exists with a default value. -# (Factor out to helper function?) -# XXX MDX Done. See new(). - sub get_makecmd_C { my $self=shift; if ($self->get_builddir()) { @@ -25,6 +20,12 @@ sub get_makecmd_C { # XXX JEH I *like* this. Yay for factoring out ugly ugly stuff! # XXX MDX TODO: this could use dh debian/rules parser. +# XXX JEH That one checks for explicit only targets, while we want +# implicit targets here too. I think the current code is ok; +# it's a bonus that it checks if the target it empty. +# Hmm, one problem is that if a target exists but will run no +# commands since it's already built, the approach below will return +# nothing and assume it doesn't exist. sub exists_make_target { my ($self, $target) = @_; my $makecmd=$self->get_makecmd_C(); |