[Kdenlive-devel] Build error

[Jared Henley]

On Tue, May 6, 2008 at 5:57 PM, bridd <bridd at bridd.com> wrote:

> On Tue, 2008-05-06 at 15:15 +1000, Jared Henley wrote:
> > rather I'm building an ArchLinux package, basing my PKGBUILD file on
> > one I found on the ArchLinux site for kdenlive 0.6-svn (svn number
> > 1727).
>
>
> > After 41% i get a big slab of warnings, and this error:
> >
> > /home/j/dev/pkgs/kdenlive/src/kdenlive/kdenlive/effect.h: At global
> > scope:
> > /home/j/dev/pkgs/kdenlive/src/kdenlive/kdenlive/effect.h:39: error:
> > multiple parameters named 'id'
> >
> > This doesn't look like a dependency problem to me.  Should I be using
> > a specific version of gcc or some other tool?  Or can I expect that a
> > lot of svn revisions won't build?  Because I haven't yet found a
> > subversion revision that actually builds.
>
> I've been trying this myself, under fully up to date ArchLinux, except
> rather than going for building an ArchLinux package I've just been
> trying to compile it and install.  I get exactly the same error.
>
> I've had a few things break on me building from source due to Arch
> running GCC 4.3 and it being strict about the way C libraries are
> declared in scope in C++ files, and usually they're easy enough to patch
> up, move on.
>
> Anyone else have any experience with running Kdenlive under Arch or
> compiled with GCC 4.3 ?
>

I figured, given the lack of response, I'd better try to solve this myself.
I think you're right about the C library declarations in C++.  Here's a
patch which I've just used against revision 2167 to build an arch package
(on a pretty up-to-date arch installation)

Index: kdenlive/kdenlive.cpp
===================================================================
--- kdenlive/kdenlive.cpp    (revision 2167)
+++ kdenlive/kdenlive.cpp    (working copy)
@@ -17,6 +17,9 @@

 #define _ISOC99_SOURCE

+// Include file for exit()
+#include <stdlib.h>
+
 // include files for QT
 #include <qdir.h>
 #include <qprinter.h>
Index: kdenlive/effect.h
===================================================================
--- kdenlive/effect.h    (revision 2167)
+++ kdenlive/effect.h    (working copy)
@@ -36,7 +36,7 @@

 class Effect {
   public:
-    Effect(const EffectDesc & desc, const QString & id, const QString & id
= NULL);
+    Effect(const EffectDesc & desc, const QString & id, const QString &
group = NULL);

     ~Effect();

Index: kdenlive/ktrackview.cpp
===================================================================
--- kdenlive/ktrackview.cpp    (revision 2167)
+++ kdenlive/ktrackview.cpp    (working copy)
@@ -15,6 +15,9 @@
  *
*

***************************************************************************/

+// For abs()
+#include <stdlib.h>
+
 #include <iostream>
 #include <math.h>

Index: kdenlive/krender.cpp
===================================================================
--- kdenlive/krender.cpp    (revision 2167)
+++ kdenlive/krender.cpp    (working copy)
@@ -22,6 +22,9 @@
  *
*

***************************************************************************/

+// for exit()
+#include <stdlib.h>
+
 // ffmpeg Header files

 extern "C" {
Index: kdenlive/kruler.cpp
===================================================================
--- kdenlive/kruler.cpp    (revision 2167)
+++ kdenlive/kruler.cpp    (working copy)
@@ -15,6 +15,9 @@
  *
*

***************************************************************************/

+// For abs()
+#include <stdlib.h>
+
 #include <qpainter.h>
 #include <qcolor.h>
 #include <qtextstream.h>

Jared.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdenlive/attachments/20080508/1a9ae5f6/attachment.html>