[FFmpeg-devel] [PATCH] [4/??] [1/1] Libavfilter - Build changes

[Vitor Sessak]

Vitor Sessak wrote:
> Hi
> 
> M?ns Rullg?rd wrote:
>> Vitor Sessak <vitor1001 at gmail.com> writes:
>>> M?ns Rullg?rd wrote:
>>>> Vitor Sessak <vitor1001 at gmail.com> writes:
>>>>>
>>>>> This patch is unrelated to the graph patch (in the sense that it can
>>>>> be applied before or after it). It allows to compile libavfilter with
>>>>> "configure --enable-avfilter", but it will not be used at this point
>>>>> for anything.
>>>>>
>>>>> (no need to hurry in reviewing this, I'll be without internet until
>>>>> next week)
>>>>>
>>>>> Index: configure
>>>>> ===================================================================
>>>>> --- configure    (revision 11882)
>>>>> +++ configure    (working copy)
>>>>> @@ -72,6 +72,8 @@
>>>>>    echo "                           and ffmpeg will be 
>>>>> unredistributable [default=no]"
>>>>>    echo "  --enable-pp              enable GPLed postprocessing 
>>>>> support [default=no]"
>>>>>    echo "  --enable-swscaler        software scaler support 
>>>>> [default=no]"
>>>>> +  echo "  --enable-avfilter        video filter support (replaces 
>>>>> vhook) [default=no]"
>>>>> +  echo "  --enable-avfilter-lavf   video filters dependant on 
>>>>> avformat [default=no]"
>>>> Why this distinction?
>>> Because there is no point of having such a big dependence for just a 
>>> few filters (like the vsrc_movie). Think about someone who wants just 
>>> to filter video, but don't cares about encoding/decoding.
>>
>> So why not simply make the relevant filters depend on libavformat
>> directly?
> 
> You mean compile it, unless configure is called with --disable-avformat? 
> Sounds reasonable to me. V?ctor, Diego?

Sorry to reply to self, but I was not clear. I meant to ask your opinion 
(but if you want to fix it you are welcome too!).

-Vitor