]> code.delx.au - gnu-emacs/commitdiff
Progress towards allowing installation in directories with whitespace
authorGlenn Morris <rgm@gnu.org>
Wed, 23 Oct 2013 07:20:57 +0000 (00:20 -0700)
committerGlenn Morris <rgm@gnu.org>
Wed, 23 Oct 2013 07:20:57 +0000 (00:20 -0700)
* Makefile.in (COPYDESTS, write_subdir, install-arch-dep)
(install-arch-indep, install-etcdoc, install-info, install-man)
(install-etc, uninstall): Quote entities that might contain whitespace.

* build-aux/update-subdirs: Handle whitespace in argument.
Check cd return value.

* doc/emacs/Makefile.in (install-dvi, install-html, install-pdf)
(install-ps, uninstall-dvi, uninstall-html, uninstall-ps)
(uninstall-pdf): Quote entities that might contain whitespace.

* doc/lispintro/Makefile.in (install-dvi, install-html, install-pdf)
(install-ps, uninstall-dvi, uninstall-html, uninstall-ps)
(uninstall-pdf): Quote entities that might contain whitespace.

* doc/lispref/Makefile.in (install-dvi, install-html, install-pdf)
(install-ps, uninstall-dvi, uninstall-html, uninstall-ps)
(uninstall-pdf): Quote entities that might contain whitespace.

* doc/misc/Makefile.in (install-dvi, install-html, install-pdf)
(install-ps, uninstall-dvi, uninstall-html, uninstall-ps)
(uninstall-pdf): Quote entities that might contain whitespace.

* lib-src/Makefile.in ($(DESTDIR)${archlibdir}, need-blessmail, install)
(uninstall): Quote entities that might contain whitespace.

* nt/Makefile.in ($(DESTDIR)${archlibdir}, install, uninstall):
Quote entities that might contain whitespace.

15 files changed:
ChangeLog
Makefile.in
build-aux/update-subdirs
doc/emacs/ChangeLog
doc/emacs/Makefile.in
doc/lispintro/ChangeLog
doc/lispintro/Makefile.in
doc/lispref/ChangeLog
doc/lispref/Makefile.in
doc/misc/ChangeLog
doc/misc/Makefile.in
lib-src/ChangeLog
lib-src/Makefile.in
nt/ChangeLog
nt/Makefile.in

index afa99cd58eac20b886cf153a9890e798586692b7..00658d0df8831f4dc1b2c6ac89978b6bd190fc32 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,12 @@
 2013-10-23  Glenn Morris  <rgm@gnu.org>
 
