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

Bug#337479: yaird: should use /usr/bin/perl, not /usr/local/bin/perl



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 :)

-- 
mattia
:wq!
Index: configure.in
===================================================================
--- configure.in	(revision 4760)
+++ configure.in	(working copy)
@@ -47,8 +47,29 @@
 		[Define to nuke initramfs before moving to real root])
 fi
 
+# Force a specific perl binary
+AC_ARG_WITH(perl,
+	AC_HELP_STRING([--with-perl=PATH],
+		[Force a specific perl executable path (autodetected by default)]),
+	[
+		case "${withval}" in
+		yes|auto)
+			with_perl=auto ;;
+		*)	PERL=${withval}
+			if ! test -x ${PERL} ; then
+				AC_MSG_ERROR(bad value ${PERL} for --with-perl) 
+			fi
+			AC_SUBST(PERL)
+			;;
+		esac
+	],[
+		with_perl=auto
+		])
+if test "$with_perl" = auto; then
+	AC_PATH_PROG(PERL,perl)
+fi
+
 # Checks for programs.
-AC_PATH_PROG(PERL,perl)
 AC_PROG_CC
 AC_PROG_RANLIB
 

Reply to: