[kdelibs/frameworks] tier1/kwindowsystem/src: The config file appears in public headers, so it must be installed.

Stephen Kelly steveire at gmail.com
Thu May 24 21:57:05 UTC 2012


Git commit 39486d4854fabdf5e74a0f7c26404c2a90b46c1d by Stephen Kelly.
Committed on 24/05/2012 at 23:31.
Pushed by skelly into branch 'frameworks'.

The config file appears in public headers, so it must be installed.

Otherwise downstreams can't build.

We probably need a better solution than HAVE_X11 in the config file.
Why not keep the headers not-ifdefed and ifdef the implementation
only?

CCMAIL: kde-frameworks-devel at kde.org

M  +2    -0    tier1/kwindowsystem/src/CMakeLists.txt

http://commits.kde.org/kdelibs/39486d4854fabdf5e74a0f7c26404c2a90b46c1d

diff --git a/tier1/kwindowsystem/src/CMakeLists.txt b/tier1/kwindowsystem/src/CMakeLists.txt
index 11a81ea..b81fa1d 100644
--- a/tier1/kwindowsystem/src/CMakeLists.txt
+++ b/tier1/kwindowsystem/src/CMakeLists.txt
@@ -73,6 +73,8 @@ install( FILES
    netwm_def.h
    netwm.h
 
+   # FIXME: It seems odd to install this.
+   ${CMAKE_CURRENT_BINARY_DIR}/config-kwindowsystem.h
    ${CMAKE_CURRENT_BINARY_DIR}/kwindowsystem_export.h
    DESTINATION ${INCLUDE_INSTALL_DIR} COMPONENT Devel
 )



More information about the Kde-frameworks-devel mailing list