[Kde-bindings] Re: Building on Mac OS X

[Arno Rehn]

On Sunday 06 March 2011 01:26:16 Elliott Slaughter wrote:
> On Sat, Mar 5, 2011 at 3:59 PM, Arno Rehn <arno at arnorehn.de> wrote:
> > On Sunday 06 March 2011 00:54:17 Elliott Slaughter wrote:
> > > On Sat, Mar 5, 2011 at 3:37 PM, Arno Rehn <arno at arnorehn.de> wrote:
> > > > On Sunday 06 March 2011 00:29:15 Elliott Slaughter wrote:
> > > > > On Sat, Mar 5, 2011 at 3:14 PM, Arno Rehn <arno at arnorehn.de> wrote:
> > > > > > On Sunday 06 March 2011 00:03:36 Elliott Slaughter wrote:
> > > > > > > On Sat, Mar 5, 2011 at 2:51 PM, Arno Rehn <arno at arnorehn.de>
> > 
> > wrote:
> > > > > > > > On Saturday 05 March 2011 22:46:43 Cyrus Harmon wrote:
> > > > > > > > > is required to get past Elliott's latest bug. Now, I get
> > > > > > > > > _almost_ to the end:
> > > > > > > > > 
> > > > > > > > > [100%] Building CXX object
> > > > > > 
> > > > > > phonon/CMakeFiles/smokephonon.dir/x_10.cpp.o
> > > > > > 
> > > > > > > > > /Users/sly/src/kde/smokeqt/phonon/x_10.cpp: In static
> > > > > > > > > member function ‘static void
> > > > > > > > > __smokephonon::x_QGlobalSpace::x_1(Smoke::StackItem*)’:
> > > > > > 
> > > > > > > > > /Users/sly/src/kde/smokeqt/phonon/x_10.cpp:18: error:
> > > > > > ‘Q_COMPLEX_TYPE’
> > > > > > 
> > > > > > > > was
> > > > > > > > 
> > > > > > > > > not declared in this scope
> > > > > > 
> > > > > > /Users/sly/src/kde/smokeqt/phonon/x_10.cpp:
> > > > > > > > > In static member function ‘static void
> > > > > > > > > __smokephonon::x_QGlobalSpace::x_2(Smoke::StackItem*)’:
> > > > > > > > > /Users/sly/src/kde/smokeqt/phonon/x_10.cpp:21: error:
> > > > > > > > > ‘Q_PRIMITIVE_TYPE’ was not declared in this scope ...
> > > > > > > > 
> > > > > > > > Does this only happen with phonon? These constants are all
> > > > > > > > unconditionally declared in qglobal.h, so this shouldn't
> > 
> > happen.
> > 
> > > > Can
> > > > 
> > > > > > you
> > > > > > 
> > > > > > > > post the complete error message?
> > > > > > > 
> > > > > > > Thanks for pushing :-).
> > > > > > > 
> > > > > > > I can duplicate this error. It's long so I put it in a paste.
> > > > > > > 
> > > > > > > Error: http://pastebin.com/spA85f1F
> > > > > > > x_10.cpp: http://pastebin.com/sDaXpTjk
> > > > > > 
> > > > > > Right, I think this should be fixed now.
> > > > > 
> > > > > I'm still getting the same error. New pastes are attached (although
> > > > > they look the same to me as the originals).
> > > > > 
> > > > > Error: http://pastebin.com/MCZD1fQp
> > > > > x_10.cpp: http://pastebin.com/bSPNuZUG
> > > > 
> > > > Strange. Can you try adding #include <QtCore/qglobal.h> at the
> > 
> > beginning
> > 
> > > > and/or end of <src>/phonon/phonon_includes.h? Something's going
> > > > really wrong
> > > > here as it seems.
> > > 
> > > Somehow that didn't fix it. Here's the patch I used.
> > > 
> > > diff --git a/phonon/phonon_includes.h b/phonon/phonon_includes.h
> > > index acf03d6..a3fe857 100644
> > > --- a/phonon/phonon_includes.h
> > > +++ b/phonon/phonon_includes.h
> > > @@ -38,3 +38,4 @@
> > > 
> > >  #include <phonon/volumeslider.h>
> > >  
> > >  #include <QtCore/qurl.h>
> > > 
> > > +#include <QtCore/qglobal.h>
> > 
> > That's fine. I didn't really expect it to be fixed by this... try to
> > build with 'make VERBOSE=1' and post the output. I'm really interested
> > in what it's
> > doing there..
> 
> Here you go: http://pastebin.com/xzpspSCn
> 
> I think you're most interested in the part starting at line 698.
> 
> By the way, I did a git checkout HEAD phonon/phonon_includes.h before doing
> this, so this is without the qglobal include.
Hm, this doesn't make any sense to me. There is
-I/Library/Frameworks/QtCore.framework/Headers
on the command line, so we can be sure we have the qglobal.h header included. 
I don't have the faintest clue why it's not picking up the constants.
You could run line 699 manually with 'g++ -E' to see the preprocessed contents 
only and guess where the constants have gone. (Or nopaste that again ;)).
For the time being, you could also simply skip phonon by passing
-DDISABLE_Phonon=TRUE
to cmake. But that's not really a solution.

-- 
Arno Rehn
arno at arnorehn.de