On 2015-07-07 10:15, Nicolas Morey-Chaisemartin wrote:
> Second iteration.
> 
> I removed the factorization of TESTS_ENVIRONMENT flags because I have not 
> been able to find a proper way
>  to include Makefile.inc between each other due to test/Makefile.inc having 
> automake variable in it.
> 
> This becomes a much simpler pathset as it only add EXEEXT to all the tests 
> TEST_ENVIRONMENTS so that scripts can automatically 
> add the right binary extensions when calling built executables
> 

I haven't looked at the rebased version, but I actually did look at this one 
when you originaly sent it
... but forgot to reply:
Note also that platform specific stuff should not go under linux-generic,
But I feel this EXE extension will be needed by others, so I assume it makes 
sense to have it here.
for the series:

Reviewed-by: Christophe Milard <christophe.mil...@linaro.org>

> Nicolas Morey-Chaisemartin (3):
>   linux-generic: test: add mising $(EXEEXT) for crypto_main test
>   test: add EXEEXT to TESTS_ENVIRONMENT
>   test: suffix binaries with ${EXEEXT} in test scripts
> 
>  platform/linux-generic/test/Makefile.am     | 4 ++--
>  platform/linux-generic/test/pktio/pktio_run | 6 +++---
>  test/performance/Makefile.am                | 2 +-
>  test/performance/odp_l2fwd_run              | 4 ++--
>  test/performance/odp_scheduling_run         | 2 +-
>  5 files changed, 9 insertions(+), 9 deletions(-)
> 
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org
> https://lists.linaro.org/mailman/listinfo/lng-odp
_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to