Re: [FFmpeg-devel] [PATCH 1/4] fate: Fix dependencies for ffprobe tests with external samples
Andreas Rheinhardt: > Andreas Rheinhardt: >> Up until now, they were appended to the FATE_EXTERN-$(CONFIG_FFMPEG) >> variable and were therefore activated when ffmpeg was enabled regardless >> of whether ffprobe was enabled. >> >> Also the same happened with FATE_SAMPLES_FASTSTART, although the >> corresponding test (mov-faststart-4gb-overflow) only requires external >> samples. >> >> Furthermore, remove the unused FATE_FULL variable (FATE_EXTERN_FFPROBE has >> taken its place). >> >> Signed-off-by: Andreas Rheinhardt >> --- >> tests/Makefile | 6 +++--- >> 1 file changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/tests/Makefile b/tests/Makefile >> index e5f41008d4..477f5bef26 100644 >> --- a/tests/Makefile >> +++ b/tests/Makefile >> @@ -203,8 +203,9 @@ FATE-$(CONFIG_FFPROBE) += $(FATE_FFPROBE) >> >> FATE_SAMPLES_AVCONV += $(FATE_SAMPLES_AVCONV-yes) >> FATE_SAMPLES_FFMPEG += $(FATE_SAMPLES_FFMPEG-yes) >> -FATE_EXTERN-$(CONFIG_FFMPEG) += $(FATE_SAMPLES_AVCONV) >> $(FATE_SAMPLES_FFMPEG) $(FATE_SAMPLES_FFPROBE) $(FATE_SAMPLES_FASTSTART) >> -FATE_EXTERN += $(FATE_EXTERN-yes) >> +FATE_EXTERN-$(CONFIG_FFMPEG) += $(FATE_SAMPLES_AVCONV) >> $(FATE_SAMPLES_FFMPEG) >> +FATE_EXTERN-$(CONFIG_FFPROBE) += $(FATE_SAMPLES_FFPROBE) >> +FATE_EXTERN += $(FATE_EXTERN-yes) >> $(FATE_SAMPLES_FASTSTART) >> >> FATE += $(FATE-yes) >> >> @@ -218,7 +219,6 @@ $(FATE_FFPROBE) $(FATE_SAMPLES_FFPROBE): >> ffprobe$(PROGSSUF)$(EXESUF) >> $(FATE_SAMPLES_FASTSTART): tools/qt-faststart$(EXESUF) >> >> ifdef SAMPLES >> -FATE += $(FATE_FULL) $(FATE_FULL-yes) >> FATE += $(FATE_EXTERN) >> fate-rsync: >> rsync $(RSYNC_OPTIONS) rsync://fate-suite.ffmpeg.org/fate-suite/ >> $(SAMPLES) >> > Will push this set tomorrow if there are no objections. > > - Andreas > Applied. - Andreas ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Re: [FFmpeg-devel] [PATCH 1/4] fate: Fix dependencies for ffprobe tests with external samples
Andreas Rheinhardt: > Up until now, they were appended to the FATE_EXTERN-$(CONFIG_FFMPEG) > variable and were therefore activated when ffmpeg was enabled regardless > of whether ffprobe was enabled. > > Also the same happened with FATE_SAMPLES_FASTSTART, although the > corresponding test (mov-faststart-4gb-overflow) only requires external > samples. > > Furthermore, remove the unused FATE_FULL variable (FATE_EXTERN_FFPROBE has > taken its place). > > Signed-off-by: Andreas Rheinhardt > --- > tests/Makefile | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/tests/Makefile b/tests/Makefile > index e5f41008d4..477f5bef26 100644 > --- a/tests/Makefile > +++ b/tests/Makefile > @@ -203,8 +203,9 @@ FATE-$(CONFIG_FFPROBE) += $(FATE_FFPROBE) > > FATE_SAMPLES_AVCONV += $(FATE_SAMPLES_AVCONV-yes) > FATE_SAMPLES_FFMPEG += $(FATE_SAMPLES_FFMPEG-yes) > -FATE_EXTERN-$(CONFIG_FFMPEG) += $(FATE_SAMPLES_AVCONV) > $(FATE_SAMPLES_FFMPEG) $(FATE_SAMPLES_FFPROBE) $(FATE_SAMPLES_FASTSTART) > -FATE_EXTERN += $(FATE_EXTERN-yes) > +FATE_EXTERN-$(CONFIG_FFMPEG) += $(FATE_SAMPLES_AVCONV) > $(FATE_SAMPLES_FFMPEG) > +FATE_EXTERN-$(CONFIG_FFPROBE) += $(FATE_SAMPLES_FFPROBE) > +FATE_EXTERN += $(FATE_EXTERN-yes) $(FATE_SAMPLES_FASTSTART) > > FATE += $(FATE-yes) > > @@ -218,7 +219,6 @@ $(FATE_FFPROBE) $(FATE_SAMPLES_FFPROBE): > ffprobe$(PROGSSUF)$(EXESUF) > $(FATE_SAMPLES_FASTSTART): tools/qt-faststart$(EXESUF) > > ifdef SAMPLES > -FATE += $(FATE_FULL) $(FATE_FULL-yes) > FATE += $(FATE_EXTERN) > fate-rsync: > rsync $(RSYNC_OPTIONS) rsync://fate-suite.ffmpeg.org/fate-suite/ > $(SAMPLES) > Will push this set tomorrow if there are no objections. - Andreas ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
[FFmpeg-devel] [PATCH 1/4] fate: Fix dependencies for ffprobe tests with external samples
Up until now, they were appended to the FATE_EXTERN-$(CONFIG_FFMPEG) variable and were therefore activated when ffmpeg was enabled regardless of whether ffprobe was enabled. Also the same happened with FATE_SAMPLES_FASTSTART, although the corresponding test (mov-faststart-4gb-overflow) only requires external samples. Furthermore, remove the unused FATE_FULL variable (FATE_EXTERN_FFPROBE has taken its place). Signed-off-by: Andreas Rheinhardt --- tests/Makefile | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/Makefile b/tests/Makefile index e5f41008d4..477f5bef26 100644 --- a/tests/Makefile +++ b/tests/Makefile @@ -203,8 +203,9 @@ FATE-$(CONFIG_FFPROBE) += $(FATE_FFPROBE) FATE_SAMPLES_AVCONV += $(FATE_SAMPLES_AVCONV-yes) FATE_SAMPLES_FFMPEG += $(FATE_SAMPLES_FFMPEG-yes) -FATE_EXTERN-$(CONFIG_FFMPEG) += $(FATE_SAMPLES_AVCONV) $(FATE_SAMPLES_FFMPEG) $(FATE_SAMPLES_FFPROBE) $(FATE_SAMPLES_FASTSTART) -FATE_EXTERN += $(FATE_EXTERN-yes) +FATE_EXTERN-$(CONFIG_FFMPEG) += $(FATE_SAMPLES_AVCONV) $(FATE_SAMPLES_FFMPEG) +FATE_EXTERN-$(CONFIG_FFPROBE) += $(FATE_SAMPLES_FFPROBE) +FATE_EXTERN += $(FATE_EXTERN-yes) $(FATE_SAMPLES_FASTSTART) FATE += $(FATE-yes) @@ -218,7 +219,6 @@ $(FATE_FFPROBE) $(FATE_SAMPLES_FFPROBE): ffprobe$(PROGSSUF)$(EXESUF) $(FATE_SAMPLES_FASTSTART): tools/qt-faststart$(EXESUF) ifdef SAMPLES -FATE += $(FATE_FULL) $(FATE_FULL-yes) FATE += $(FATE_EXTERN) fate-rsync: rsync $(RSYNC_OPTIONS) rsync://fate-suite.ffmpeg.org/fate-suite/ $(SAMPLES) -- 2.20.1 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".