Opened 3 years ago

Last modified 3 years ago

#4455 open defect

Better dependency tracking for ffprobe FATE tests needed

Reported by: Timothy_Gu Owned by:
Priority: normal Component: undetermined
Version: git-master Keywords: fate
Cc: Blocked By:
Blocking: Reproduced by developer: yes
Analyzed by developer: no


Right now, ffprobe tests uses tests/data/ffprobe-test.nut, which depends on ffmpeg$(EXESUF). However, ffprobe tests are not disabled with ffmpeg is disabled, like when --disable-avdevice is passed. This leads to errors when testing ffprobe.

There are two variants:

GEN	tests/data/ffprobe-test.nut
/home/ux/fate/ffmpeg/tests/Makefile:47: recipe for target 'tests/data/ffprobe-test.nut' failed
make: *** [tests/data/ffprobe-test.nut] Error 1

make: *** No rule to make target 'ffmpeg', needed by 'tests/data/ffprobe-test.nut'.

Attachments (1)

patchfate.diff (953 bytes) - added by cehoyos 3 years ago.

Download all attachments as: .zip

Change History (3)

comment:1 Changed 3 years ago by cehoyos

  • Reproduced by developer set
  • Status changed from new to open
  • Version changed from unspecified to git-master

FFprobe tests are disabled if ffmpeg is disabled, ffmpeg does not depend on avdevice.
The FFprobe tests depend on a file that can currently only be created if avdevice was enabled, I will attach a patch that works around this issue by not using avdevice when creating the file. Unfortunately, it slightly changes the output file because the audio stream gets a default stream (this may or may not be related to ticket #3622).
A solution would be to first change the test file so the audio stream will be set to default and then apply my patch.

Last edited 3 years ago by cehoyos (previous) (diff)

Changed 3 years ago by cehoyos

comment:2 Changed 3 years ago by cehoyos

Fixed an error in my first comment and sent a patch to change the test file so the audio stream gets disposition default.

Note: See TracTickets for help on using tickets.