[Tarantool-patches] [PATCH v2 luajit 3/7] cmake: introduce AppendTestEnvVar macro
Sergey Bronnikov
sergeyb at tarantool.org
Tue Sep 24 14:07:56 MSK 2024
Hi, Sergey
LGTM
On 24.09.2024 13:29, Sergey Kaplun wrote:
> It is useful to update the environment variable for some tests. For
> CMake versions >= 3.22, we can use ENVIRONMENT_MODIFICATION [1] instead.
> But unless we bump the CMake version, this macro is a workaround.
>
> [1]:https://cmake.org/cmake/help/latest/prop_test/ENVIRONMENT_MODIFICATION.html
>
> Part of tarantool/tarantool#9898
> ---
> test/tarantool-tests/CMakeLists.txt | 28 ++++++++++++++++++++++++++++
> 1 file changed, 28 insertions(+)
>
> diff --git a/test/tarantool-tests/CMakeLists.txt b/test/tarantool-tests/CMakeLists.txt
> index 11a84496..ff402dc0 100644
> --- a/test/tarantool-tests/CMakeLists.txt
> +++ b/test/tarantool-tests/CMakeLists.txt
> @@ -18,6 +18,34 @@ macro(BuildTestCLib lib sources)
> set(LD_LIBRARY_PATH "${CMAKE_CURRENT_BINARY_DIR}:${LD_LIBRARY_PATH}" PARENT_SCOPE)
> endmacro()
>
> +# FIXME: This is used only due to ancient CMake requirements.
> +# If we update to CMake >= 3.22, we can use
> +# ENVIRONMENT_MODIFICATION [1] instead.
> +# [1]:https://cmake.org/cmake/help/latest/prop_test/ENVIRONMENT_MODIFICATION.html
> +# This macro appends to the given test's environment variable the
> +# given value. If the variable doesn't exist, it is created equal
> +# to the given argument.
> +macro(AppendTestEnvVar testname var value)
> + get_test_property(${testname} ENVIRONMENT old_env)
> + foreach(loopvar ${old_env})
> + # XXX: `foreach()` unescapes ";" in string, escape them back.
> + string(REPLACE ";" "\;" loopvar "${loopvar}")
> + if("${loopvar}" MATCHES "^${var}=(.*)")
> + set(envvar_found TRUE)
> + set(loopvar "${var}=${value}${CMAKE_MATCH_1}")
> + endif()
> + list(APPEND new_env "${loopvar}")
> + endforeach()
> + if(NOT "${envvar_found}")
> + list(APPEND new_env "${var}=${value}")
> + endif()
> + set_tests_properties(${testname} PROPERTIES ENVIRONMENT "${new_env}")
> +
> + unset(envvar_found)
> + unset(old_env)
> + unset(new_env)
> +endmacro()
> +
> add_subdirectory(ffi-ccall)
> add_subdirectory(fix-bit-shift-generation)
> add_subdirectory(gh-4427-ffi-sandwich)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.tarantool.org/pipermail/tarantool-patches/attachments/20240924/89fa852f/attachment.htm>
More information about the Tarantool-patches
mailing list