Akonadi move next monday.

[Kevin Krammer]

Hi Alex,

On Friday 25 April 2008, Alexander Neundorf wrote:

> From a very quick look:
> it seems you're using automoc, I guess that's the reason you have the
> KDE4Macros.cmake, kde4_exec_via_sh.cmake and kde4automoc.files.in there.
> I'm about to move automoc from kdelibs to kdesupport, once this is done you
> can then get rid of these files.

Excellent!

> Do you really need MacroAdditionalCleanFiles.cmake ? It shouldn't be
> necessary.

Probably not.
Since the CMakeLists.txt were used in a KDE module (kdepim) before, quite some 
dependencies on kdelibs build magic had crept in and since we had to do the 
move in a short timeframe to just copy files until it built again.

> Do you need the custom copy of FindQt4.cmake ? cmake 2.4.5 comes with a
> FindQt4.cmake, which may be good enough for akonadi. Just try it and see if
> it works for you.

Allen, Jaroslav and Will started to do some of the build system cleanup, but 
it is likely that there are more things that can be removed or replaced with 
standard CMake features.
This is why I asked on k-d-c that build system gurus would have a look as 
well, since you probably can identify unnecessary duplications we might not.

Btw, one of our remaining problems is how to find D-Bus XML introspection 
files installed by Akonadi (from kdesupport) when we need them in 
kdepimlibs/akonadi. The FindAkonadi macro correctly finds libs and includes.

I saw that CMakeLists.txt in kdelibs generates a KDELibsDependencies.cmake 
file at runtime. Is there some kind of CMake magic involved for finding/using 
such files? (assuming this would allow us to "export" the Akonadi D-Bus 
interface path)

Cheers,
Kevin

-- 
Kevin Krammer, KDE developer, xdg-utils developer
KDE user support, developer mentoring
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080425/72470231/attachment.sig>