From: jb <jb...@kd...> - 2008-07-06 22:14:40
|
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 |