diff options
author | George Boyle <george@thebuds.net> | 2015-07-03 00:38:26 +0100 |
---|---|---|
committer | Michael Niedermayer <michaelni@gmx.at> | 2015-07-04 02:15:16 +0200 |
commit | 4385a1ce8f5359418c7401805331af3818913040 (patch) | |
tree | 7eb12429944190962b055eb548342ecbf406aeab | |
parent | 126271138803fdb1eb6059f4f71c88254ae2a20b (diff) | |
download | ffmpeg-4385a1ce8f5359418c7401805331af3818913040.tar.gz |
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 <michaelni@gmx.at>
-rw-r--r-- | tests/fate/api.mak | 18 |
1 files 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) |