[PATCH] KDE/kdelibs/cmake/modules/KDE4Macros.cmake

Alexander Neundorf neundorf at kde.org
Thu Oct 25 14:30:22 BST 2007


On Thursday 25 October 2007, Charles Samuels wrote:
> You wrote, on Tuesday 2007 October 23:
> > Which files were the problem ?
>
> Some C library that uses an automatic generator of some sort. the actual .h
> file isn't listed anywhere.
>
> Sorry I can't remember.

Hmm, if it's a generated file it should actually be caught with the test for 
the GENERATED property. 

> > Maybe it would be better to check for (.C|.cpp|.cxx|.cc) and only use
> > these.
>
> sounds reasonable.

Here's a patch which changes the test so that now only C++ file are included 
in the final file (extensions cpp|cc|cxx|C).

I'm right now in the process of setting up my machine again from scratch I 
don't have a working kdelibs build right now here, so it would be nice if 
somebody could verify that enable-final works with this patch.

Alex



-------------- next part --------------
A non-text attachment was scrubbed...
Name: KDE4Macros.cmake.diff
Type: text/x-diff
Size: 1215 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20071025/c89c6a93/attachment.diff>


More information about the kde-core-devel mailing list