Bug#337479: yaird: should use /usr/bin/perl, not /usr/local/bin/perl
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.
Regards,
Erik
Reply to: