[Kdenlive-devel] FFMPEG include path and some changes

[Marco Gittler]

Am 07.07.2008 um 00:21 schrieb jb:

> On Tuesday 01 July 2008 23:02:57 Lúcio Corrêa wrote:
>> So, why are we using the old one method?  What I suggest:
>>
>> 1. Copy FindFFmpeg.cmake from KDE's trunk into kdenlive;
>> 2. Change renderer.cpp  to #include <libavformat/avformat.h>;
>
> Hi.
>
> The only problem I see is that not everybody has a recent FFmpeg  
> version, and
> this <dir/header.h> stuff did not work on older FFmpeg versions. I  
> don't know
> how most distros manage this, but for example in ubuntu (last  
> release), the
> avformat header is installed in: /usr/include/ffmpeg/avformat.h
> ( see: http://packages.ubuntu.com/hardy/i386/libavformat-dev/ 
> filelist )
>
> So that if we update it, it will not compile on ubuntu, and I am  
> afraid that
> it will be the same for most distros...
>
>> 3. Remove these modules, as they are not used anymore or are part of
>> kdelibs:
>>
>> FindGettext.cmake
>> FindKdeLibs.cmake
> (...)
>
>> I'm proposing this because if we start duplicating the modules, we  
>> end
>> up like the autotools hell, mantaining every module on our own.
>
> It's ok for me if we remove unused or duplicated modules, but I  
> think we have
> to be careful with the FFmpeg detection. The current cmake module  
> seems to
> work for everyone and I would prefer to keep it that way until the  
> major linux
> distributions follow the FFmpeg guidelines.

this i my meaning too, since there are different ffmpeg versions out,  
which are ot compatible in header detection, and so not useable for  
everyone, we should wait until it is.

use cmake modules from kde would be the prefered way, if there is one  
header install scheme.

>
>
> regards
> jb
>

regards marco
>
>
>
>
> -------------------------------------------------------------------------
> Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
> Studies have shown that voting for your favorite open source project,
> along with a healthy diet, reduces your potential for chronic lameness
> and boredom. Vote Now at http://www.sourceforge.net/community/cca08
> _______________________________________________
> Kdenlive-devel mailing list
> Kdenlive-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/kdenlive-devel