[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#966396: debhelper: should invoke perl as /usr/bin/perl



Package: debhelper
Version: 13.2
X-Debbugs-Cc: debian-perl@lists.debian.org

On Tue, Jul 21, 2020 at 03:38:05PM +0200, Stefan Hornburg (Racke) wrote:
> On 7/21/20 3:32 PM, Dominic Hargreaves wrote:
> > On Tue, Jul 21, 2020 at 11:54:50AM +0200, Stefan Hornburg (Racke) wrote:
> >> I'm trying to update a really old Debian package (safe-hole-perl) because of bug
> >> https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=965811.
> >>
> >> It uses Module::Build and is a XS module.
> >>
> >> Error message on build:
> >>
> >> dh binary
> >>    dh_testroot
> >>    dh_prep
> >>    dh_auto_install
> >> 	perl Build install --destdir /var/tmp/debaux-racke/safe-hole-perl/safe-hole-perl-0.14/debian/libsafe-hole-perl
> >> --create_packlist 0
> >> Building Safe-Hole
> >> WARNING: Can't figure out install path for types: arch lib
> >> Files will not be installed.
> >> Installing
> >> /var/tmp/debaux-racke/safe-hole-perl/safe-hole-perl-0.14/debian/libsafe-hole-perl/home/racke/perl5/perlbrew/perls/perl-5.30.0/man/man3/Safe::Hole.3
> > 
> > At a guess, the problem is related to the fact your perl is a perlbrew
> > one (see the install path above).
> > 
> > Does it work if you arrange for the perl in your path to be /usr/bin/perl?
> > 
> > This smells like a bug in debhelper to me - perl policy says that perl
> > should be /usr/bin/perl in shebangs, and I think the same should apply
> > here. I'm amazed this hasn't bit us before, if I'm right (it's been like
> > this since at least 2009, and probably forever).
> > 
> > https://salsa.debian.org/debian/debhelper/-/blob/master/lib/Debian/Debhelper/Buildsystem/perl_build.pm#L44
> > 
> > Dominic
> > 
> > 
> 
> Yes, that was the problem. After "perlbrew off" it works as expected. It is an old problem indeed.
> 
> Thanks a lot (to Dominique as well) for the quick answer.

Filing this as a bug, with a proposed patch at
https://salsa.debian.org/debian/debhelper/-/merge_requests/40


Reply to: