FYI: EMACS and EMACSLOADPATH as precious (Was: Re: Bug in elisp-comp)

[Alexandre Duret-Lutz]

>>> "Ryan" == Ryan T Sammartino <address@hidden> writes:

[...]

 Ryan> It would also be nice if Automake automatically added
 Ryan> EMACS and EMACSLOADPATH as a list of environment variables
 Ryan> that influence builds ("previous" variables in Autoconf
 Ryan> parlance) whenever lisp sources are being used.

Thanks for the suggestion.  I've installed the following on HEAD.

2003-01-08  Alexandre Duret-Lutz  <address@hidden>

        * m4/lispdir.m4: Register EMACS and EMACSLOADPATH as precious
        variables.  Suggested by Ryan T. Sammartino.

Index: m4/lispdir.m4
===================================================================
RCS file: /cvs/automake/automake/m4/lispdir.m4,v
retrieving revision 1.18
diff -u -r1.18 lispdir.m4
--- m4/lispdir.m4       19 Jul 2002 20:17:58 -0000      1.18
+++ m4/lispdir.m4       8 Jan 2003 18:13:20 -0000
@@ -4,7 +4,7 @@
 ## Almost entirely rewritten by Alexandre Oliva
 ## ------------------------
 
-# Copyright 1996, 1997, 1998, 1999, 2000, 2001, 2002
+# Copyright 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003
 #   Free Software Foundation, Inc.
 
 # This program is free software; you can redistribute it and/or modify
@@ -27,7 +27,7 @@
 # AM_PATH_LISPDIR
 # ---------------
 AC_DEFUN([AM_PATH_LISPDIR],
-[AC_ARG_WITH(lispdir,
+[AC_ARG_WITH([lispdir],
  [  --with-lispdir          Override the default lisp directory ],
  [ lispdir="$withval"
    AC_MSG_CHECKING([where .elc files should go])
@@ -36,7 +36,9 @@
  # If set to t, that means we are running in a shell under Emacs.
  # If you have an Emacs named "t", then use the full path.
  test x"$EMACS" = xt && EMACS=
- AC_CHECK_PROGS(EMACS, emacs xemacs, no)
+ AC_CHECK_PROGS([EMACS], [emacs xemacs], [no])
+ AC_ARG_VAR([EMACS], [the Emacs editor command])
+ AC_ARG_VAR([EMACSLOADPATH], [the Emacs library search path])
  if test $EMACS != "no"; then
    if test x${lispdir+set} != xset; then
      AC_CACHE_CHECK([where .elc files should go], [am_cv_lispdir],
@@ -60,7 +62,7 @@
    fi
  fi
 ])
-AC_SUBST(lispdir)
+AC_SUBST([lispdir])
 ])# AM_PATH_LISPDIR
 
 AU_DEFUN([ud_PATH_LISPDIR], [AM_PATH_LISPDIR])

-- 
Alexandre Duret-Lutz