[FFmpeg-devel] [PATCH] make img_convert symbol conditional on lavc version, not libswscale

[Baptiste Coudurier]

Hi guys,

Diego Biurrun wrote:
> On Sat, Jul 05, 2008 at 04:11:32PM +0200, Michael Niedermayer wrote:
>> On Sat, Jul 05, 2008 at 03:19:17PM +0200, Diego Biurrun wrote:
>>> On Wed, Jun 25, 2008 at 05:23:47PM +0200, Michael Niedermayer wrote:
>>>> We can leave things as they are or we can bump the major version number and
>>>> remove the old API and other accumulated junk.
>>> I'm all for it, but have you tried compiling FFmpeg after bumping the
>>> libavcodec version number?  Compilation fails when bumping either the
>>> libavcodec or libavformat or libavutil version numbers...
>> Nothing unexpected ...
>> these things will have to be fixed, the only real question is if we want
>> to bump the versions or not.
> 
> I'm all for it :)
> 
>> If we do, then we should first make a little announcement on the list that
>> people now would have 2 weeks time to do ABI/API breaking changes ...
>> In the meantime we (well I, i guess) would fix the compilation issues.
> 
> Let's go for it then.  I'm sure you will get help fixing remaining
> issues...
> 

While I'm all for dropping the old scaler, I'd like to raise 3 issues:

1 - Roundup #504 which is really important IMHO.
2 - Regression tests broken with new scaler.
3 - GPL license

I don't think we can drop the old scaler until at least these issues
(well #3 is not really an issue) are fixed.

What we need IMHO:
- Fix #504.
- Fix regs tests (how is the soc student going ?)
- Make the libswscale compile without --enable-gpl

I think this is really important. Thanks guys.

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