>>-FATE_SAMPLES_DEMUX-$(call ALLYES, S337M_DEMUXER DOLBY_E_PARSER 
>>FRAMECRC_MUXER) += fate-s337m-demux
>>+FATE_SAMPLES_DEMUX-$(call FRAMECRC, S337M,, DOLBY_E_PARSER 
>>+FRAMECRC_MUXER) += fate-s337m-demux
>> fate-s337m-demux: CMD = framecrc -i $(TARGET_SAMPLES)/dolby_e/16-11 -c 
>> copy -ss 2 -t 1
>
>Ooops, I failed to remove the redundant FRAMECRC_MUXER here.
>I'll send a v2 early next week including other potential feedbacks if any.
>
>Nicolas

By the way, maybe I should have mentioned it, but I inserted many "call 
FRAMECRC" which may add some unrequired dependencies when streamcopying like 
here.
I think it is acceptable as a comment in the Makefile already states that 
PCM_S16LE_ENCODER and RAWVIDEO_ENCODER may be required although not necessary.
It is some kind of compromise to avoid numerous overlapping macros, and I guess 
anyone wanting a serious fate would have pcm_s16 and rawvideo encoders enabled.

Nicolas
_______________________________________________
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".

Reply via email to