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

[Lute Kamstra <Lute.Kamstra.lists@xs4all.nl>]

Hi Jérôme,

> 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

Could you explain what you want to accomplish?  It seems to me that you
want to be able to _add_ a dir to locallisppath.  However, when you do
"./configure --enable-locallisppath=/my/dir", you _replace_ the default
value of locallisppath (which is '${datadir}/emacs/${version}/site-lisp:'\
'${datadir}/emacs/site-lisp:${datadir}/emacs/${version}/leim') with
/my/dir.

Lute.