Bug#706923: [debhelper] dh_auto_install: Not supplying params when testing for make install target

[Sven Eckelmann]

Now as promised the results on kfreebsd-amd64

On Monday 06 May 2013 16:47:34 Joey Hess wrote:
> Sven Eckelmann wrote:
> > Not a working one for kfreebsd. That's why the system detection is forced
> > to Linux-userspace with this parameter.
> 
> Does make -n print out the target if it's passed the parameter?

Yes

> diff --git a/Debian/Debhelper/Buildsystem/makefile.pm
> b/Debian/Debhelper/Buildsystem/makefile.pm index c63b58e..8d0def1 100644
> --- a/Debian/Debhelper/Buildsystem/makefile.pm
> +++ b/Debian/Debhelper/Buildsystem/makefile.pm
> @@ -20,7 +20,7 @@ sub exists_make_target {
>       unshift @opts, "-C", $buildpath if $buildpath ne ".";
>       open(SAVEDERR, ">&STDERR");
>       open(STDERR, ">/dev/null");
> -     open(MAKE, "-|", $this->{makecmd}, @opts, $target);
> +     open(MAKE, "-|", $this->{makecmd}, @opts, $target, @_);
>       my $output=<MAKE>;
>       chomp $output;
>       close MAKE;
> @@ -46,7 +46,7 @@ sub make_first_existing_target {
>       my $targets=shift;
> 
>       foreach my $target (@$targets) {
> -             if ($this->exists_make_target($target)) {
> +             if ($this->exists_make_target($target, @_)) {
>                       $this->do_make($target, @_);
>                       return $target;
>               }

This patch works for for my purposes.

Thanks,
Sven

signature.asc
Description: This is a digitally signed message part.