[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