aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlex Merry <alex.merry@kde.org>2014-05-12 23:39:13 +0100
committerAlex Merry <alex.merry@kde.org>2014-05-12 23:49:23 +0100
commitbba8e7415828db03d30d1c5f35894f829fd299d7 (patch)
treed198faaa9f6f3df93fc6dfc068ae85ba974147d0
parent7213137a75e832976c25168ae640d26369c771a7 (diff)
downloadextra-cmake-modules-bba8e7415828db03d30d1c5f35894f829fd299d7.tar.gz
extra-cmake-modules-bba8e7415828db03d30d1c5f35894f829fd299d7.tar.bz2
Check that KDEInstallDirs defines the variables it claims to
-rw-r--r--kde-modules/KDEInstallDirs.cmake2
-rw-r--r--tests/ExecuteKDEModules/CMakeLists.txt54
2 files changed, 55 insertions, 1 deletions
diff --git a/kde-modules/KDEInstallDirs.cmake b/kde-modules/KDEInstallDirs.cmake
index cadbba13..b2312cdd 100644
--- a/kde-modules/KDEInstallDirs.cmake
+++ b/kde-modules/KDEInstallDirs.cmake
@@ -178,7 +178,7 @@ endif()
# directories are reused from the installed kdelibs
macro(_SET_FANCY _var _value _comment)
if (NOT DEFINED ${_var})
- set(${_var} ${_value})
+ set(${_var} "${_value}")
else()
set(${_var} "${${_var}}" CACHE STRING "${_comment}")
endif()
diff --git a/tests/ExecuteKDEModules/CMakeLists.txt b/tests/ExecuteKDEModules/CMakeLists.txt
index cdfd07c4..3a556d15 100644
--- a/tests/ExecuteKDEModules/CMakeLists.txt
+++ b/tests/ExecuteKDEModules/CMakeLists.txt
@@ -16,4 +16,58 @@ foreach(module ${all_kde_modules})
include("${module}")
endforeach()
+set(expected_vars
+ AUTOSTART_INSTALL_DIR
+ BIN_INSTALL_DIR
+ CMAKECONFIG_INSTALL_PREFIX
+ CMAKE_INSTALL_DEFAULT_COMPONENT_NAME
+ CONFIG_INSTALL_DIR
+ DATA_INSTALL_DIR
+ DBUS_INTERFACES_INSTALL_DIR
+ DBUS_SERVICES_INSTALL_DIR
+ DBUS_SYSTEM_SERVICES_INSTALL_DIR
+ EXEC_INSTALL_PREFIX
+ HTML_INSTALL_DIR
+ ICON_INSTALL_DIR
+ IMPORTS_INSTALL_DIR
+ INCLUDE_INSTALL_DIR
+ INSTALL_TARGETS_DEFAULT_ARGS
+ KCFG_INSTALL_DIR
+ KCONF_UPDATE_INSTALL_DIR
+ KF5_DATA_INSTALL_DIR
+ KF5_INCLUDE_INSTALL_DIR
+ KF5_INSTALL_TARGETS_DEFAULT_ARGS
+ KF5_LIBEXEC_INSTALL_DIR
+ KNOTIFYRC_INSTALL_DIR
+ LIBEXEC_INSTALL_DIR
+ LIB_INSTALL_DIR
+ LOCALE_INSTALL_DIR
+ MAN_INSTALL_DIR
+ PLUGIN_INSTALL_DIR
+ QML_INSTALL_DIR
+ QT_PLUGIN_INSTALL_DIR
+ SBIN_INSTALL_DIR
+ SERVICES_INSTALL_DIR
+ SERVICETYPES_INSTALL_DIR
+ SHARE_INSTALL_PREFIX
+ SOUND_INSTALL_DIR
+ SYSCONF_INSTALL_DIR
+ TEMPLATES_INSTALL_DIR
+ WALLPAPER_INSTALL_DIR
+ XDG_APPS_INSTALL_DIR
+ XDG_DIRECTORY_INSTALL_DIR
+ XDG_MIME_INSTALL_DIR
+)
+if(APPLE)
+ list(APPEND expected_vars
+ BUNDLE_INSTALL_DIR
+ )
+endif()
+
+foreach(variable ${expected_vars})
+ if(NOT DEFINED ${variable})
+ message(SEND_ERROR "${variable} is not defined.")
+ endif()
+endforeach()
+
add_executable(dummy main.c)