diff options
| author | Paul Eggert | 2017-03-27 11:22:54 -0700 |
|---|---|---|
| committer | Paul Eggert | 2017-03-27 11:30:08 -0700 |
| commit | 55086ef6517bd7294783d08d2b3cd50c330a9e96 (patch) | |
| tree | 9d06a6909dd620aa6ab26546b5b86c7dd36d9d91 | |
| parent | 8ce827426e5400f2be80ae5d7394b74d8dd90373 (diff) | |
| download | emacs-55086ef6517bd7294783d08d2b3cd50c330a9e96.tar.gz emacs-55086ef6517bd7294783d08d2b3cd50c330a9e96.zip | |
Fix obsolete ‘test/automated’ references
* Makefile.in (mostlyclean, clean, maybeclean_dirs, distclean)
(bootstrap-clean, maintainer-clean):
Clean ‘test’, not ‘test/automated’. Test for existence of
subdirectory only for ‘test’, not for directories that should
always exist.
* admin/MAINTAINERS, etc/TODO, lisp/emacs-lisp/bytecomp.el:
* lisp/emacs-lisp/seq.el, lisp/emacs-lisp/thunk.el:
* lisp/man.el (Man-parse-man-k):
* lisp/url/url-domsuf.el, make-dist:
* test/file-organization.org:
Fix obsolete references to test/automated.
| -rw-r--r-- | Makefile.in | 26 | ||||
| -rw-r--r-- | admin/MAINTAINERS | 12 | ||||
| -rw-r--r-- | etc/TODO | 2 | ||||
| -rw-r--r-- | lisp/emacs-lisp/bytecomp.el | 2 | ||||
| -rw-r--r-- | lisp/emacs-lisp/seq.el | 2 | ||||
| -rw-r--r-- | lisp/emacs-lisp/thunk.el | 2 | ||||
| -rw-r--r-- | lisp/man.el | 5 | ||||
| -rw-r--r-- | lisp/url/url-domsuf.el | 2 | ||||
| -rwxr-xr-x | make-dist | 9 | ||||
| -rw-r--r-- | test/file-organization.org | 6 |
10 files changed, 30 insertions, 38 deletions
diff --git a/Makefile.in b/Makefile.in index cf8ff84212f..06909c34445 100644 --- a/Makefile.in +++ b/Makefile.in | |||
| @@ -821,9 +821,7 @@ mostlyclean_dirs = src oldXMenu lwlib lib lib-src nt doc/emacs doc/misc \ | |||
| 821 | $(foreach dir,$(mostlyclean_dirs),$(eval $(call submake_template,$(dir),mostlyclean))) | 821 | $(foreach dir,$(mostlyclean_dirs),$(eval $(call submake_template,$(dir),mostlyclean))) |
| 822 | 822 | ||
| 823 | mostlyclean: $(mostlyclean_dirs:=_mostlyclean) | 823 | mostlyclean: $(mostlyclean_dirs:=_mostlyclean) |
| 824 | for dir in test/automated; do \ | 824 | [ ! -d test ] || $(MAKE) -C test $@ |
| 825 | [ ! -d $$dir ] || $(MAKE) -C $$dir mostlyclean; \ | ||
| 826 | done | ||
| 827 | 825 | ||
| 828 | ### 'clean' | 826 | ### 'clean' |
| 829 | ### Delete all files from the current directory that are normally | 827 | ### Delete all files from the current directory that are normally |
| @@ -838,9 +836,8 @@ clean_dirs = $(mostlyclean_dirs) nextstep | |||
| 838 | $(foreach dir,$(clean_dirs),$(eval $(call submake_template,$(dir),clean))) | 836 | $(foreach dir,$(clean_dirs),$(eval $(call submake_template,$(dir),clean))) |
| 839 | 837 | ||
| 840 | clean: $(clean_dirs:=_clean) | 838 | clean: $(clean_dirs:=_clean) |
| 841 | for dir in test/automated admin/charsets; do \ | 839 | $(MAKE) -C admin/charsets $@ |
| 842 | [ ! -d $$dir ] || $(MAKE) -C $$dir clean; \ | 840 | [ ! -d test ] || $(MAKE) -C test $@ |
| 843 | done | ||
| 844 | -rm -f *.tmp etc/*.tmp* | 841 | -rm -f *.tmp etc/*.tmp* |
| 845 | -rm -rf info-dir.* | 842 | -rm -rf info-dir.* |
| 846 | 843 | ||
| @@ -864,12 +861,13 @@ distclean_dirs = $(clean_dirs) leim lisp | |||
| 864 | 861 | ||
| 865 | $(foreach dir,$(distclean_dirs),$(eval $(call submake_template,$(dir),distclean))) | 862 | $(foreach dir,$(distclean_dirs),$(eval $(call submake_template,$(dir),distclean))) |
| 866 | 863 | ||
| 867 | maybeclean_dirs = test/automated admin/grammars admin/unidata admin/charsets | 864 | maybeclean_dirs = test admin/grammars admin/unidata admin/charsets |
| 868 | 865 | ||
| 869 | distclean: $(distclean_dirs:=_distclean) | 866 | distclean: $(distclean_dirs:=_distclean) |
| 870 | for dir in ${maybeclean_dirs}; do \ | 867 | for dir in $(filter-out test,$(maybeclean_dirs)); do \ |
| 871 | [ ! -d $$dir ] || $(MAKE) -C $$dir distclean; \ | 868 | $(MAKE) -C $$dir $@ || exit; \ |
| 872 | done | 869 | done |
| 870 | [ ! -d test ] || $(MAKE) -C test $@ | ||
| 873 | ${top_distclean} | 871 | ${top_distclean} |
| 874 | 872 | ||
| 875 | ### 'bootstrap-clean' | 873 | ### 'bootstrap-clean' |
| @@ -878,9 +876,10 @@ distclean: $(distclean_dirs:=_distclean) | |||
| 878 | $(foreach dir,$(distclean_dirs),$(eval $(call submake_template,$(dir),bootstrap-clean))) | 876 | $(foreach dir,$(distclean_dirs),$(eval $(call submake_template,$(dir),bootstrap-clean))) |
| 879 | 877 | ||
| 880 | bootstrap-clean: $(distclean_dirs:=_bootstrap-clean) | 878 | bootstrap-clean: $(distclean_dirs:=_bootstrap-clean) |
| 881 | for dir in ${maybeclean_dirs}; do \ | 879 | for dir in $(filter-out test,$(maybeclean_dirs)); do \ |
| 882 | [ ! -d $$dir ] || $(MAKE) -C $$dir bootstrap-clean; \ | 880 | $(MAKE) -C $$dir $@ || exit; \ |
| 883 | done | 881 | done |
| 882 | [ ! -d test ] || $(MAKE) -C test $@ | ||
| 884 | [ ! -f config.log ] || mv -f config.log config.log~ | 883 | [ ! -f config.log ] || mv -f config.log config.log~ |
| 885 | rm -rf ${srcdir}/info | 884 | rm -rf ${srcdir}/info |
| 886 | rm -f ${srcdir}/etc/refcards/emacsver.tex | 885 | rm -f ${srcdir}/etc/refcards/emacsver.tex |
| @@ -906,9 +905,10 @@ maintainer_clean_dirs = src leim lisp | |||
| 906 | $(foreach dir,$(maintainer_clean_dirs),$(eval $(call submake_template,$(dir),maintainer-clean))) | 905 | $(foreach dir,$(maintainer_clean_dirs),$(eval $(call submake_template,$(dir),maintainer-clean))) |
| 907 | 906 | ||
| 908 | maintainer-clean: bootstrap-clean $(maintainer_clean_dirs:=_maintainer-clean) | 907 | maintainer-clean: bootstrap-clean $(maintainer_clean_dirs:=_maintainer-clean) |
| 909 | for dir in ${maybeclean_dirs}; do \ | 908 | for dir in $(filter-out test,$(maybeclean_dirs)); do \ |
| 910 | [ ! -d $$dir ] || $(MAKE) -C $$dir maintainer-clean; \ | 909 | $(MAKE) -C $$dir $@ || exit; \ |
| 911 | done | 910 | done |
| 911 | [ ! -d test ] || $(MAKE) -C test $@ | ||
| 912 | ${top_maintainer_clean} | 912 | ${top_maintainer_clean} |
| 913 | 913 | ||
| 914 | ### This doesn't actually appear in the coding standards, but Karl | 914 | ### This doesn't actually appear in the coding standards, but Karl |
diff --git a/admin/MAINTAINERS b/admin/MAINTAINERS index 5aff8242887..c13cb552a78 100644 --- a/admin/MAINTAINERS +++ b/admin/MAINTAINERS | |||
| @@ -82,11 +82,11 @@ Simen Heggestøyl | |||
| 82 | 82 | ||
| 83 | Nicolas Petton | 83 | Nicolas Petton |
| 84 | lisp/emacs-lisp/map.el | 84 | lisp/emacs-lisp/map.el |
| 85 | test/automated/map-tests.el | 85 | test/lisp/emac-lisp/map-tests.el |
| 86 | lisp/emacs-lisp/seq.el | 86 | lisp/emacs-lisp/seq.el |
| 87 | test/automated/seq-tests.el | 87 | test/lisp/emac-lisp/seq-tests.el |
| 88 | lisp/emacs-lisp/thunk.el | 88 | lisp/emacs-lisp/thunk.el |
| 89 | test/automated/thunk-tests.el | 89 | test/lisp/emac-lisp/thunk-tests.el |
| 90 | 90 | ||
| 91 | The GNU AUCTeX maintainers (auctex-devel@gnu.org) | 91 | The GNU AUCTeX maintainers (auctex-devel@gnu.org) |
| 92 | RefTeX | 92 | RefTeX |
| @@ -104,7 +104,7 @@ The GNU AUCTeX maintainers (auctex-devel@gnu.org) | |||
| 104 | 104 | ||
| 105 | Dmitry Gutov | 105 | Dmitry Gutov |
| 106 | lisp/progmodes/ruby-mode.el | 106 | lisp/progmodes/ruby-mode.el |
| 107 | test/automated/ruby-mode-tests.el | 107 | test/lisp/progmodes/ruby-mode-tests.el |
| 108 | test/indent/ruby.rb | 108 | test/indent/ruby.rb |
| 109 | lisp/progmodes/xref.el | 109 | lisp/progmodes/xref.el |
| 110 | lisp/progmodes/project.el | 110 | lisp/progmodes/project.el |
| @@ -119,11 +119,11 @@ Ulf Jasper | |||
| 119 | lisp/net/newst-ticker.el | 119 | lisp/net/newst-ticker.el |
| 120 | lisp/net/newst-treeview.el | 120 | lisp/net/newst-treeview.el |
| 121 | lisp/net/newsticker.el | 121 | lisp/net/newsticker.el |
| 122 | test/automated/newsticker-tests.el | 122 | test/lisp/net/newsticker-tests.el |
| 123 | 123 | ||
| 124 | Icalendar | 124 | Icalendar |
| 125 | lisp/calendar/icalendar.el | 125 | lisp/calendar/icalendar.el |
| 126 | test/automated/icalendar-tests.el | 126 | test/lisp/calendar/icalendar-tests.el |
| 127 | 127 | ||
| 128 | ============================================================================== | 128 | ============================================================================== |
| 129 | 2. Areas that someone is willing to maintain, although he would not | 129 | 2. Areas that someone is willing to maintain, although he would not |
| @@ -139,7 +139,7 @@ things in their .emacs. | |||
| 139 | case to make sure it stays fixed. Or pick your favorite programming | 139 | case to make sure it stays fixed. Or pick your favorite programming |
| 140 | major-mode, and write a test for its indentation. Or a version | 140 | major-mode, and write a test for its indentation. Or a version |
| 141 | control backend, and write a test for its status parser. Etc. | 141 | control backend, and write a test for its status parser. Etc. |
| 142 | See test/automated for examples. | 142 | See the 'test' directory for examples. |
| 143 | 143 | ||
| 144 | * Small but important fixes needed in existing features: | 144 | * Small but important fixes needed in existing features: |
| 145 | 145 | ||
diff --git a/lisp/emacs-lisp/bytecomp.el b/lisp/emacs-lisp/bytecomp.el index f45ae359f6c..2c2996ebab4 100644 --- a/lisp/emacs-lisp/bytecomp.el +++ b/lisp/emacs-lisp/bytecomp.el | |||
| @@ -443,7 +443,7 @@ Return the compile-time value of FORM." | |||
| 443 | ;; Macroexpand (not macroexpand-all!) form at toplevel in case it | 443 | ;; Macroexpand (not macroexpand-all!) form at toplevel in case it |
| 444 | ;; expands into a toplevel-equivalent `progn'. See CLHS section | 444 | ;; expands into a toplevel-equivalent `progn'. See CLHS section |
| 445 | ;; 3.2.3.1, "Processing of Top Level Forms". The semantics are very | 445 | ;; 3.2.3.1, "Processing of Top Level Forms". The semantics are very |
| 446 | ;; subtle: see test/automated/bytecomp-tests.el for interesting | 446 | ;; subtle: see test/lisp/emacs-lisp/bytecomp-tests.el for interesting |
| 447 | ;; cases. | 447 | ;; cases. |
| 448 | (setf form (macroexp-macroexpand form byte-compile-macro-environment)) | 448 | (setf form (macroexp-macroexpand form byte-compile-macro-environment)) |
| 449 | (if (eq (car-safe form) 'progn) | 449 | (if (eq (car-safe form) 'progn) |
diff --git a/lisp/emacs-lisp/seq.el b/lisp/emacs-lisp/seq.el index 41187646624..10de2484798 100644 --- a/lisp/emacs-lisp/seq.el +++ b/lisp/emacs-lisp/seq.el | |||
| @@ -52,7 +52,7 @@ | |||
| 52 | ;; - `seq-copy' | 52 | ;; - `seq-copy' |
| 53 | ;; - `seq-into' | 53 | ;; - `seq-into' |
| 54 | ;; | 54 | ;; |
| 55 | ;; All functions are tested in test/automated/seq-tests.el | 55 | ;; All functions are tested in test/lisp/emacs-lisp/seq-tests.el |
| 56 | 56 | ||
| 57 | ;;; Code: | 57 | ;;; Code: |
| 58 | 58 | ||
diff --git a/lisp/emacs-lisp/thunk.el b/lisp/emacs-lisp/thunk.el index be0a90fefde..f4c075d22ce 100644 --- a/lisp/emacs-lisp/thunk.el +++ b/lisp/emacs-lisp/thunk.el | |||
| @@ -42,8 +42,6 @@ | |||
| 42 | ;; | 42 | ;; |
| 43 | ;; (thunk-force delayed) | 43 | ;; (thunk-force delayed) |
| 44 | 44 | ||
| 45 | ;; Tests are located at test/automated/thunk-tests.el | ||
| 46 | |||
| 47 | ;;; Code: | 45 | ;;; Code: |
| 48 | 46 | ||
| 49 | (defmacro thunk-delay (&rest body) | 47 | (defmacro thunk-delay (&rest body) |
diff --git a/lisp/man.el b/lisp/man.el index 4f6e1187e0d..0e1c92956b3 100644 --- a/lisp/man.el +++ b/lisp/man.el | |||
| @@ -832,10 +832,7 @@ indicating optional parts and whitespace being interpreted | |||
| 832 | somewhat loosely. | 832 | somewhat loosely. |
| 833 | 833 | ||
| 834 | foo[, bar [, ...]] [other stuff] (sec) - description | 834 | foo[, bar [, ...]] [other stuff] (sec) - description |
| 835 | foo(sec)[, bar(sec) [, ...]] [other stuff] - description | 835 | foo(sec)[, bar(sec) [, ...]] [other stuff] - description" |
| 836 | |||
| 837 | For more details and some regression tests, please see | ||
| 838 | test/automated/man-tests.el in the emacs repository." | ||
| 839 | (goto-char (point-min)) | 836 | (goto-char (point-min)) |
| 840 | ;; See man-tests for data about which systems use which format (hopefully we | 837 | ;; See man-tests for data about which systems use which format (hopefully we |
| 841 | ;; will be able to simplify the code if/when some of those formats aren't | 838 | ;; will be able to simplify the code if/when some of those formats aren't |
diff --git a/lisp/url/url-domsuf.el b/lisp/url/url-domsuf.el index 25a6a423eb4..b24f24531a6 100644 --- a/lisp/url/url-domsuf.el +++ b/lisp/url/url-domsuf.el | |||
| @@ -89,7 +89,7 @@ | |||
| 89 | 89 | ||
| 90 | ;; Tests: | 90 | ;; Tests: |
| 91 | 91 | ||
| 92 | ;; TODO convert to a proper test/automated test. | 92 | ;; TODO convert to a proper test. |
| 93 | ;; (url-domsuf-cookie-allowed-p "com") => nil | 93 | ;; (url-domsuf-cookie-allowed-p "com") => nil |
| 94 | ;; (url-domsuf-cookie-allowed-p "foo.bar.bd") => t | 94 | ;; (url-domsuf-cookie-allowed-p "foo.bar.bd") => t |
| 95 | ;; (url-domsuf-cookie-allowed-p "bar.bd") => nil | 95 | ;; (url-domsuf-cookie-allowed-p "bar.bd") => nil |
| @@ -538,12 +538,9 @@ done | |||
| 538 | 538 | ||
| 539 | if [ "$with_tests" = "yes" ]; then | 539 | if [ "$with_tests" = "yes" ]; then |
| 540 | echo "Making links to 'test' and its subdirectories" | 540 | echo "Making links to 'test' and its subdirectories" |
| 541 | for f in `find test -type f`; do | 541 | for f in `find test -type f ! -name '*.log' ! -name a.out \ |
| 542 | case $f in | 542 | ! -path test/Makefile |
| 543 | test/automated/*.log) continue ;; | 543 | `; do |
| 544 | test/automated/flymake/warnpred/a.out) continue ;; | ||
| 545 | test/automated/Makefile) continue ;; | ||
| 546 | esac | ||
| 547 | ln $f $tempdir/$f | 544 | ln $f $tempdir/$f |
| 548 | done | 545 | done |
| 549 | fi | 546 | fi |
diff --git a/test/file-organization.org b/test/file-organization.org index 8d9c6716372..4d76c0068e3 100644 --- a/test/file-organization.org +++ b/test/file-organization.org | |||
| @@ -50,9 +50,9 @@ not run by the "make check" command and its derivatives. | |||
| 50 | Resource files for tests (containing test data) should reside in a | 50 | Resource files for tests (containing test data) should reside in a |
| 51 | directory named after the feature with a ~-resources~ suffix, and | 51 | directory named after the feature with a ~-resources~ suffix, and |
| 52 | located in the same directory as the feature. Hence, the lisp file | 52 | located in the same directory as the feature. Hence, the lisp file |
| 53 | ~flymake.el~ should have test files in | 53 | ~lisp/progmodes/flymake.el~, with tests in |
| 54 | ~/test/automated/lisp/progmodes/flymake-tests.el~ should reside in a | 54 | ~test/lisp/progmodes/flymake-tests.el~, should have resources in a |
| 55 | directory called ~/test/automated/lisp/progmodes/flymake-resources~. | 55 | directory called ~test/lisp/progmodes/flymake-resources~. |
| 56 | 56 | ||
| 57 | No guidance is given for the organization of resource files inside the | 57 | No guidance is given for the organization of resource files inside the |
| 58 | ~-resource~ directory; files can be organized at the author's | 58 | ~-resource~ directory; files can be organized at the author's |