On Thu, Nov 02, 2017 at 11:45:20 -0400, Bjorn Roche wrote: > > - I don’t know if/how to update the FATE tests. > > Can anyone comment on this? Do I update the tests in the same patch or > separate that?
Your patch is not allowed to break fate. If the fate results are correct references, then fate is catching an error in your patch which you need to correct. If the change in fate is expected (implied by the patch), then you need to change the fate results or adapt the fate test within the same patch. New additional fate tests are welcome (or preferred? not sure) as separate patches. How to update fate: There's a web (or wiki) page about that, and James wrote this excellent stuff recently (perhaps not related to your quest here): http://ffmpeg.org/pipermail/ffmpeg-devel/2017-October/217473.html Moritz _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
