diff options
| author | Bhushan Shah <bhush94@gmail.com> | 2016-09-15 18:14:43 +0530 |
|---|---|---|
| committer | Bhushan Shah <bhush94@gmail.com> | 2016-09-15 18:14:43 +0530 |
| commit | 0cc2de65d111ee1244f3d3ce53c83b3b55b58dfb (patch) | |
| tree | ed2023d85bcf312532ecbe11481efb80dff48b17 | |
| parent | 423e7e9fb6efeb33694dfc1e20a279cdfadd823d (diff) | |
| download | extra-cmake-modules-0cc2de65d111ee1244f3d3ce53c83b3b55b58dfb.tar.gz extra-cmake-modules-0cc2de65d111ee1244f3d3ce53c83b3b55b58dfb.tar.bz2 | |
Make sure we don't add the appstream test twicev5.27.0-rc2v5.27.0-rc1v5.27.0v5.26.0-rc2
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
| -rw-r--r-- | kde-modules/KDECMakeSettings.cmake | 4 |
1 files changed, 2 insertions, 2 deletions
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.") |