+       Progress towards allowing installation in directories with whitespace.
+       * Makefile.in (COPYDESTS, write_subdir, install-arch-dep)
+       (install-arch-indep, install-etcdoc, install-info, install-man)
+       (install-etc, uninstall): Quote entities that might contain whitespace.
+       * build-aux/update-subdirs: Handle whitespace in argument.
+       Check cd return value.
+
        Make building in directories with whitespace possible.  (Bug#15675)
        * configure.ac (srcdir): Don't make it absolute - abs_srcdir exists.
        * Makefile.in (abs_srcdir): New, set by configure.
index 25e170200a3a490d204a1d98fe993b755929baaa..aa3ade158ed769ff01de06ae3781e714723c25a5 100644 (file)
@@ -284,7 +284,7 @@ SUBDIR_MAKEFILES = `echo $(SUBDIR_MAKEFILES_IN:.in=) | sed 's|$(srcdir)/||g'`
 # separately.  quail appears twice because in out-of-tree builds, it
 # exists twice.
 COPYDIR = ${srcdir}/etc ${srcdir}/lisp ${srcdir}/leim/ja-dic ${srcdir}/leim/quail leim/quail
-COPYDESTS = $(DESTDIR)${etcdir} $(DESTDIR)${lispdir} $(DESTDIR)${leimdir}/ja-dic $(DESTDIR)${leimdir}/quail $(DESTDIR)${leimdir}/quail
+COPYDESTS = "$(DESTDIR)${etcdir}" "$(DESTDIR)${lispdir}" "$(DESTDIR)${leimdir}/ja-dic" "$(DESTDIR)${leimdir}/quail" "$(DESTDIR)${leimdir}/quail"
 
 all: ${SUBDIR}
 
@@ -469,32 +469,32 @@ install: all install-arch-indep install-etcdoc install-arch-dep install-$(NTDIR)
 ## world-readable.
 ## TODO it might be good to warn about non-standard permissions of
 ## pre-existing directories, but that does not seem easy.
-write_subdir=if [ -f $${subdir}/subdirs.el ]; \
+write_subdir=if [ -f "$${subdir}/subdirs.el" ]; \
        then true; \
        else \
          umask 022; \
-         ${MKDIR_P} $${subdir}; \
+         ${MKDIR_P} "$${subdir}"; \
          (echo "(if (fboundp 'normal-top-level-add-subdirs-to-load-path)"; \
           echo "    (normal-top-level-add-subdirs-to-load-path))") \
-           > $${subdir}/subdirs.el; \
+           > "$${subdir}/subdirs.el"; \
        fi
 
 ### Install the executables that were compiled specifically for this machine.
 ### We do install-arch-indep first because the executable needs the
 ### Lisp files and DOC file to work properly.
 install-arch-dep: src install-arch-indep install-etcdoc install-$(NTDIR)
-       umask 022; ${MKDIR_P} $(DESTDIR)${bindir}
+       umask 022; ${MKDIR_P} "$(DESTDIR)${bindir}"
        cd lib-src && \
          $(MAKE) install $(MFLAGS) prefix=${prefix} \
            exec_prefix=${exec_prefix} bindir=${bindir} \
            libexecdir=${libexecdir} archlibdir=${archlibdir} \
            INSTALL_STRIP=${INSTALL_STRIP}
        if test "${ns_self_contained}" = "no"; then \
-         ${INSTALL_PROGRAM} $(INSTALL_STRIP) src/emacs${EXEEXT} $(DESTDIR)${bindir}/$(EMACSFULL) || exit 1 ; \
-         chmod 1755 $(DESTDIR)${bindir}/$(EMACSFULL) || true; \
+         ${INSTALL_PROGRAM} $(INSTALL_STRIP) src/emacs${EXEEXT} "$(DESTDIR)${bindir}/$(EMACSFULL)" || exit 1 ; \
+         chmod 1755 "$(DESTDIR)${bindir}/$(EMACSFULL)" || true; \
          if test "x${NO_BIN_LINK}" = x; then \
-           rm -f $(DESTDIR)${bindir}/$(EMACS) ; \
-           cd $(DESTDIR)${bindir} && $(LN_S_FILEONLY) $(EMACSFULL) $(EMACS); \
+           rm -f "$(DESTDIR)${bindir}/$(EMACS)" ; \
+           cd "$(DESTDIR)${bindir}" && $(LN_S_FILEONLY) $(EMACSFULL) $(EMACS); \
          fi; \
        else \
          subdir=${ns_appresdir}/site-lisp; \
@@ -568,62 +568,63 @@ install-arch-indep: lisp leim install-info install-man ${INSTALL_ARCH_INDEP_EXTR
        $(set_installuser); \
        for dir in ${COPYDIR} ; do \
          [ -d $${dir} ] || exit 1 ; \
-         dest=$$1 ; shift ; \
-         [ -d $${dest} ] && \
-           [ "`cd $${dest} && /bin/pwd`" = "`cd $${dir} && /bin/pwd`" ] && \
+         dest="$$1" ; shift ; \
+         [ -d "$${dest}" ] && \
+           [ "`cd \"$${dest}\" && /bin/pwd`" = "`cd $${dir} && /bin/pwd`" ] && \
            continue ; \
          if [ "$${dir}" = "leim/quail" ]; then \
            [ "`cd $${dir} && /bin/pwd`" = "`cd ${srcdir}/leim/quail && /bin/pwd`" ] && \
              continue ; \
          else \
-           rm -rf $${dest} ; \
-           umask 022; ${MKDIR_P} $${dest} ; \
+           rm -rf "$${dest}" ; \
+           umask 022; ${MKDIR_P} "$${dest}" ; \
          fi ; \
          echo "Copying $${dir} to $${dest}..." ; \
          (cd $${dir}; tar -chf - . ) \
-           | (cd $${dest}; umask 022; \
+           | (cd "$${dest}"; umask 022; \
               tar -xvf - && cat > /dev/null) || exit 1; \
          if [ "$${dir}" = "${srcdir}/etc" ]; then \
-             rm -f $${dest}/DOC* ; \
-             rm -f $${dest}/refcards/*.aux $${dest}/refcards/*.dvi; \
-             rm -f $${dest}/refcards/*.log; \
+             rm -f "$${dest}/DOC"* ; \
+             rm -f "$${dest}/refcards"/*.aux "$${dest}/refcards"/*.dvi; \
+             rm -f "$${dest}/refcards"/*.log; \
          else true; \
          fi; \
-         for subdir in `find $${dest} -type d -print` ; do \
-           chmod a+rx $${subdir} ; \
-           rm -f $${subdir}/.gitignore ; \
-           rm -f $${subdir}/.arch-inventory ; \
-           rm -f $${subdir}/.DS_Store ; \
-           rm -f $${subdir}/\#* ; \
-           rm -f $${subdir}/.\#* ; \
-           rm -f $${subdir}/*~ ; \
-           rm -f $${subdir}/*.orig ; \
-           rm -f $${subdir}/ChangeLog* ; \
-           [ "$${dir}" != "${srcdir}/etc" ] && \
-             rm -f $${subdir}/[mM]akefile*[.-]in $${subdir}/[mM]akefile ; \
-         done ; \
-         find $${dest} -exec chown $${installuser} {} ';' ;\
+         (cd "$${dest}" || exit 1; \
+           for subdir in `find . -type d -print` ; do \
+             chmod a+rx $${subdir} ; \
+             rm -f $${subdir}/.gitignore ; \
+             rm -f $${subdir}/.arch-inventory ; \
+             rm -f $${subdir}/.DS_Store ; \
+             rm -f $${subdir}/\#* ; \
+             rm -f $${subdir}/.\#* ; \
+             rm -f $${subdir}/*~ ; \
+             rm -f $${subdir}/*.orig ; \
+             rm -f $${subdir}/ChangeLog* ; \
+             [ "$${dir}" != "${srcdir}/etc" ] && \
+               rm -f $${subdir}/[mM]akefile*[.-]in $${subdir}/[mM]akefile ; \
+           done ); \
+         find "$${dest}" -exec chown $${installuser} {} ';' ;\
        done
-       -rm -f $(DESTDIR)${leimdir}/leim-list.el
-       ${INSTALL_DATA} leim/leim-list.el $(DESTDIR)${leimdir}/leim-list.el
-       -rm -f $(DESTDIR)${lispdir}/subdirs.el
-       umask 022; $(srcdir)/build-aux/update-subdirs $(DESTDIR)${lispdir}
-       subdir=$(DESTDIR)${datadir}/emacs/${version}/site-lisp ; \
+       -rm -f "$(DESTDIR)${leimdir}/leim-list.el"
+       ${INSTALL_DATA} leim/leim-list.el "$(DESTDIR)${leimdir}/leim-list.el"
+       -rm -f "$(DESTDIR)${lispdir}/subdirs.el"
+       umask 022; $(srcdir)/build-aux/update-subdirs "$(DESTDIR)${lispdir}"
+       subdir="$(DESTDIR)${datadir}/emacs/${version}/site-lisp" ; \
          ${write_subdir}
-       subdir=$(DESTDIR)${datadir}/emacs/site-lisp ; \
+       subdir="$(DESTDIR)${datadir}/emacs/site-lisp" ; \
          ${write_subdir} || true
        [ -z "${GZIP_PROG}" ] || \
          ( echo "Compressing *.el ..." ; \
            unset CDPATH; \
            thisdir=`/bin/pwd`; \
-           for dir in $(DESTDIR)${lispdir} $(DESTDIR)${leimdir}; do \
+           for dir in "$(DESTDIR)${lispdir}" "$(DESTDIR)${leimdir}"; do \
              cd "$${thisdir}" ; \
-             cd $${dir} || exit 1 ; \
+             cd "$${dir}" || exit 1 ; \
              for f in `find . -name "*.elc" -print`; do \
-               ${GZIP_PROG} -9n `echo $$f|sed 's/.elc$$/.el/'` ; \
+               ${GZIP_PROG} -9n "`echo \"$$f\" | sed 's/.elc$$/.el/'`" ; \
              done ; \
            done )
-       -chmod -R a+r $(DESTDIR)${datadir}/emacs/${version} ${COPYDESTS}
+       -chmod -R a+r "$(DESTDIR)${datadir}/emacs/${version}" ${COPYDESTS}
 
 ## The above chmods are needed because "umask 022; tar ..." is not
 ## guaranteed to do the right thing; eg if we are root and tar is
@@ -633,40 +634,40 @@ install-arch-indep: lisp leim install-info install-man ${INSTALL_ARCH_INDEP_EXTR
 ## installed etc/ directory, so we need it to run before this does.
 install-etcdoc: src install-arch-indep
        -unset CDPATH; \
-       umask 022; ${MKDIR_P} $(DESTDIR)${etcdocdir} ; \
-       if [ "`cd ./etc; /bin/pwd`" != "`cd $(DESTDIR)${etcdocdir}; /bin/pwd`" ]; \
+       umask 022; ${MKDIR_P} "$(DESTDIR)${etcdocdir}" ; \
+       if [ "`cd ./etc; /bin/pwd`" != "`cd \"$(DESTDIR)${etcdocdir}\"; /bin/pwd`" ]; \
        then \
           docfile="DOC"; \
           echo "Copying etc/$${docfile} to $(DESTDIR)${etcdocdir} ..." ; \
-          ${INSTALL_DATA} etc/$${docfile} $(DESTDIR)${etcdocdir}/$${docfile}; \
+          ${INSTALL_DATA} etc/$${docfile} "$(DESTDIR)${etcdocdir}/$${docfile}"; \
           $(set_installuser); \
-            chown $${installuser} $(DESTDIR)${etcdocdir}/$${docfile} || true ; \
+            chown $${installuser} "$(DESTDIR)${etcdocdir}/$${docfile}" || true ; \
        else true; fi
 
 install-info: info
-       umask 022; ${MKDIR_P} $(DESTDIR)${infodir}
+       umask 022; ${MKDIR_P} "$(DESTDIR)${infodir}"
        -unset CDPATH; \
        thisdir=`/bin/pwd`; \
-       if [ "`cd ${srcdir}/info && /bin/pwd`" = "`cd $(DESTDIR)${infodir} && /bin/pwd`" ]; then \
+       if [ "`cd ${srcdir}/info && /bin/pwd`" = "`cd \"$(DESTDIR)${infodir}\" && /bin/pwd`" ]; then \
          true; \
        else \
          (cd "$(DESTDIR)${infodir}"; \
           [ -f dir ] || \
             (cd "$${thisdir}"; \
-             ${INSTALL_DATA} ${srcdir}/info/dir $(DESTDIR)${infodir}/dir) ); \
+             ${INSTALL_DATA} ${srcdir}/info/dir "$(DESTDIR)${infodir}/dir") ); \
           info_misc=`cd doc/misc && $(QUIET_SUBMAKE) $(MAKE) -s echo-info`; \
           cd ${srcdir}/info ; \
           for elt in ${INFO_NONMISC} $${info_misc}; do \
              test "$(HAVE_MAKEINFO)" = "no" && test ! -f $$elt && continue; \
              for f in `ls $$elt $$elt-[1-9] $$elt-[1-9][0-9] 2>/dev/null`; do \
               (cd "$${thisdir}"; \
-               ${INSTALL_DATA} ${srcdir}/info/$$f $(DESTDIR)${infodir}/$$f); \
+               ${INSTALL_DATA} ${srcdir}/info/$$f "$(DESTDIR)${infodir}/$$f"); \
                [ -n "${GZIP_PROG}" ] || continue ; \
-               rm -f $(DESTDIR)${infodir}/$$f.gz; \
-               ${GZIP_PROG} -9n $(DESTDIR)${infodir}/$$f; \
+               rm -f "$(DESTDIR)${infodir}/$$f.gz"; \
+               ${GZIP_PROG} -9n "$(DESTDIR)${infodir}/$$f"; \
              done; \
             (cd "$${thisdir}"; \
-             ${INSTALL_INFO} --info-dir=$(DESTDIR)${infodir} $(DESTDIR)${infodir}/$$elt); \
+             ${INSTALL_INFO} --info-dir="$(DESTDIR)${infodir}" "$(DESTDIR)${infodir}/$$elt"); \
           done; \
        fi
 
@@ -677,16 +678,16 @@ install-info: info
 ## ctags.1 is compressed or not.  "gzip -f" is another option here,
 ## but not sure if portable.
 install-man:
-       umask 022; ${MKDIR_P} $(DESTDIR)${man1dir}
+       umask 022; ${MKDIR_P} "$(DESTDIR)${man1dir}"
        thisdir=`/bin/pwd`; \
        cd ${mansrcdir}; \
        for page in *.1; do \
          dest=`echo "$${page}" | sed -e 's/\.1$$//' -e '$(TRANSFORM)'`.1; \
          (cd "$${thisdir}"; \
-          ${INSTALL_DATA} ${mansrcdir}/$${page} $(DESTDIR)${man1dir}/$${dest}); \
+          ${INSTALL_DATA} ${mansrcdir}/$${page} "$(DESTDIR)${man1dir}/$${dest}"); \
          [ -n "${GZIP_PROG}" ] || continue ; \
-         rm -f $(DESTDIR)${man1dir}/$${dest}.gz; \
-         ${GZIP_PROG} -9n $(DESTDIR)${man1dir}/$${dest} || true; \
+         rm -f "$(DESTDIR)${man1dir}/$${dest}.gz"; \
+         ${GZIP_PROG} -9n "$(DESTDIR)${man1dir}/$${dest}" || true; \
        done
 
 ## Install those items from etc/ that need to end up elsewhere.
@@ -696,25 +697,25 @@ install-man:
 EMACS_ICON=emacs
 
 install-etc:
-       umask 022; ${MKDIR_P} $(DESTDIR)${desktopdir}
+       umask 022; ${MKDIR_P} "$(DESTDIR)${desktopdir}"
        tmp=etc/emacs.tmpdesktop; rm -f $${tmp}; \
        emacs_name=`echo emacs | sed '$(TRANSFORM)'`; \
        sed -e "/^Exec=emacs/ s/emacs/$${emacs_name}/" \
          -e "/^Icon=emacs/ s/emacs/$${emacs_name}/" \
          ${srcdir}/etc/emacs.desktop > $${tmp}; \
-       ${INSTALL_DATA} $${tmp} $(DESTDIR)${desktopdir}/${EMACS_NAME}.desktop; \
+       ${INSTALL_DATA} $${tmp} "$(DESTDIR)${desktopdir}/${EMACS_NAME}.desktop"; \
        rm -f $${tmp}
        thisdir=`/bin/pwd`; \
        cd ${iconsrcdir} || exit 1; umask 022 ; \
        for dir in */*/apps */*/mimetypes; do \
          [ -d $${dir} ] || continue ; \
-         ( cd "$${thisdir}"; ${MKDIR_P} $(DESTDIR)${icondir}/$${dir} ) ; \
+         ( cd "$${thisdir}"; ${MKDIR_P} "$(DESTDIR)${icondir}/$${dir}" ) ; \
          for icon in $${dir}/${EMACS_ICON}[.-]*; do \
            [ -r $${icon} ] || continue ; \
            ext=`echo "$${icon}" | sed -e 's|.*\.||'`; \
            dest=`echo "$${icon}" | sed -e 's|.*/||' -e "s|\.$${ext}$$||" -e 's/$(EMACS_ICON)/emacs/' -e '$(TRANSFORM)'`.$${ext} ; \
            ( cd "$${thisdir}"; \
-             ${INSTALL_DATA} ${iconsrcdir}/$${icon} $(DESTDIR)${icondir}/$${dir}/$${dest} ) \
+             ${INSTALL_DATA} ${iconsrcdir}/$${icon} "$(DESTDIR)${icondir}/$${dir}/$${dest}" ) \
            || exit 1; \
          done ; \
        done
@@ -734,26 +735,26 @@ uninstall: uninstall-$(NTDIR) uninstall-doc
            bindir=${bindir} libexecdir=${libexecdir} archlibdir=${archlibdir}
 
        -unset CDPATH; \
-       for dir in $(DESTDIR)${lispdir} $(DESTDIR)${etcdir} ; do        \
-         if [ -d $${dir} ]; then                       \
-           case "`cd $${dir} ; /bin/pwd`" in           \
+       for dir in "$(DESTDIR)${lispdir}" "$(DESTDIR)${etcdir}" ; do    \
+         if [ -d "$${dir}" ]; then                     \
+           case `cd "$${dir}" ; /bin/pwd` in           \
              "`cd ${srcdir} ; /bin/pwd`"* ) ;;         \
-             * ) rm -rf $${dir} ;;                     \
+             * ) rm -rf "$${dir}" ;;                   \
            esac ;                                      \
-           case $${dir} in                             \
-             $(DESTDIR)${datadir}/emacs/${version}/* )         \
-               rm -rf $(DESTDIR)${datadir}/emacs/${version}    \
+           case "$${dir}" in                           \
+             "$(DESTDIR)${datadir}/emacs/${version}"/* )               \
+               rm -rf "$(DESTDIR)${datadir}/emacs/${version}"  \
              ;;                                        \
            esac ;                                      \
          fi ;                                          \
        done
-       -rm -rf $(DESTDIR)${libexecdir}/emacs/${version}
+       -rm -rf "$(DESTDIR)${libexecdir}/emacs/${version}"
        thisdir=`/bin/pwd`; \
        (info_misc=`cd doc/misc && $(QUIET_SUBMAKE) $(MAKE) -s echo-info`; \
-        if cd $(DESTDIR)${infodir}; then \
+        if cd "$(DESTDIR)${infodir}"; then \
           for elt in ${INFO_NONMISC} $${info_misc}; do \
             (cd "$${thisdir}"; \
-             $(INSTALL_INFO) --remove --info-dir=$(DESTDIR)${infodir} $(DESTDIR)${infodir}/$$elt); \
+             $(INSTALL_INFO) --remove --info-dir="$(DESTDIR)${infodir}" "$(DESTDIR)${infodir}/$$elt"); \
             if [ -n "${GZIP_PROG}" ]; then \
                ext=.gz; else ext=; fi; \
             rm -f $$elt$$ext $$elt-[1-9]$$ext $$elt-[1-9][0-9]$$ext; \
@@ -763,18 +764,18 @@ uninstall: uninstall-$(NTDIR) uninstall-doc
            ext=.gz; else ext=; fi; \
         if cd ${mansrcdir}; then \
           for page in *.1; do \
-            rm -f $(DESTDIR)${man1dir}/`echo "$${page}" | sed -e 's/\.1$$//' -e '$(TRANSFORM)'`.1$$ext; done; \
+            rm -f "$(DESTDIR)${man1dir}"/`echo "$${page}" | sed -e 's/\.1$$//' -e '$(TRANSFORM)'`.1$$ext; done; \
         fi)
-       (cd $(DESTDIR)${bindir} && rm -f $(EMACSFULL) $(EMACS) || true)
-       (if cd $(DESTDIR)${icondir}; then \
+       (cd "$(DESTDIR)${bindir}" && rm -f $(EMACSFULL) $(EMACS) || true)
+       (if cd "$(DESTDIR)${icondir}"; then \
           rm -f hicolor/*x*/apps/${EMACS_NAME}.png \
             hicolor/scalable/apps/${EMACS_NAME}.svg \
             hicolor/scalable/mimetypes/`echo emacs-document | sed '$(TRANSFORM)'`.svg; \
        fi)
