summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStefano Lattarini <stefano.lattarini@gmail.com>2012-09-02 21:55:08 +0200
committerJim Meyering <meyering@redhat.com>2012-09-05 08:48:28 +0200
commit85d0a65ffb86a732b88221eb5a85ac3169f2cddd (patch)
treeeb32ce2679643a8f8dcaf9ab865c64ea4d918206
parente8f9c131baf50f094228d2181c5ac83e6182be5a (diff)
downloadcoreutils-85d0a65ffb86a732b88221eb5a85ac3169f2cddd.tar.gz
tests: remove the unused 'root-hint' target
* tests/Makefile.am (root-hint): Here. The interested user can see the reasons why some tests are skipped by looking at the messages they display on the console; here's an excerpt: ... PASS: misc/id-groups.sh id-setgid.sh: skipped test: must be run as root SKIP: misc/id-setgid.sh PASS: misc/md5sum.pl ... PASS: df/total-verify.sh 2g.sh: skipped test: very expensive: disabled by default SKIP: du/2g.sh ... Clear enough, and more specific and precise that a generic "some tests might need to be run as root" message. And if that user is interested in making those tests run anyway, he'll just take a look to the README files to look for info. So there's no reason to pollute the stdout with another "hint" that is subsumed by those messages, and that might go unnoticed anyway. Moreover, and possibly more importantly, that hint wasn't being displayed anyway, even before this change! That's because the 'root-hint' target was listed as prerequisite for the 'check-recursive' target, which however was not a dependency of the 'check' target in 'tests/Makefile.am', because that file contains no $(SUBDIRS) definition.
-rw-r--r--tests/Makefile.am10
1 files changed, 0 insertions, 10 deletions
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 68ea6b49b..2ef2f10fa 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -162,16 +162,6 @@ all_root_tests = \
check-root:
$(MAKE) check TESTS='$(root_tests)'
-check-recursive: root-hint
-
-# Advertise 'check-root' target.
-.PHONY: root-hint
-root-hint:
- @echo '***********************************************************'
- @echo "NOTICE: Some tests may be run only as root."
- @echo " See the 'Running tests as root' section in README."
- @echo '***********************************************************'
-
EXTRA_DIST += $(all_tests)
# Do not choose a name that is a shell keyword like 'if', or a