Bug#337479: yaird: should use /usr/bin/perl, not /usr/local/bin/perl
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On Wed, 9 Nov 2005 23:28:40 +0100
Erik van Konijnenburg <ekonijn@xs4all.nl> wrote:
> On Wed, Nov 09, 2005 at 10:20:47PM +0100, Mattia Dongili wrote:
> > On Wed, Nov 09, 2005 at 08:16:08PM +0100, Laurent Bonnaud wrote:
> > > On sam, 2005-11-05 at 16:31 +0100, Mattia Dongili wrote:
> > [...]
> > > > I's like
> > > > configuring a package with silly ./configure options and
> > > > pretending it works everywhere. :)
> > >
> > > I did not intervene in the build process. I used this command:
> > >
> > > apt-get -b source yaird
> > >
> > > With this command I expect to get a working package as long as I
> > > do not mess up the system. And putting files in /usr/local/ is
> > > clearly allowed and does not count as "messing up the system".
> >
> > Yes, sorry. I was thinking of a lot of silly corner cases when I
> > wrote that (I hit the same issue when playing with ./configure and
> > stuff so that's what I had in mind in the first place).
> >
> > By the way, Erik, Jonas I'm attaching a patch to the configure.in
> > to add --with-perl option (defaults to autodetection if not given).
> > I'm sorry I'm not into cdbs enough to provide a patch for
> > debian/rules also :)
>
> Thanks. Note that the patch only helps if cdbs or debian/rules is
> also adapted, and then only for yaird, only for perl; not for other
> stuff in /usr/local, not for other packages being built.
>
> For this reason, I'll hold off incorporating the patch until it
> becomes clear this cannot be fixed in cdbs or dpkg-dev.
>
> Outline of possible fix:
>
> * do 'configure PATH=/bin:/usr/bin' rather than plain configure.
> Tested from the command line, helps hiding /usr/local/bin/perl
> effectively.
> * in cdbs, class/autotools-vars.mk, do that path setting in
> DEB_CONFIGURE_SCRIPT_ENV. This bit is untested.
After discussion on #debian-devel I found that
1) It is sane of upstream autotools to rely on $PATH of build
environment
2) Packaging scripts (dpkg-*) and packaging helper tools (debhelper and
cdbs) should preserve $PATH from build environment
3) Official Debian packages must use /usr/bin/perl
So all in all, the proper fix, I believe, is to pass PERL=/usr/bin/perl
to configure in debian/rules.
I'll do that in next upload.
- Jonas
- --
* Jonas Smedegaard - idealist og Internet-arkitekt
* Tlf.: +45 40843136 Website: http://dr.jones.dk/
- Enden er nær: http://www.shibumi.org/eoti.htm
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (GNU/Linux)
iD8DBQFDc4Oyn7DbMsAkQLgRAuT5AJ0RYCKcNi7S9zLFOba7JADh6M9C6QCdGU5j
xRQTIl7Mu+S23QiJsoGEF4U=
=/GKs
-----END PGP SIGNATURE-----
Reply to: