On 16/06/14 13:16, Diego Biurrun wrote:
> On Sat, Jun 14, 2014 at 09:54:17PM +0200, Anton Khirnov wrote:
>>
>> --- a/configure
>> +++ b/configure
>> @@ -108,7 +108,7 @@ Program options:
>>    --disable-avconv         disable avconv build
>>    --disable-avplay         disable avplay build
>>    --disable-avprobe        disable avprobe build
>> -  --disable-avserver       disable avserver build
>> +  --disable-avserver       deprecated, does nothing
>>  
>>  Component options:
>>    --disable-doc            do not build documentation
>> @@ -4406,6 +4403,10 @@ enabled scale_filter    && prepend avfilter_deps 
>> "swscale"
>>  
>>  enabled opus_decoder    && prepend avcodec_deps "avresample"
>>  
>> +disabled avserver &&
>> +    warn "avserver has been removed, the --disable-avserver option is"\
>> +         "only provided for compatibility and will be removed in the future"
> 
> You leave avserver as part of PROGRAM_LIST.  I think it's better to remove
> it from there as well and handle --disable-avserver as part of the option
> parsing loop.
> 
> Alternatively, one could just drop --disable-avserver as well.  Since the
> code is gone, it's automatically disabled now.  The question is just if
> we want to keep scripts working or if people should update them.
> 

I'd keep it this way, not that would make much difference beside when
we'd remove it.

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to