emacs-devel
[Top][All Lists]
Advanced

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

Re: Creating (or not) the directories in --locallisppath


From: Juanma Barranquero
Subject: Re: Creating (or not) the directories in --locallisppath
Date: Tue, 25 Jun 2013 12:03:45 +0200

If no one objects, then, I will shortly install the following patch.


2013-06-25  Juanma Barranquero  <address@hidden>

        * Makefile.in (install-arch-indep): Do not create directories passed
        with --enable-locallisppath.


2013-06-25  Juanma Barranquero  <address@hidden>

        * NEWS: Mention policy change with respect to locallisppath dirs.



=== modified file 'Makefile.in'
--- Makefile.in 2013-06-07 08:08:29 +0000
+++ Makefile.in 2013-06-25 09:26:31 +0000
@@ -548,13 +548,9 @@
 ## Note that the Makefiles in the etc directory are potentially useful
 ## in an installed Emacs, so should not be excluded.

-## I'm not sure creating locallisppath here serves any useful purpose.
-## If it has the default value, then the later write_subdir commands
-## will ensure all these components exist.
-## This will only do something if locallisppath has a non-standard value.
-## Is it really Emacs's job to create those directories?
-## Should we also be ensuring they contain subdirs.el files?
-## It would be easy to do, just use write_subdir.
+## We used to create locallisppath, but if it points to non-standard
+## locations, is not really Emacs's job to create these directories,
+## so it is no longer done.

 ## Note that we use tar instead of plain old cp -R/-r because the latter
 ## is apparently not portable (even in 2012!).
@@ -564,12 +560,6 @@
 ## See also these comments from 2004 about cp -r working fine:
 ## http://lists.gnu.org/archive/html/autoconf-patches/2004-11/msg00005.html
 install-arch-indep: lisp leim install-info install-man
${INSTALL_ARCH_INDEP_EXTRA}
- umask 022 ; \
- locallisppath='${locallisppath}'; \
- IFS=:; \
- for d in $$locallisppath; do \
-  ${MKDIR_P} "$(DESTDIR)$$d"; \
- done
  -set ${COPYDESTS} ; \
  unset CDPATH; \
  $(set_installuser); \

=== modified file 'etc/NEWS'
--- etc/NEWS 2013-06-23 19:24:27 +0000
+++ etc/NEWS 2013-06-25 09:36:11 +0000
@@ -57,6 +57,9 @@
 directories and will find the files in there automatically; there's no
 need to set any variables due to this change.)

+** Directories passed to configure option `--enable-locallisppath' are
+no longer created during installation.
+

 * Startup Changes in Emacs 24.4



reply via email to

[Prev in Thread] Current Thread [Next in Thread]