From 4385a1ce8f5359418c7401805331af3818913040 Mon Sep 17 00:00:00 2001 From: George Boyle Date: Fri, 3 Jul 2015 00:38:26 +0100 Subject: [PATCH] fate/api-tests: Tests that need samples should only run if SAMPLES is set This change fixes a bug where a test that required a sample was being included in the suite when SAMPLES was not set. It also improves the consistency of variable names relating to the API tests. Signed-off-by: Michael Niedermayer --- tests/fate/api.mak | 18 +++++++++++++----- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/tests/fate/api.mak b/tests/fate/api.mak index da5003646b..4a1ee814a0 100644 --- a/tests/fate/api.mak +++ b/tests/fate/api.mak @@ -4,12 +4,20 @@ fate-api-flac: CMD = run $(APITESTSDIR)/api-flac-test fate-api-flac: CMP = null fate-api-flac: REF = /dev/null -FATE_API_SAMPLES-LIBAVFORMAT-$(call DEMDEC, H264, H264) += fate-api-h264 +FATE_API_SAMPLES_LIBAVFORMAT-$(call DEMDEC, H264, H264) += fate-api-h264 fate-api-h264: $(APITESTSDIR)/api-h264-test$(EXESUF) fate-api-h264: CMD = run $(APITESTSDIR)/api-h264-test $(TARGET_SAMPLES)/h264-conformance/SVA_NL2_E.264 -FATE-$(CONFIG_AVCODEC) += $(FATE_API_LIBAVCODEC-yes) -FATE_API_SAMPLES-$(CONFIG_AVFORMAT) += $(FATE_API_SAMPLES-LIBAVFORMAT-yes) -FATE_SAMPLES_AVCONV += $(FATE_API_SAMPLES-yes) +FATE_API_SAMPLES-$(CONFIG_AVFORMAT) += $(FATE_API_SAMPLES_LIBAVFORMAT-yes) -fate-api: $(FATE_API_LIBAVCODEC-yes) $(FATE_API_SAMPLES-yes) +ifdef SAMPLES + FATE_API_SAMPLES += $(FATE_API_SAMPLES-yes) +endif + +FATE_API-$(CONFIG_AVCODEC) += $(FATE_API_LIBAVCODEC-yes) +FATE_API-$(CONFIG_AVFORMAT) += $(FATE_API_LIBAVFORMAT-yes) +FATE_API = $(FATE_API-yes) + +FATE-yes += $(FATE_API) $(FATE_API_SAMPLES) + +fate-api: $(FATE_API) $(FATE_API_SAMPLES)