From 6685da4ec9e3b1eb8a09052c3abb2d614f96d7cc Mon Sep 17 00:00:00 2001
From: Alexey Edelev <alexey.edelev@qt.io>
Date: Thu, 27 Feb 2025 17:29:50 +0100
Subject: [PATCH] Ensure optional_arg is reset in
 qt_internal_add_global_definition

Othewise there is a risk that variable is taken from other scopes.
(Random finding).

Pick-to: 6.5 6.8 6.9
Change-Id: I2517adec5d82639e95ac2758633109413811cf9a
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
---
 cmake/QtInternalTargets.cmake | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cmake/QtInternalTargets.cmake b/cmake/QtInternalTargets.cmake
index 8be08163550..d7c823995c0 100644
--- a/cmake/QtInternalTargets.cmake
+++ b/cmake/QtInternalTargets.cmake
@@ -136,7 +136,7 @@ endfunction()
 #            APP - set the definition for all Qt applications
 #        TODO: Add a tests specific platform target and the definition scope for it.
 function(qt_internal_add_global_definition definition)
-    set(optional_args)
+    set(optional_args "")
     set(single_value_args VALUE)
     set(multi_value_args SCOPE)
     cmake_parse_arguments(arg