[PATCH] akonadi can't be built in kdesupport

[Matthew Woehlke]

Christophe Giboudeaux wrote:
> On Saturday 13 June 2009 00:49:57 Matthew Woehlke wrote:
>> Slightly over-dramatic, but...
>>
>> I've gotten tired of having to bootstrap kdesupport because akonadi
>> thinks it doesn't exist inside of there. I don't think this used to
>> happen, but for some time it has been impossible to build (or even
>> configure!) kdesupport after nuking the KDE install directory (which I
>> have been doing often due to katepart continually breaking).
>>
>> Patch below fixes it. I will commit this in 1-2 weeks if I don't hear
>> otherwise. (But please let me know if I can commit sooner!)
>>
>> Index: akonadi/cmake/modules/FindSoprano.cmake
> 
> If Soprano wasn't found, we're already checking if we're currently building 
> kdesupport or just kdesupport/akonadi.

Hmm... well cmake is failing without the change, due to the include 
directory (or the header, at least) being missing when it goes to look 
for soprano's version.

It may be that it's finding soprano in kdesupport, but because version.h 
doesn't exist, the version check breaks. Maybe it is enough to skip the 
version check when building in kdesupport, but that still requires 
FindSoprano to be kdesupport-aware. In which case, I'm inclined to 
prefer the patch as posted.

> Please post the CMake log to see what happens.

I'll take another look, but I don't know if I am able to reproduce 
without nuking $KDEDIR again.

But I can tell you what happens. FindSoprano tries to read 
${SOPRANO_INCLUDE}/soprano/version.h, which doesn't exist. It then has 
*a cmake error* in the next (REGEX MATCH) line. This isn't a "couldn't 
find package", this is a "your buildsystem has a syntax error".

-- 
Matthew
Please do not quote my e-mail address unobfuscated in message bodies.
-- 
 >> This is a big deal, because now some tiny minority has lobbied for
 >> changes that end up hurting everyone else.
 > Yeah, I know. Par for the course. I hate the American government.
Actually, I was talking about X.org...