Hi Martin,

Thanks for the information. I've added the includes for the next version.

Cheers,
Joachim



Am Montag, 2. April 2007 12:12 schrieb Martin Michlmayr:
> Package: kdiff3
> Version: 0.9.91-1
> Usertags: ftbfs-gcc-4.3
> Tags: patch
>
> Your package fails to build with GCC 4.3.  Version 4.3 has not been
> released yet but I'm building with a snapshot in order to find errors
> and give people an advance warning.  In GCC 4.3, the C++ header
> dependencies have been cleaned up.  The advantage of this is that
> programs will compile faster.  The downside is that you actually
> need to directly #include everything you use (but you really should
> do this anyway, otherwise your program won't work with any compiler
> other than GCC).  Some background of this can be found at
> http://gcc.gnu.org/PR28080
>
> You can reproduce this problem with gcc-snapshot from unstable.
>
> > Automatic build of kdiff3_0.9.91-1 on em64t by sbuild/amd64 0.53
>
> ...
>
> > difftextwindow.cpp: In member function 'virtual void
> > DiffTextWindow::mouseMoveEvent(QMouseEvent*)': difftextwindow.cpp:536:
> > error: 'abs' was not declared in this scope difftextwindow.cpp:537:
> > error: 'abs' was not declared in this scope difftextwindow.cpp:541:
> > error: 'abs' was not declared in this scope difftextwindow.cpp:542:
> > error: 'abs' was not declared in this scope
>
> --- src/diff.cpp~     2007-04-02 09:57:35.000000000 +0000
> +++ src/diff.cpp      2007-04-02 09:57:41.000000000 +0000
> @@ -15,6 +15,7 @@
>   *                                                                        
> *
> ***************************************************************************
>/
>
> +#include <cstdlib>
>  #include <stdio.h>
>  #include <iostream>
>
> --- src/difftextwindow.cpp~   2007-04-02 09:56:39.000000000 +0000
> +++ src/difftextwindow.cpp    2007-04-02 09:56:48.000000000 +0000
> @@ -18,6 +18,7 @@
>  #include "difftextwindow.h"
>  #include "merger.h"
>  #include <qpainter.h>
> +#include <cstdlib>
>  #include <assert.h>
>  #include <qpixmap.h>
>  #include <qstatusbar.h>
> --- src/fileaccess.cpp~       2007-04-02 10:04:49.000000000 +0000
> +++ src/fileaccess.cpp        2007-04-02 10:04:56.000000000 +0000
> @@ -9,6 +9,7 @@
>  
> ***************************************************************************
>/
>
>  #include "fileaccess.h"
> +#include <cstdlib>
>  #include <iostream>
>  #include <kio/global.h>
>  #include <kmessagebox.h>
>
> Now the package still fails to build because of undefined references,
> but I haven't investigated this yet.  Let's fix the header includes for
> now.


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to