-       -rm -f $(DESTDIR)${desktopdir}/${EMACS_NAME}.desktop
+       -rm -f "$(DESTDIR)${desktopdir}/${EMACS_NAME}.desktop"
        for file in snake-scores tetris-scores; do \
-         file=$(DESTDIR)${gamedir}/$${file}; \
-         [ -s $${file} ] || rm -f $$file; \
+         file="$(DESTDIR)${gamedir}/$${file}"; \
+         [ -s "$${file}" ] || rm -f "$$file"; \
        done
 
 ### Windows-specific uninstall target for removing programs produced
index 3c429b8b5e431b43cc6406e2b1f09b05768c2bc7..fb271f76ddd4f30c98c386c71802df6ed0cd59bc 100755 (executable)
@@ -20,7 +20,7 @@
 # along with GNU Emacs.  If not, see <http://www.gnu.org/licenses/>.
 
 
-cd $1
+cd "$1" || exit 1
 for file in *; do
     case $file in
        *.elc | *.el | term | RCS | CVS | Old | . | .. | =* | *~ | *.orig | *.rej)
index f36097e5ed6517c530916b441daac8a7df807e16..eb82535031f86f8c5563d78f96fc3448099f049c 100644 (file)
@@ -1,6 +1,13 @@
+2013-10-23  Glenn Morris  <rgm@gnu.org>
+
+       * Makefile.in (install-dvi, install-html, install-pdf)
+       (install-ps, uninstall-dvi, uninstall-html, uninstall-ps)
+       (uninstall-pdf): Quote entities that might contain whitespace.
+
 2013-10-20  Xue Fuqiao  <xfq.free@gmail.com>
 
-       * custom.texi (Init Syntax, Terminal Init, Terminal Init): Remove @refill.
+       * custom.texi (Init Syntax, Terminal Init, Terminal Init):
+       Remove @refill.
 
 2013-10-13  Glenn Morris  <rgm@gnu.org>
 
index bf9f8152ad467d2bd3d3965bdd8f9fe1af303666..9c4a90db4aa1868b32c67f16dd7eb4648d369f9e 100644 (file)
@@ -229,21 +229,21 @@ dist:
 .PHONY: install-dvi install-html install-pdf install-ps install-doc
 
 install-dvi: dvi
-       umask 022; $(MKDIR_P) $(DESTDIR)$(dvidir)
-       $(INSTALL_DATA) $(DVI_TARGETS) $(DESTDIR)$(dvidir)
+       umask 022; $(MKDIR_P) "$(DESTDIR)$(dvidir)"
+       $(INSTALL_DATA) $(DVI_TARGETS) "$(DESTDIR)$(dvidir)"
 install-html: html
