Re: [PATCH 00/17] De-recursion for the 'tests' subdirectory.

[Stefano Lattarini]

On 09/04/2012 12:58 PM, Jim Meyering wrote:
> Stefano Lattarini wrote:
> ...
>>> I still haven't reviewed your changes, but in using them I did notice
>>> a missing dependency:
>>>
>>> I ran this command to run the single test:
>>>
>>>     $ make tests/rm/v-slash.log
>>>     PASS: tests/rm/v-slash.log
>>>
>> The above usage is not actually supported Automake interface (for one thing,
>> it won't honour $(check_PROGRAMS), nor $(BUILT_SOURCES)).  To run a subset
>> of tests, you should do this:
>
> Thanks for the explanation, but running the commands you suggest generates
> over 200 hundreds of lines of output, 
>
Even this command?

   $ make check TESTS="tests/rm/v-slash" RECHECK_LOGS=

That would be a bug :-(

> all but one or two of which are
> irrelevant to me.  Running the technically-unsupported command produces
> precisely the one or two lines that I wanted to see.  Which do you think
> I will be using most of the time? ;-)
> 
>>> From 45bf1ab13682f8842b2b9ed81bb9069f6e5b01b3 Mon Sep 17 00:00:00 2001
>>> From: Jim Meyering <address@hidden>
>>> Date: Tue, 4 Sep 2012 12:12:19 +0200
>>> Subject: [PATCH] tests: depend on the programs we're testing
>>>
>>> * tests/local.mk ($(TEST_LOGS)): Depend on $(PROGRAMS),
>>>
>> Beware that $(PROGRAMS) is not a documented Automake interface either.
> 
> If it changes, I'm sure I'll adapt ;-)
>
Perfect, I just want to raise awareness to avoid unpleasant surprises
later :-)

> Unless you can propose an alternative or choose to document it.
>

Regards,
  Stefano