From 0cc2de65d111ee1244f3d3ce53c83b3b55b58dfb Mon Sep 17 00:00:00 2001 From: Bhushan Shah Date: Thu, 15 Sep 2016 18:14:43 +0530 Subject: Make sure we don't add the appstream test twice Guard variable we used to ensure this doesn't happen was not scopped to parent and hence was being reset when funciton returns REVIEW: 128917 --- kde-modules/KDECMakeSettings.cmake | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'kde-modules/KDECMakeSettings.cmake') diff --git a/kde-modules/KDECMakeSettings.cmake b/kde-modules/KDECMakeSettings.cmake index 523271a1..f9649632 100644 --- a/kde-modules/KDECMakeSettings.cmake +++ b/kde-modules/KDECMakeSettings.cmake @@ -144,8 +144,8 @@ endif() find_program(APPSTREAMCLI appstreamcli) function(appstreamtest) - if(APPSTREAMCLI AND NOT _done) - set(_done TRUE) + if(APPSTREAMCLI AND NOT appstreamtest_added) + set(appstreamtest_added TRUE PARENT_SCOPE) add_test(NAME appstreamtest COMMAND cmake -DAPPSTREAMCLI=${APPSTREAMCLI} -DINSTALL_FILES=${CMAKE_BINARY_DIR}/install_manifest.txt -P ${CMAKE_CURRENT_LIST_DIR}/appstreamtest.cmake) else() message(STATUS "Could not set up the appstream test. appstreamcli is missing.") -- cgit v1.2.1