-       umask 022; $(MKDIR_P) $(DESTDIR)$(htmldir)
-       $(INSTALL_DATA) $(HTML_TARGETS) $(DESTDIR)$(htmldir)
+       umask 022; $(MKDIR_P) "$(DESTDIR)$(htmldir)"
+       $(INSTALL_DATA) $(HTML_TARGETS) "$(DESTDIR)$(htmldir)"
 install-pdf: pdf
-        umask 022;$(MKDIR_P) $(DESTDIR)$(pdfdir)
-       $(INSTALL_DATA) $(PDF_TARGETS) $(DESTDIR)$(pdfdir)
+        umask 022;$(MKDIR_P) "$(DESTDIR)$(pdfdir)"
+       $(INSTALL_DATA) $(PDF_TARGETS) "$(DESTDIR)$(pdfdir)"
 install-ps: ps
-       umask 022; $(MKDIR_P) $(DESTDIR)$(psdir)
+       umask 022; $(MKDIR_P) "$(DESTDIR)$(psdir)"
        for file in $(PS_TARGETS); do \
-         $(INSTALL_DATA) $${file} $(DESTDIR)$(psdir); \
+         $(INSTALL_DATA) $${file} "$(DESTDIR)$(psdir)"; \
          [ -n "${GZIP_PROG}" ] || continue; \
-         rm -f $(DESTDIR)$(psdir)/$${file}.gz; \
-         ${GZIP_PROG} -9n $(DESTDIR)$(psdir)/$${file}; \
+         rm -f "$(DESTDIR)$(psdir)/$${file}.gz"; \
+         ${GZIP_PROG} -9n "$(DESTDIR)$(psdir)/$${file}"; \
        done
 
 ## Top-level Makefile installs the info pages.
@@ -254,20 +254,20 @@ install-doc: install-dvi install-html install-pdf install-ps
 
 uninstall-dvi:
        for file in $(DVI_TARGETS); do \
-         rm -f $(DESTDIR)$(dvidir)/$${file}; \
+         rm -f "$(DESTDIR)$(dvidir)/$${file}"; \
        done
 uninstall-html:
        for file in $(HTML_TARGETS); do \
-         rm -f $(DESTDIR)$(htmldir)/$${file}; \
+         rm -f "$(DESTDIR)$(htmldir)/$${file}"; \
        done
 uninstall-ps:
        ext= ; [ -n "${GZIP_PROG}" ] && ext=.gz; \
        for file in $(PS_TARGETS); do \
-         rm -f $(DESTDIR)$(psdir)/$${file}$${ext}; \
+         rm -f "$(DESTDIR)$(psdir)/$${file}$${ext}"; \
        done
 uninstall-pdf:
        for file in $(PDF_TARGETS); do \
-         rm -f $(DESTDIR)$(pdfdir)/$${file}; \
+         rm -f "$(DESTDIR)$(pdfdir)/$${file}"; \
        done
 
 uninstall-doc: uninstall-dvi uninstall-html uninstall-pdf uninstall-ps
index ec082eecb528fe093493925b1c14c98533b521cb..2e6164968c391cea7d0256236c9478ca03320e99 100644 (file)
@@ -1,3 +1,9 @@
+2013-10-23  Glenn Morris  <rgm@gnu.org>
+
+       * Makefile.in (install-dvi, install-html, install-pdf)
+       (install-ps, uninstall-dvi, uninstall-html, uninstall-ps)
+       (uninstall-pdf): Quote entities that might contain whitespace.
+
 2013-09-01  Glenn Morris  <rgm@gnu.org>
 
        * emacs-lisp-intro.texi (beginning-of-buffer complete):
index 363e12215b3b06c71831eb02a274f98649ab0082..9a643c004ed8ff72f634ef7577e2107cc5682e53 100644 (file)
@@ -145,21 +145,21 @@ dist:
 .PHONY: install-dvi install-html install-pdf install-ps install-doc
 
 install-dvi: dvi
-       umask 022; $(MKDIR_P) $(DESTDIR)$(dvidir)
-       $(INSTALL_DATA) $(DVI_TARGETS) $(DESTDIR)$(dvidir)
+       umask 022; $(MKDIR_P) "$(DESTDIR)$(dvidir)"
+       $(INSTALL_DATA) $(DVI_TARGETS) "$(DESTDIR)$(dvidir)"
 install-html: html
-       umask 022; $(MKDIR_P) $(DESTDIR)$(htmldir)
-       $(INSTALL_DATA) $(HTML_TARGETS) $(DESTDIR)$(htmldir)
+       umask 022; $(MKDIR_P) "$(DESTDIR)$(htmldir)"
+       $(INSTALL_DATA) $(HTML_TARGETS) "$(DESTDIR)$(htmldir)"
 install-pdf: pdf
-        umask 022;$(MKDIR_P) $(DESTDIR)$(pdfdir)
-       $(INSTALL_DATA) $(PDF_TARGETS) $(DESTDIR)$(pdfdir)
+        umask 022;$(MKDIR_P) "$(DESTDIR)$(pdfdir)"
+       $(INSTALL_DATA) $(PDF_TARGETS) "$(DESTDIR)$(pdfdir)"
 install-ps: ps
-       umask 022; $(MKDIR_P) $(DESTDIR)$(psdir)
+       umask 022; $(MKDIR_P) "$(DESTDIR)$(psdir)"
        for file in $(PS_TARGETS); do \
-         $(INSTALL_DATA) $${file} $(DESTDIR)$(psdir); \
+         $(INSTALL_DATA) $${file} "$(DESTDIR)$(psdir)"; \
          [ -n "${GZIP_PROG}" ] || continue; \
-         rm -f $(DESTDIR)$(psdir)/$${file}.gz; \
-         ${GZIP_PROG} -9n $(DESTDIR)$(psdir)/$${file}; \
+         rm -f "$(DESTDIR)$(psdir)/$${file}.gz"; \
+         ${GZIP_PROG} -9n "$(DESTDIR)$(psdir)/$${file}"; \
        done
 
 ## Top-level Makefile installs the info pages.
@@ -170,20 +170,20 @@ install-doc: install-dvi install-html install-pdf install-ps
 
 uninstall-dvi:
        for file in $(DVI_TARGETS); do \
-         rm -f $(DESTDIR)$(dvidir)/$${file}; \
+         rm -f "$(DESTDIR)$(dvidir)/$${file}"; \
        done
 uninstall-html:
        for file in $(HTML_TARGETS); do \
-         rm -f $(DESTDIR)$(htmldir)/$${file}; \
+         rm -f "$(DESTDIR)$(htmldir)/$${file}"; \
        done
 uninstall-ps:
        ext= ; [ -n "${GZIP_PROG}" ] && ext=.gz; \
        for file in $(PS_TARGETS); do \
-         rm -f $(DESTDIR)$(psdir)/$${file}$${ext}; \
+         rm -f "$(DESTDIR)$(psdir)/$${file}$${ext}"; \
        done
 uninstall-pdf:
        for file in $(PDF_TARGETS); do \
-         rm -f $(DESTDIR)$(pdfdir)/$${file}; \
+         rm -f "$(DESTDIR)$(pdfdir)/$${file}"; \
        done
 
 uninstall-doc: uninstall-dvi uninstall-html uninstall-pdf uninstall-ps
index 426591500588061ad6a01a80d0196ca96ab37987..7efe70f683ded91c426af00f03a4b4b9e177c94a 100644 (file)
@@ -1,3 +1,9 @@
+2013-10-23  Glenn Morris  <rgm@gnu.org>
+
+       * Makefile.in (install-dvi, install-html, install-pdf)
+       (install-ps, uninstall-dvi, uninstall-html, uninstall-ps)
+       (uninstall-pdf): Quote entities that might contain whitespace.
+
 2013-10-19  Xue Fuqiao  <xfq.free@gmail.com>
 
        * display.texi (Face Attributes): Add indexes for the ‘:box’ face attribute.
index 164646ac4332f96632e838c3b67e88c97a8c43b2..e2cc9ac088a9eaa925253b2dfed69c85394bead7 100644 (file)
@@ -201,21 +201,21 @@ dist:
 .PHONY: install-dvi install-html install-pdf install-ps install-doc
 
 install-dvi: dvi
-       umask 022; $(MKDIR_P) $(DESTDIR)$(dvidir)
-       $(INSTALL_DATA) $(DVI_TARGETS) $(DESTDIR)$(dvidir)
+       umask 022; $(MKDIR_P) "$(DESTDIR)$(dvidir)"
+       $(INSTALL_DATA) $(DVI_TARGETS) "$(DESTDIR)$(dvidir)"
 install-html: html
-       umask 022; $(MKDIR_P) $(DESTDIR)$(htmldir)
-       $(INSTALL_DATA) $(HTML_TARGETS) $(DESTDIR)$(htmldir)
+       umask 022; $(MKDIR_P) "$(DESTDIR)$(htmldir)"
+       $(INSTALL_DATA) $(HTML_TARGETS) "$(DESTDIR)$(htmldir)"
 install-pdf: pdf
-        umask 022;$(MKDIR_P) $(DESTDIR)$(pdfdir)
-       $(INSTALL_DATA) $(PDF_TARGETS) $(DESTDIR)$(pdfdir)
+        umask 022;$(MKDIR_P) "$(DESTDIR)$(pdfdir)"
+       $(INSTALL_DATA) $(PDF_TARGETS) "$(DESTDIR)$(pdfdir)"
 install-ps: ps
-       umask 022; $(MKDIR_P) $(DESTDIR)$(psdir)
+       umask 022; $(MKDIR_P) "$(DESTDIR)$(psdir)"
        for file in $(PS_TARGETS); do \
-         $(INSTALL_DATA) $${file} $(DESTDIR)$(psdir); \
+         $(INSTALL_DATA) $${file} "$(DESTDIR)$(psdir)"; \
          [ -n "${GZIP_PROG}" ] || continue; \
-         rm -f $(DESTDIR)$(psdir)/$${file}.gz; \
-         ${GZIP_PROG} -9n $(DESTDIR)$(psdir)/$${file}; \
+         rm -f "$(DESTDIR)$(psdir)/$${file}.gz"; \
+         ${GZIP_PROG} -9n "$(DESTDIR)$(psdir)/$${file}"; \
        done
 
 ## Top-level Makefile installs the info pages.
@@ -226,20 +226,20 @@ install-doc: install-dvi install-html install-pdf install-ps
 
 uninstall-dvi:
        for file in $(DVI_TARGETS); do \
-         rm -f $(DESTDIR)$(dvidir)/$${file}; \
+         rm -f "$(DESTDIR)$(dvidir)/$${file}"; \
        done
 uninstall-html:
        for file in $(HTML_TARGETS); do \
-         rm -f $(DESTDIR)$(htmldir)/$${file}; \
+         rm -f "$(DESTDIR)$(htmldir)/$${file}"; \
        done
 uninstall-ps:
        ext= ; [ -n "${GZIP_PROG}" ] && ext=.gz; \
        for file in $(PS_TARGETS); do \
-         rm -f $(DESTDIR)$(psdir)/$${file}$${ext}; \
+         rm -f "$(DESTDIR)$(psdir)/$${file}$${ext}"; \
        done
 uninstall-pdf:
        for file in $(PDF_TARGETS); do \
-         rm -f $(DESTDIR)$(pdfdir)/$${file}; \
+         rm -f "$(DESTDIR)$(pdfdir)/$${file}"; \
        done
 
 uninstall-doc: uninstall-dvi uninstall-html uninstall-pdf uninstall-ps
index 76d3954cfcf19ee632672b309463ceb5ad287268..3563c1e768d489eb1c865bce1718ae7ba22bbd68 100644 (file)
@@ -1,3 +1,9 @@
+2013-10-23  Glenn Morris  <rgm@gnu.org>
+
+       * Makefile.in (install-dvi, install-html, install-pdf)
+       (install-ps, uninstall-dvi, uninstall-html, uninstall-ps)
+       (uninstall-pdf): Quote entities that might contain whitespace.
+
 2013-10-17  Jay Belanger  <jay.p.belanger@gmail.com>
 
        * calc.el (Data Type Formats): Don't specify the size at
index 30fc495317224f5c77f99728623deeb63d87dede..f43520e97a43c5a91a0afda734263075b6a59a55 100644 (file)
@@ -877,21 +877,21 @@ dist:
 .PHONY: install-dvi install-html install-pdf install-ps install-doc
 
 install-dvi: dvi
-       umask 022; $(MKDIR_P) $(DESTDIR)$(dvidir)
-       $(INSTALL_DATA) $(DVI_TARGETS) $(DESTDIR)$(dvidir)
+       umask 022; $(MKDIR_P) "$(DESTDIR)$(dvidir)"
+       $(INSTALL_DATA) $(DVI_TARGETS) "$(DESTDIR)$(dvidir)"
 install-html: html
-       umask 022; $(MKDIR_P) $(DESTDIR)$(htmldir)
-       $(INSTALL_DATA) $(HTML_TARGETS) $(DESTDIR)$(htmldir)
+       umask 022; $(MKDIR_P) "$(DESTDIR)$(htmldir)"
+       $(INSTALL_DATA) $(HTML_TARGETS) "$(DESTDIR)$(htmldir)"
 install-pdf: pdf
-        umask 022;$(MKDIR_P) $(DESTDIR)$(pdfdir)
-       $(INSTALL_DATA) $(PDF_TARGETS) $(DESTDIR)$(pdfdir)
+        umask 022;$(MKDIR_P) "$(DESTDIR)$(pdfdir)"
+       $(INSTALL_DATA) $(PDF_TARGETS) "$(DESTDIR)$(pdfdir)"
 install-ps: ps
-       umask 022; $(MKDIR_P) $(DESTDIR)$(psdir)
+       umask 022; $(MKDIR_P) "$(DESTDIR)$(psdir)"
        for file in $(PS_TARGETS); do \
-         $(INSTALL_DATA) $${file} $(DESTDIR)$(psdir); \
+         $(INSTALL_DATA) $${file} "$(DESTDIR)$(psdir)"; \
          [ -n "${GZIP_PROG}" ] || continue; \
-         rm -f $(DESTDIR)$(psdir)/$${file}.gz; \
-         ${GZIP_PROG} -9n $(DESTDIR)$(psdir)/$${file}; \
+         rm -f "$(DESTDIR)$(psdir)/$${file}.gz"; \
+         ${GZIP_PROG} -9n "$(DESTDIR)$(psdir)/$${file}"; \
        done
 
 ## Top-level Makefile installs the info pages.
@@ -903,20 +903,20 @@ install-doc: install-dvi install-html install-pdf install-ps
 
 uninstall-dvi:
        for file in $(DVI_TARGETS); do \
-         rm -f $(DESTDIR)$(dvidir)/$${file}; \
+         rm -f "$(DESTDIR)$(dvidir)/$${file}"; \
        done
 uninstall-html:
        for file in $(HTML_TARGETS); do \
-         rm -f $(DESTDIR)$(htmldir)/$${file}; \
+         rm -f "$(DESTDIR)$(htmldir)/$${file}"; \
        done
 uninstall-ps:
        ext= ; [ -n "${GZIP_PROG}" ] && ext=.gz; \
        for file in $(PS_TARGETS); do \
-         rm -f $(DESTDIR)$(psdir)/$${file}$${ext}; \
+         rm -f "$(DESTDIR)$(psdir)/$${file}$${ext}"; \
        done
 uninstall-pdf:
        for file in $(PDF_TARGETS); do \
-         rm -f $(DESTDIR)$(pdfdir)/$${file}; \
+         rm -f "$(DESTDIR)$(pdfdir)/$${file}"; \
        done
 
 uninstall-doc: uninstall-dvi uninstall-html uninstall-pdf uninstall-ps
index 46faf95c0c0fec839afc0a1429aa78ca624f925f..fd4209ab225d6e294d0417a26027ccea4a48f5fd 100644 (file)
@@ -1,7 +1,7 @@
 2013-10-23  Glenn Morris  <rgm@gnu.org>
 
-       * Makefile.in ($(DESTDIR)${archlibdir}):
-       Quote entities that might contain whitespace.
+       * Makefile.in ($(DESTDIR)${archlibdir}, need-blessmail, install)
+       (uninstall): Quote entities that might contain whitespace.
 
 2013-10-10  Glenn Morris  <rgm@gnu.org>
 
index 01c03e162ccb8dd2dc3700224b0c4d9d9c29d9d9..3aff16fed768d2620de476b4e777ab31b8c6a593 100644 (file)
@@ -214,10 +214,10 @@ blessmail:
 need-blessmail: blessmail
        @if [ `wc -l <blessmail` != 2 ] ; then \
          dir=`sed -n -e 's/echo mail directory = \(.*\)/\1/p' blessmail`; \
-         echo Assuming $$dir is really the mail spool directory, you should; \
-         echo run  lib-src/blessmail $(DESTDIR)${archlibdir}/movemail${EXEEXT}; \
-         echo as root, to give  movemail${EXEEXT}  appropriate permissions.; \
-         echo Do that after running  make install.; \
+         echo "Assuming $$dir is really the mail spool directory, you should"; \
+         echo "run  lib-src/blessmail $(DESTDIR)${archlibdir}/movemail${EXEEXT}"; \
+         echo "as root, to give  movemail${EXEEXT}  appropriate permissions."; \
+         echo "Do that after running  make install."; \
        fi
 
 ## This is the target invoked by the top-level Makefile.
@@ -231,23 +231,23 @@ maybe-blessmail: $(BLESSMAIL_TARGET)
 $(DESTDIR)${archlibdir}: all
        @echo
        @echo "Installing utilities run internally by Emacs."
-       umask 022; ${MKDIR_P} $(DESTDIR)${archlibdir}
-       if [ "`cd $(DESTDIR)${archlibdir} && /bin/pwd`" != "`/bin/pwd`" ]; then \
+       umask 022; ${MKDIR_P} "$(DESTDIR)${archlibdir}"
+       if [ "`cd \"$(DESTDIR)${archlibdir}\" && /bin/pwd`" != "`/bin/pwd`" ]; then \
          for file in ${UTILITIES}; do \
-           $(INSTALL_PROGRAM) $(INSTALL_STRIP) $$file $(DESTDIR)${archlibdir}/$$file ; \
+           $(INSTALL_PROGRAM) $(INSTALL_STRIP) $$file "$(DESTDIR)${archlibdir}/$$file" ; \
          done ; \
         fi
-       umask 022; ${MKDIR_P} $(DESTDIR)${gamedir}; \
-       touch $(DESTDIR)${gamedir}/snake-scores; \
-       touch $(DESTDIR)${gamedir}/tetris-scores
-       -if chown ${gameuser} $(DESTDIR)${archlibdir}/update-game-score${EXEEXT} && chmod u+s $(DESTDIR)${archlibdir}/update-game-score${EXEEXT}; then \
-         chown ${gameuser} $(DESTDIR)${gamedir}; \
-         chmod u=rwx,g=rwx,o=rx $(DESTDIR)${gamedir}; \
+       umask 022; ${MKDIR_P} "$(DESTDIR)${gamedir}"; \
+       touch "$(DESTDIR)${gamedir}/snake-scores"; \
+       touch "$(DESTDIR)${gamedir}/tetris-scores"
+       -if chown ${gameuser} "$(DESTDIR)${archlibdir}/update-game-score${EXEEXT}" && chmod u+s "$(DESTDIR)${archlibdir}/update-game-score${EXEEXT}"; then \
+         chown ${gameuser} "$(DESTDIR)${gamedir}"; \
+         chmod u=rwx,g=rwx,o=rx "$(DESTDIR)${gamedir}"; \
        fi
-       if [ "`cd $(DESTDIR)${archlibdir} && /bin/pwd`" \
+       if [ "`cd \"$(DESTDIR)${archlibdir}\" && /bin/pwd`" \
              != "`cd ${srcdir} && /bin/pwd`" ]; then \
          for file in ${SCRIPTS}; do \
-           $(INSTALL_SCRIPT) ${srcdir}/$$file $(DESTDIR)${archlibdir}/$$file; \
+           $(INSTALL_SCRIPT) ${srcdir}/$$file "$(DESTDIR)${archlibdir}/$$file"; \
          done ; \
        fi
 
@@ -257,23 +257,23 @@ $(DESTDIR)${archlibdir}: all
 install: $(DESTDIR)${archlibdir}
        @echo
        @echo "Installing utilities for users to run."
-       umask 022; ${MKDIR_P} $(DESTDIR)${bindir}
+       umask 022; ${MKDIR_P} "$(DESTDIR)${bindir}"
        for file in ${INSTALLABLES} ; do \
-         $(INSTALL_PROGRAM) $(INSTALL_STRIP) $${file} $(DESTDIR)${bindir}/`echo $${file} | sed -e 's/${EXEEXT}$$//' -e '$(TRANSFORM)'`${EXEEXT} ; \
+         $(INSTALL_PROGRAM) $(INSTALL_STRIP) $${file} "$(DESTDIR)${bindir}"/`echo $${file} | sed -e 's/${EXEEXT}$$//' -e '$(TRANSFORM)'`${EXEEXT} ; \
        done
        for file in ${INSTALLABLE_SCRIPTS} ; do \
-         $(INSTALL_SCRIPT) ${srcdir}/$${file} $(DESTDIR)${bindir}/`echo $${file} | sed '$(TRANSFORM)'`  ; \
+         $(INSTALL_SCRIPT) ${srcdir}/$${file} "$(DESTDIR)${bindir}"/`echo $${file} | sed '$(TRANSFORM)'`  ; \
        done
 
 uninstall:
        for file in ${INSTALLABLES}; do \
-         rm -f $(DESTDIR)${bindir}/`echo $${file} | sed -e 's/${EXEEXT}$$//' -e '$(TRANSFORM)'`${EXEEXT} ; \
+         rm -f "$(DESTDIR)${bindir}"/`echo $${file} | sed -e 's/${EXEEXT}$$//' -e '$(TRANSFORM)'`${EXEEXT} ; \
        done
        for file in ${INSTALLABLE_SCRIPTS}; do \
-         rm -f $(DESTDIR)${bindir}/`echo $${file} | sed '$(TRANSFORM)'` ; \
+         rm -f "$(DESTDIR)${bindir}"/`echo $${file} | sed '$(TRANSFORM)'` ; \
        done
-       if [ -d $(DESTDIR)${archlibdir} ]; then \
-         (cd $(DESTDIR)${archlibdir} && rm -f ${UTILITIES} ${SCRIPTS}) \
+       if [ -d "$(DESTDIR)${archlibdir}" ]; then \
+         (cd "$(DESTDIR)${archlibdir}" && rm -f ${UTILITIES} ${SCRIPTS}) \
        fi
 
 mostlyclean:
index a04a67b7ce9e20d9984fa2607b2443f1dde6fea0..53381f1e8b526aa5ab680b548e0fb2b8177d02d6 100644 (file)
@@ -1,6 +1,6 @@
 2013-10-23  Glenn Morris  <rgm@gnu.org>
 
-       * Makefile.in ($(DESTDIR)${archlibdir}):
+       * Makefile.in ($(DESTDIR)${archlibdir}, install, uninstall):
        Quote entities that might contain whitespace.
 
 2013-10-14  Eli Zaretskii  <eliz@gnu.org>
index cc640152cb6198dcbd939e9feb32b7d8d3cb73d4..c29cc7e91ec4c6a94783ffad86ff3fef55e28a1a 100644 (file)
@@ -141,10 +141,10 @@ all: ${EXE_FILES}
 $(DESTDIR)${archlibdir}: all
        @echo
        @echo "Installing utilities run internally by Emacs."
-       umask 022; ${MKDIR_P} $(DESTDIR)${archlibdir}
-       if [ "`cd $(DESTDIR)${archlibdir} && /bin/pwd`" != "`/bin/pwd`" ]; then \
+       umask 022; ${MKDIR_P} "$(DESTDIR)${archlibdir}"
+       if [ "`cd \"$(DESTDIR)${archlibdir}\" && /bin/pwd`" != "`/bin/pwd`" ]; then \
          for file in ${UTILITIES}; do \
-           $(INSTALL_PROGRAM) $(INSTALL_STRIP) $$file $(DESTDIR)${archlibdir}/$$file ; \
+           $(INSTALL_PROGRAM) $(INSTALL_STRIP) $$file "$(DESTDIR)${archlibdir}/$$file" ; \
          done ; \
         fi
 
@@ -154,17 +154,17 @@ $(DESTDIR)${archlibdir}: all
 install: $(DESTDIR)${archlibdir}
        @echo
        @echo "Installing utilities for users to run."
-       umask 022; ${MKDIR_P} $(DESTDIR)${bindir}
+       umask 022; ${MKDIR_P} "$(DESTDIR)${bindir}"
        for file in ${INSTALLABLES} ; do \
-         $(INSTALL_PROGRAM) $(INSTALL_STRIP) $${file} $(DESTDIR)${bindir}/`echo $${file} | sed -e 's/${EXEEXT}$$//' -e '$(TRANSFORM)'`${EXEEXT} ; \
+         $(INSTALL_PROGRAM) $(INSTALL_STRIP) $${file} "$(DESTDIR)${bindir}"/`echo $${file} | sed -e 's/${EXEEXT}$$//' -e '$(TRANSFORM)'`${EXEEXT} ; \
        done
 
 uninstall:
        for file in ${INSTALLABLES}; do \
-         rm -f $(DESTDIR)${bindir}/`echo $${file} | sed -e 's/${EXEEXT}$$//' -e '$(TRANSFORM)'`${EXEEXT} ; \
+         rm -f "$(DESTDIR)${bindir}"/`echo $${file} | sed -e 's/${EXEEXT}$$//' -e '$(TRANSFORM)'`${EXEEXT} ; \
        done
-       if [ -d $(DESTDIR)${archlibdir} ]; then \
-         (cd $(DESTDIR)${archlibdir} && rm -f ${UTILITIES}) \
+       if [ -d "$(DESTDIR)${archlibdir}" ]; then \
+         (cd "$(DESTDIR)${archlibdir}" && rm -f ${UTILITIES}) \
        fi
 
 mostlyclean: