diff options
author | Noam Postavsky <npostavs@gmail.com> | 2018-06-06 21:25:52 -0400 |
---|---|---|
committer | Noam Postavsky <npostavs@gmail.com> | 2018-06-12 07:26:06 -0400 |
commit | 5bdc344780faabbc91b7e55306b2071dffb44fa2 (patch) | |
tree | 675856e8329dbd3bb77afd86e166869bbbcc7ce6 /test/Makefile.in | |
parent | b6b793bd77cb8be0a2d2745262e53037dc6798a0 (diff) | |
download | emacs-5bdc344780faabbc91b7e55306b2071dffb44fa2.tar.gz |
; Reduce quoting for SELECTOR in 'make -C test' (Bug#31744)
Before:
make -C test SELECTOR='\"foo\"'
make -C test SELECTOR='(quote (tag :some-tag))'
After:
make -C test SELECTOR='"foo"'
make -C test SELECTOR='(tag :some-tag)'
* test/Makefile.in: Use single quotes around the command line call to
ert, this means the user doesn't have to backslash escape double
quotes when writing lisp strings for the selector. Also wrap the
SELECTOR value in (quote ...) so the user won't have to type it
in (and not get tempted to use the '... reader syntax form which would
now fail to work due to using single quotes around the whole shell
arg).
* test/README: Update instructions accordingly.
Diffstat (limited to 'test/Makefile.in')
-rw-r--r-- | test/Makefile.in | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/test/Makefile.in b/test/Makefile.in index 451513a7474..597ef913113 100644 --- a/test/Makefile.in +++ b/test/Makefile.in @@ -98,9 +98,9 @@ TEST_LOCALE = C TEST_INTERACTIVE ?= no ifeq ($(TEST_INTERACTIVE),yes) -TEST_RUN_ERT = --eval "(ert ${SELECTOR_ACTUAL})" +TEST_RUN_ERT = --eval '(ert (quote ${SELECTOR_ACTUAL}))' else -TEST_RUN_ERT = --batch --eval "(ert-run-tests-batch-and-exit ${SELECTOR_ACTUAL})" ${WRITE_LOG} +TEST_RUN_ERT = --batch --eval '(ert-run-tests-batch-and-exit (quote ${SELECTOR_ACTUAL}))' ${WRITE_LOG} endif # Whether to run tests from .el files in preference to .elc, we do @@ -140,8 +140,8 @@ test_module_dir := $(srcdir)/data/emacs-module all: check -SELECTOR_DEFAULT = (quote (not (or (tag :expensive-test) (tag :unstable)))) -SELECTOR_EXPENSIVE = (quote (not (tag :unstable))) +SELECTOR_DEFAULT = (not (or (tag :expensive-test) (tag :unstable))) +SELECTOR_EXPENSIVE = (not (tag :unstable)) SELECTOR_ALL = t ifdef SELECTOR SELECTOR_ACTUAL=$(SELECTOR) |