coreutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[coreutils] [PATCH] tests: don't hide all trace of the vc_exe_in_TESTS t


From: Jim Meyering
Subject: [coreutils] [PATCH] tests: don't hide all trace of the vc_exe_in_TESTS test
Date: Wed, 26 Jan 2011 08:41:46 +0100

FYI,

I was tempted to remove this rule altogether, since
test scripts no longer have to be executable.
However, it's not that expensive and does serve to
ensure consistency, and as a useful cross-check.

>From 034e496e983d286011fa5f6058b541b163ba9afd Mon Sep 17 00:00:00 2001
From: Jim Meyering <address@hidden>
Date: Tue, 25 Jan 2011 14:29:07 +0100
Subject: [PATCH] tests: don't hide all trace of the vc_exe_in_TESTS test

There was a non-negligible delay after running a single test.
Now, you'll know why when you see this test's name.
* tests/check.mk (vc_exe_in_TESTS): Don't @-hide commands.
Use $(AM_V_GEN) instead.
---
 tests/check.mk |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/check.mk b/tests/check.mk
index cddfe89..f931806 100644
--- a/tests/check.mk
+++ b/tests/check.mk
@@ -20,8 +20,8 @@
 _v = TESTS
 _w = root_tests
 vc_exe_in_TESTS: Makefile
-       @rm -f t1 t2
-       @if test -d $(top_srcdir)/.git && test $(srcdir) = .; then      \
+       $(AM_V_GEN)rm -f t1 t2;                                         \
+       if test -d $(top_srcdir)/.git && test $(srcdir) = .; then       \
          { sed -n '/^$(_v) =[   ]*\\$$/,/[^\]$$/p'                     \
                $(srcdir)/Makefile.am                                   \
            | sed 's/^  *//;/^\$$.*/d;/^$(_v) =/d';                     \
--
1.7.3.5.38.gb312b



reply via email to

[Prev in Thread] Current Thread [Next in Thread]