On Sunday 22 May 2011 10:51:39 Matthias Fuchs wrote: > Am Sonntag 22 Mai 2011, 00:26:58 schrieb Dawit A: > > On Sat, May 21, 2011 at 5:50 PM, Andreas Pakulat <[email protected]> wrote: > > > On 21.05.11 13:25:32, Dawit A wrote: > > >> No that is a moc issue. The code is properly @ifdef'ed out in the > > >> code, but the generated moc file does not seem to exclude it. That is > > >> why you get the message: > > >> > > >> /home/arysin/kde/build/KDE/kdelibs/kdewebkit/kgraphicswebview.moc:91:6 > > >> : error: prototype for ‘void > > >> KGraphicsWebView::selectionClipboardUrlPasted(const KUrl&)’ does not > > >> match any in class ‘KGraphicsWebView’ > > >> /home/arysin/work/OSS/KDE/kdelibs/kdewebkit/kgraphicswebview.h:146:10: > > >> error: candidate is: void > > >> KGraphicsWebView::selectionClipboardUrlPasted(const KUrl&, const > > >> QString&) > > >> > > >> Dunno why that would be... Perhaps whomever it is that added the > > >> ifndef around that signal could comment. > > > > > > Maybe that changed these days, but moc does not know anything about > > > preprocessor-defines - except for a few hardcoded ones. So if you want > > > moc to ignore some code, you'll possibly have to find another way. > > > > That is why I said whomever added the #ifdef around the signal can > > deal with it. I surely did not put that there since I no need to build > > kdelibs that way. Perhaps there is a trick to work around this > > limitation, since this cannot be the only signal that is deprecated, > > but I do not know what that might be... > > Well in that case I think it would be good to contact the right person, and > that is Kevin Ottens here, so I cced him in that mail.
Well, I proposed a workaround in january already, and somehow it got forgotten along the way, reattaching it here for reference. If that fixes the issue for you guys, I think we really should commit it (although I still didn't find the time to investigate *why* it'd actually fix the issue). Regards. -- Kévin Ottens, http://ervin.ipsquad.net KDAB - proud patron of KDE, http://www.kdab.com
Index: kgraphicswebview.h =================================================================== --- kgraphicswebview.h (revision 1208818) +++ kgraphicswebview.h (working copy) @@ -27,6 +27,7 @@ #define KGRAPHICSWEBVIEW_H #include <kdewebkit_export.h> +#include <kdemacros.h> #include <QtWebKit/QGraphicsWebView> Index: kwebview.h =================================================================== --- kwebview.h (revision 1208818) +++ kwebview.h (working copy) @@ -27,6 +27,7 @@ #define KWEBVIEW_H #include <kdewebkit_export.h> +#include <kdemacros.h> #include <QtWebKit/QWebView>
signature.asc
Description: This is a digitally signed message part.
