Re: Using $bindir in configure.ac

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

 



Hi,

On Fri, Jan 14, 2005 at 04:49:54PM +0100, Peter Simons wrote:
> I have applied the last patch you sent to the macro and
...
> dnl @author Stepan Kasal <kasal@xxxxxx>, ...

Thanks.  Perhaps we should give credit to Andreas, too.

There is still one problem with the macro--it changes prefix=NONE
to prefix="$ac_default_prefix", so subsequent macros don't have
the information whether prefix is defaulted or not.

The patch attached to this mail fixes this issue.  It also sets
exec_prefix to current value of prefix, thus we need one less level of
indirection.

Do you like the patch?

Regards,
	Stepan
Index: legacy/misc/ac_define_dir.m4
===================================================================
RCS file: /cvsroot/ac-archive/ac-archive/legacy/misc/ac_define_dir.m4,v
retrieving revision 1.6
diff -u -r1.6 ac_define_dir.m4
--- legacy/misc/ac_define_dir.m4	14 Jan 2005 15:28:54 -0000	1.6
+++ legacy/misc/ac_define_dir.m4	17 Jan 2005 16:02:57 -0000
@@ -15,13 +15,16 @@
 dnl    AC_DEFINE_DIR(PROG_PATH, bindir, [Location of installed binaries])
 dnl
 dnl @version $Id: ac_define_dir.m4,v 1.6 2005/01/14 15:28:54 simons Exp $
-dnl @author Stepan Kasal  <kasal@xxxxxx>, Guido Draheim <guidod@xxxxxx>, Alexandre Oliva
+dnl @author Stepan Kasal <kasal@xxxxxx>, Andreas Schwab <schwab@xxxxxxx>, Guido Draheim <guidod@xxxxxx>, Alexandre Oliva
 
 AC_DEFUN([AC_DEFINE_DIR], [
-  test "x$prefix" = xNONE && prefix="$ac_default_prefix"
-  test "x$exec_prefix" = xNONE && exec_prefix='${prefix}'
+  prefix_NONE=
+  exec_prefix_NONE=
+  test "x$prefix" = xNONE && prefix_NONE=yes && prefix=$ac_default_prefix
+  test "x$exec_prefix" = xNONE && exec_prefix_NONE=yes && exec_prefix=$prefix
   eval ac_define_dir="\"[$]$2\""
-  eval ac_define_dir="\"[$]ac_define_dir\""
   AC_SUBST($1, "$ac_define_dir")
   AC_DEFINE_UNQUOTED($1, "$ac_define_dir", [$3])
+  test "$prefix_NONE" && prefix=NONE
+  test "$exec_prefix_NONE" && exec_prefix=NONE
 ])
_______________________________________________
Autoconf mailing list
Autoconf@xxxxxxx
http://lists.gnu.org/mailman/listinfo/autoconf

[Index of Archives]     [GCC Help]     [Kernel Discussion]     [RPM Discussion]     [Red Hat Development]     [Yosemite News]     [Linux USB]     [Samba]

  Powered by Linux