[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Automake-NG] [PATCH] [ng] parallel-tests: use 'am__error' instead o
|
From: |
Akim Demaille |
|
Subject: |
Re: [Automake-NG] [PATCH] [ng] parallel-tests: use 'am__error' instead of hand-rolled equivalent |
|
Date: |
Thu, 7 Jun 2012 11:59:41 +0200 |
Le 6 juin 2012 à 18:40, Stefano Lattarini a écrit :
> @@ -94,8 +87,8 @@ am__tpfx = \
> ## FIXME: this will pick up the default from the environment; are we sure
> ## FIXME: we want that?
> !TEST_EXTENSIONS ?= .test
> -!$(call am__maybe_invalid_test_extensions,\
> -! $(filter-out .%,$(TEST_EXTENSIONS)))
> +!$(foreach am__e,$(filter-out .%,$(TEST_EXTENSIONS)),\
> +! $(call am__error,invalid test extension: '$(am__e)'))
> !$(foreach am__e,$(TEST_EXTENSIONS), \
> ! $(eval $(call am__handle_per_suffix_test,$(am__e))))
> ## It is *imperative* that the "empty" suffix goes last. Otherwise, a
Really, have you considered using a "block" like
approach instead of prefixing with "!"?