[Emacs-diffs] trunk r113193: Do not create locallisppath directories.

[Juanma Barranquero]

------------------------------------------------------------
revno: 113193
revision-id: address@hidden
parent: address@hidden
committer: Juanma Barranquero <address@hidden>
branch nick: trunk
timestamp: Thu 2013-06-27 03:40:46 +0200
message:
  Do not create locallisppath directories.
  
  * Makefile.in (install-arch-indep): Do not create directories passed
  with --enable-locallisppath.
  
  * etc/NEWS: Mention policy change with respect to locallisppath dirs.
modified:
  ChangeLog                      changelog-20091113204419-o5vbwnq5f7feedwu-1538
  Makefile.in                    makefile.in-20091113204419-o5vbwnq5f7feedwu-446
  etc/ChangeLog                  changelog-20091113204419-o5vbwnq5f7feedwu-1485
  etc/NEWS                       news-20100311060928-aoit31wvzf25yr1z-1

=== modified file 'ChangeLog'
--- a/ChangeLog 2013-06-24 06:58:52 +0000
+++ b/ChangeLog 2013-06-27 01:40:46 +0000
@@ -1,3 +1,8 @@
+2013-06-27  Juanma Barranquero  <address@hidden>
+
+       * Makefile.in (install-arch-indep): Do not create directories passed
+       with --enable-locallisppath.
+
 2013-06-24  Glenn Morris  <address@hidden>
 
        * configure.ac: Include X11/X.h when testing for Xft.h.  (Bug#14684)

=== modified file 'Makefile.in'
--- a/Makefile.in       2013-06-07 08:08:29 +0000
+++ b/Makefile.in       2013-06-27 01:40:46 +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/ChangeLog'
--- a/etc/ChangeLog     2013-06-19 20:10:57 +0000
+++ b/etc/ChangeLog     2013-06-27 01:40:46 +0000
@@ -1,3 +1,7 @@
+2013-06-27  Juanma Barranquero  <address@hidden>
+
+       * NEWS: Mention policy change with respect to locallisppath dirs.
+
 2013-06-18  Juanma Barranquero  <address@hidden>
 
        * NEWS: Document new Prettify Symbols mode.

=== modified file 'etc/NEWS'
--- a/etc/NEWS  2013-06-26 12:54:33 +0000
+++ b/etc/NEWS  2013-06-27 01:40:46 +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