On Wed, Oct 03, 2012 at 05:32:40PM -0700, Ethan Jackson wrote:
> In the ESX build system, autom4te is in a funny location specified
> by an environment variable.  Before this patch, this environment
> variable was overridden by the tests automake file.
> 
> Signed-off-by: Ethan Jackson <et...@nicira.com>

We can't (shouldn't, anyhow) use ?= because it's a GNU Make
extension.

Here's the canonical way to solve this:

diff --git a/configure.ac b/configure.ac
index 9bdffea..6be660e 100644
--- a/configure.ac
+++ b/configure.ac
@@ -34,6 +34,11 @@ if test "$PERL" = no; then
    AC_MSG_ERROR([Perl interpreter not found in $PATH or $PERL.])
 fi
 
+AC_PATH_PROG([AUTOM4TE], [autom4te], [autom4te], no)
+if test "$AUTOM4TE" = no; then
+   AC_MSG_ERROR([auto4mte not found in $PATH or $AUTOM4TE.])
+fi
+
 AC_USE_SYSTEM_EXTENSIONS
 AC_C_BIGENDIAN
 AC_SYS_LARGEFILE
diff --git a/tests/automake.mk b/tests/automake.mk
index 20f9e82..2977f76 100644
--- a/tests/automake.mk
+++ b/tests/automake.mk
@@ -146,7 +146,6 @@ check-valgrind: all tests/atconfig tests/atlocal 
$(TESTSUITE) \
 clean-local:
        test ! -f '$(TESTSUITE)' || $(SHELL) '$(TESTSUITE)' -C tests --clean
 
-AUTOM4TE = autom4te
 AUTOTEST = $(AUTOM4TE) --language=autotest
 $(TESTSUITE): package.m4 $(TESTSUITE_AT)
        $(AUTOTEST) -I '$(srcdir)' -o $@.tmp $@.at
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to