Re: [PATCH] Add --enable-locallisppath option to configure.in

["Jérôme Marant" <jmarant@free.fr>]

Hi,

Would anybody be kind to either consider or reject this small patch or
maybe propose a better way to achieve the goal?
I haven't read any comment so far.

(Here is an update)

Thanks in advance.

Jérôme Marant <jmarant@free.fr> writes:

> Jérôme Marant <jmarant@free.fr> writes:
>
>> Hi,
>>
>> Currently, only the configure script generates epaths, by calling the
>> 'epaths-force' target of the Makefile.
>>
>> However, it is not possible to customize the value of the
>> locallisppath variable in order to include it in epaths at
>> configure-time, which makes necessary to re-run 'make epaths-force'
>> with a customized locallisppath.
>>
>> The following patch adds a --enable-locallisppath option for this
>> purpose. It attemps to follow the --enable-* options semantic:
>
> This one should be better:

diff -ur emacs.orig/ChangeLog emacs/ChangeLog
--- emacs.orig/ChangeLog	2005-05-10 22:28:12.000000000 +0200
+++ emacs/ChangeLog	2005-05-18 20:01:34.388771568 +0200
@@ -1,3 +1,7 @@
+2005-05-11  Jerome Marant  <jmarant@marant.org>
+
+	* configure.in: Add --enable-locallisppath.
+
 2005-05-07  Jérôme Marant  <jerome@marant.org>
 
 	* make-dist: Remove references to makefile.nt and makefile.def.
diff -ur emacs.orig/configure.in emacs/configure.in
--- emacs.orig/configure.in	2005-04-23 19:02:09.000000000 +0200
+++ emacs/configure.in	2005-05-18 20:02:51.733013456 +0200
@@ -142,6 +142,16 @@
 fi
 AC_SUBST(MAINT)
 
+AC_ARG_ENABLE(locallisppath,
+[  --enable-locallisppath=PATH
+                          directories Emacs should search for lisp files
+                          specific to this site],
+if test "${enableval}" = "no"; then
+  locallisppath=
+elif test "${enableval}" != "yes"; then
+  locallisppath=${enableval}
+fi)
+
 #### Make srcdir absolute, if it isn't already.  It's important to
 #### avoid running the path through pwd unnecessarily, since pwd can
 #### give you automounter prefixes, which can go away.  We do all this


-- 
Jérôme Marant