[FFmpeg-devel] [PATCH] Enable swscale by default

Baptiste Coudurier baptiste.coudurier
Sun Sep 14 00:27:44 CEST 2008


Hi,

Michael Niedermayer wrote:
> On Sat, Sep 13, 2008 at 04:14:08PM +0100, M?ns Rullg?rd wrote:
>> Michael Niedermayer <michaelni at gmx.at> writes:
>>
>>> Hi
>>>
>>> The patch below enables swscale by default.
>>> The intent is to get more exposure, testing and to avoid people wasting time
>>> working on the old scaler.
>>>
>>> Index: configure
>>> ===================================================================
>>> --- configure	(revision 15306)
>>> +++ configure	(working copy)
>>> @@ -73,7 +73,7 @@
>>>    echo "  --enable-nonfree         allow use of nonfree code, the resulting libav*"
>>>    echo "                           and ffmpeg will be unredistributable [default=no]"
>>>    echo "  --enable-postproc        enable GPLed postprocessing support [default=no]"
>>> -  echo "  --enable-swscale         software scaler support [default=no]"
>>> +  echo "  --disable-swscale        disable 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]"
>>>    echo "  --enable-beosthreads     use BeOS threads [default=no]"
>>> @@ -972,6 +972,7 @@
>>>  enable protocols
>>>  enable static
>>>  enable stripping
>>> +enable swscale
>>>  vhook="default"
>> swscale is still marked gpl-only, which means that with this patch, a
>> plain ./configure will fail.  This is IMO unacceptable.  Before making
>> swscale the default, we need to make the gpl parts optional, so it can
>> still be built as lgpl.
> 
> You could have said that before i started to work on droping the old
> scaler. I do not really disagree that a plain ./configure should not fail
> but this leads to a deadlock because
> 1. swscale depends currently on a yuv2rgb table generator
> 2. ive no intent and made it clear in the past that i will not rewrite the
>    gpl yuv2rgb table generator
> 3. noone else works on swscale, ignoring 2-3 small patches

This might have to do with the fact that libswscale codebase is a
_mess_, and I already gave up 2 days ago adding a new pixel format for
16 bit uncompressed quicktime. It is _much_ easier for me to add it to
imgconvert.

IMHO if you want people working on libswscale, it must be cleaned up _a
lot_ before I see this happening.

Im also strongly against a ffmpeg binary defaulting to gpl.

> I thought it was agreed in the droping of the old scaler thread that whoever
> wants a lgpl scaler will have to do the work.

I didn't agree _at all_ with that, and I will vote against.

> And i suspect that noone will do this work when the old scaler is still in
> svn, even less so when it is default.

I don't think this is the main reason, see above.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Smartjog USA Inc.                                http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA




More information about the ffmpeg-devel mailing list