[Kdenlive-devel] FFMPEG include path and some changes

[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.

regards
jb