From: Ramkumar Ramachandra <artagnon@gmail.com>
Date: Tue, 18 Jun 2013 20:35:06 GMT
Subject: Re: [PATCH] t/t5800-remote-testpy: skip all tests if not built
Message-ID: <CALkWK0=6ot9ZXEyMmO1ZtEXPi5H1JXXvG5j6yvBeE56H9zQBwg@mail.gmail.com>
In-Reply-To: <7vip1b40zb.fsf@alter.siamese.dyndns.org>

Junio C Hamano wrote:
> Sounds like making "make test" build it is a more correct approach,
> at least to me.  What am I missing?

How exactly?  I'm not exactly competent in make, but this is what I
understood from what you said (and it's obviously wrong):

diff --git a/Makefile b/Makefile
index 03524d0..da91937 100644
--- a/Makefile
+++ b/Makefile
@@ -580,6 +580,7 @@ TEST_PROGRAMS_NEED_X += test-svn-fe
 TEST_PROGRAMS_NEED_X += test-wildmatch

 TEST_PROGRAMS = $(patsubst %,%$X,$(TEST_PROGRAMS_NEED_X))
+TEST_PROGRAMS += git-remote-testpy

 # List built-in command $C whose implementation cmd_$C() is not in
 # builtin/$C.o but is linked in as part of some other command.