Skip to content
Snippets Groups Projects
Commit 11a8cda2 authored by Joakim Söderberg's avatar Joakim Söderberg Committed by Andy Green
Browse files

BUGFIX: Fixed bug for installing test-apps for CMake.

Appending to the TEST_APP_LIST variable function did not work from within the create_test_app function (It would create a function scope variable with the same name and discard it). We instead need to use a macro for this.

The previous commit also removed the duplicate install for the test-apps (because we want all the install steps in one place in the file for clarity). That change made it obvious that the TEST_APP_LIST never gets populated.
parent 7eadd586
No related branches found
No related tags found
No related merge requests found
......@@ -449,7 +449,7 @@ if (NOT WITHOUT_TESTAPPS)
#
# Helper function for adding a test app.
#
function(create_test_app TEST_NAME MAIN_SRC WIN32_SRCS WIN32_HDRS)
macro(create_test_app TEST_NAME MAIN_SRC WIN32_SRCS WIN32_HDRS)
set(TEST_SRCS ${MAIN_SRC})
set(TEST_HDR)
......@@ -486,7 +486,7 @@ if (NOT WITHOUT_TESTAPPS)
# Add to the list of tests.
list(APPEND TEST_APP_LIST ${TEST_NAME})
endfunction()
endmacro()
if (WITH_SSL AND NOT USE_CYASSL)
message("Searching for OpenSSL executable and dlls")
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment