Guard against possible accidental matches with parent dirs in Makefiles
authorGlenn Morris <rgm@gnu.org>
Wed, 6 Nov 2013 17:56:48 +0000 (12:56 -0500)
committerGlenn Morris <rgm@gnu.org>
Wed, 6 Nov 2013 17:56:48 +0000 (12:56 -0500)
* lisp/Makefile.in (setwins_almost, setwins_for_subdirs):
Avoid accidental matches.

* test/automated/Makefile.in (setwins): Avoid accidental matches.

lisp/ChangeLog
lisp/Makefile.in
test/ChangeLog
test/automated/Makefile.in

index 619bca2..be01a0c 100644 (file)
@@ -1,3 +1,8 @@
+2013-11-06  Glenn Morris  <rgm@gnu.org>
+
+       * Makefile.in (setwins_almost, setwins_for_subdirs):
+       Avoid accidental matches.
+
 2013-11-06  Stefan Monnier  <monnier@iro.umontreal.ca>
 
        * menu-bar.el (popup-menu): Use key-binding.
index 9dd5a40..d00330c 100644 (file)
@@ -117,14 +117,14 @@ setwins=for file in `find . -type d -print`; do \
 
 # Find all subdirectories except `obsolete' and `term'.
 setwins_almost=for file in `find ${srcdir} -type d -print`; do \
-          case $$file in */obsolete | */term ) ;; \
+          case $$file in ${srcdir}*/obsolete | ${srcdir}*/term ) ;; \
                *) wins="$$wins$${wins:+ }$$file" ;; \
           esac; \
         done
 
 # Find all subdirectories in which we might want to create subdirs.el.
 setwins_for_subdirs=for file in `find ${srcdir} -type d -print`; do \
-          case $$file in */cedet* ) ;; \
+          case $$file in ${srcdir}*/cedet* ) ;; \
                *) wins="$$wins$${wins:+ }$$file" ;; \
           esac; \
         done
index e032af4..281ce98 100644 (file)
@@ -1,3 +1,7 @@
+2013-11-06  Glenn Morris  <rgm@gnu.org>
+
+       * automated/Makefile.in (setwins): Avoid accidental matches.
+
 2013-11-06  Michael Albinus  <michael.albinus@gmx.de>
 
        * automated/tramp-tests.el (tramp-test07-file-exists-p):
index 63cffd9..6177689 100644 (file)
@@ -39,9 +39,8 @@ emacs = unset EMACSLOADPATH; \
        LC_ALL=C EMACS_TEST_DIRECTORY=$(srcdir) "$(EMACS)" $(EMACSOPT)
 
 # Common command to find subdirectories
-setwins=subdirs=`find $(srcdir) -type d -print`; \
-       for file in $$subdirs; do \
-          case $$file in */data* | */flymake* ) ;; \
+setwins=for file in `find $(srcdir) -type d -print`; do \
+          case $$file in $(srcdir)*/data* | $(srcdir)*/flymake* ) ;; \
                *) wins="$$wins$${wins:+ }$$file" ;; \
           esac; \
         done