----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128917/ -----------------------------------------------------------
(Updated Sept. 15, 2016, 12:46 p.m.) Status ------ This change has been marked as submitted. Review request for KDE Frameworks and Aleix Pol Gonzalez. Changes ------- Submitted with commit 0cc2de65d111ee1244f3d3ce53c83b3b55b58dfb by Bhushan Shah to branch master. Repository: extra-cmake-modules Description ------- If KDECMakeSettings is included twice for any reasons cmake fails because include guard variable is not scoped to parent, and resulting in being reset when function scope was complete. See following testcase, ``` find_package(ECM REQUIRED NO_MODULE) set(CMAKE_MODULE_PATH ${ECM_MODULE_PATH}) include(KDEInstallDirs) include(KDECMakeSettings) include(KDECMakeSettings) ``` And errors, ``` CMake Error at /opt/kde/share/ECM/kde-modules/KDECMakeSettings.cmake:149 (add_test): add_test given test NAME "appstreamtest" which already exists in this directory. Call Stack (most recent call first): /opt/kde/share/ECM/kde-modules/KDECMakeSettings.cmake:167 (appstreamtest) CMakeLists.txt:6 (include) ``` Diffs ----- kde-modules/KDECMakeSettings.cmake 523271a Diff: https://git.reviewboard.kde.org/r/128917/diff/ Testing ------- see testcase in description Thanks, Bhushan Shah