[Kde-bindings] KDE/kdebindings/python/pykde4

Michael Jansen kde at michael-jansen.biz
Wed Feb 10 16:40:59 UTC 2010


SVN commit 1088369 by mjansen:

There is additional stuff needed to compile pykde4 correctly.

The KDE_INCLUDES is needed in my case to make sure the correct phonon is
picked up.

I did not copy over the rest from kdebindings/CMakeLists.txt
because it compiles fine for me without it. Is it needed anyway?

+    add_definitions (${QT_DEFINITIONS} ${KDE4_DEFINITIONS})
+    include_directories (${CMAKE_SOURCE_DIR} ${CMAKE_BINARY_DIR} ${KDE4_INCLUDES})

CCMAIL: kde-bindings at kde.org

 M  +11 -4     CMakeLists.txt  


--- trunk/KDE/kdebindings/python/pykde4/CMakeLists.txt #1088368:1088369
@@ -1,10 +1,17 @@
 PROJECT(pykde4)
 
-# search packages used by KDE
-find_package(KDE4 REQUIRED)
-include(KDE4Defaults)
-include(MacroLibrary)
+# If the module is checkedout standalone we have to do some additional stuff
+if(${CMAKE_SOURCE_DIR} STREQUAL ${CMAKE_CURRENT_SOURCE_DIR})
 
+    # search packages used by KDE
+    find_package(KDE4 REQUIRED)
+    include(KDE4Defaults)
+    include(MacroLibrary)
+
+    include_directories (${KDE4_INCLUDES})
+
+endif(${CMAKE_SOURCE_DIR} STREQUAL ${CMAKE_CURRENT_SOURCE_DIR})
+
 # This module can be built independant from the rest of kdebindings.
 SET(CMAKE_MODULE_PATH ${pykde4_SOURCE_DIR}/cmake/modules ${CMAKE_MODULE_PATH})
 



More information about the Kde-bindings mailing list