Re: [Automake-NG] [FYI] [ng] check: deps for 'check' target listed in '$(am.test-suite.deps)'

[Stefano Lattarini]

On 07/30/2012 04:57 PM, Akim Demaille wrote:
> 
> Le 30 juil. 2012 à 14:02, Stefano Lattarini a écrit :
> 
>> (generate_makefile): Define '$(am.test-suite.deps)' to the list of
>> dependencies for the 'check' target.
> 
> Define to? or as?
>
As you prefer.  I've always thought 'to' is clear and correct enough in
this situations, but I might be mistaken.

>> diff --git a/lib/am/check-target.am b/lib/am/check-target.am
>> index 225c566..852941b 100644
>> --- a/lib/am/check-target.am
>> +++ b/lib/am/check-target.am
>> @@ -26,7 +26,7 @@ endif
>> # local check dependencies, and finally run the actual tests (as given
>> # by $(TESTS), by DejaGNU, and by the 'check-local' target).
>> check-am: all-am
>> -    $(if %CHECK-DEPS%,$(MAKE) %CHECK-DEPS%,@:)
>> +    $(if  $(am.test-suite.deps),$(MAKE) $(am.test-suite.deps),@:)
> 
> Spurious space?
> 
Yep, fixed now.

Thanks,
  Stefano