Applied patch a9183756-gh#53

Author: KSDaemon

Arduino/System/BoardToolchain.cmake

@@ -217,7 +217,7 @@ function (SetupBoardToolchain boards_namespace board_id generate_dir)
 		_board_get_ref_platform("${_tool_name}" _ref_tool_pkg_name _tool_name)
 		set(ARDUINO_BOARD_${_c_tool_prop} "${_tool_name}")
 		if (_ref_tool_pkg_name AND
-			# If same as _core_pkg_name or pkg_name, no need to trasnfer
+			# If same as _core_pkg_name or pkg_name, no need to transfer
 			# tool properties, because all their platform properties got
 			# transferred already.
 			NOT "${_ref_tool_pkg_name}" STREQUAL "${_core_pkg_name}" AND

Arduino/Utilities/PropertiesReader.cmake

@@ -200,43 +200,49 @@ function(_properties_expand_value value return_value namespace)
 		return()
 	endif ()
 
-	# Get the variables list
-	string(REGEX MATCHALL "{[^{}/]+}" _var_list "${_value}")
-	if (NOT "${_var_list}" STREQUAL "")
-		list(REMOVE_DUPLICATES _var_list)
-	endif()
-	foreach(_var_str IN LISTS _var_list)
-
-		# Get the variable name
-		string(REGEX MATCH "^{(.*)}$" _match "${_var_str}")
-		set(_var_name "${CMAKE_MATCH_1}")
-
-		# Check if not resolved already
-		if (NOT DEFINED "/prop_int_resolved.${_var_name}")
-			# If such a variable is not in the namespace, no need to resolve
-			if (NOT DEFINED "${namespace}.${_var_name}")
-				properties_set_value("/prop_int_unresolved" ${_var_name} "")
-				continue()
-			endif()
-
-			# Temporarily resolve it to the same variable to handle recursive
-			# references
-			properties_set_value("/prop_int_resolved" "${_var_name}"
-				"{${_var_name}}")
-
-			# message("=> Resolve *** ${_var_name} *** : "
-			#	"${${namespace}.${_var_name}}")
-			_properties_expand_value("${${namespace}.${_var_name}}"
-				_var_value "${namespace}")
-			properties_set_value("/prop_int_resolved" "${_var_name}"
-				"${_var_value}")
-			# message("=> EXPANDED ${_var_name}: ${_var_value}")
+	# Set previous value to nothing so that there is at least one iteration
+	set(_previous_value "")
+	while(NOT "${_previous_value}" STREQUAL "${_value}")
+		set(_previous_value "${_value}")
+
+		# Get the variables list
+		string(REGEX MATCHALL "{[^{}/]+}" _var_list "${_value}")
+		if (NOT "${_var_list}" STREQUAL "")
+			list(REMOVE_DUPLICATES _var_list)
 		endif()
+		foreach(_var_str IN LISTS _var_list)
+
+			# Get the variable name
+			string(REGEX MATCH "^{(.*)}$" _match "${_var_str}")
+			set(_var_name "${CMAKE_MATCH_1}")
+
+			# Check if not resolved already
+			if (NOT DEFINED "/prop_int_resolved.${_var_name}")
+				# If such a variable is not in the namespace, no need to resolve
+				if (NOT DEFINED "${namespace}.${_var_name}")
+					properties_set_value("/prop_int_unresolved" ${_var_name} "")
+					continue()
+				endif()
+
+				# Temporarily resolve it to the same variable to handle recursive
+				# references
+				properties_set_value("/prop_int_resolved" "${_var_name}"
+					"{${_var_name}}")
+
+				# message("=> Resolve *** ${_var_name} *** : "
+				#	"${${namespace}.${_var_name}}")
+				_properties_expand_value("${${namespace}.${_var_name}}"
+					_var_value "${namespace}")
+				properties_set_value("/prop_int_resolved" "${_var_name}"
+					"${_var_value}")
+				# message("=> EXPANDED ${_var_name}: ${_var_value}")
+			endif()
 
-		string(REPLACE "${_var_str}" "${/prop_int_resolved.${_var_name}}"
-			_value "${_value}")
+			string(REPLACE "${_var_str}" "${/prop_int_resolved.${_var_name}}"
+				_value "${_value}")
 
-	endforeach()
+		endforeach()
+	endwhile()
 
 	properties_set_parent_scope("/prop_int_resolved")
 	properties_set_parent_scope("/prop_int_unresolved")

Tests/CMakeLists.txt

@@ -218,6 +218,7 @@ function(SetupBoardTests)
 		set_tests_properties("${distinct_id}" PROPERTIES
 			FIXTURES_REQUIRED "PlTestResults"
 			SKIP_RETURN_CODE 100
+			TIMEOUT 3600
 		)
 
 		if ("${shell_cmd}" STREQUAL "" AND
@@ -354,6 +355,7 @@ function(SetupPlatformTests pl_url ref_url_list return_pl_tests)
 		set_tests_properties("${test_id}" PROPERTIES
 			FIXTURES_REQUIRED "TestResults"
 			SKIP_RETURN_CODE 100
+			TIMEOUT 3600
 		)
 		if ("${shell_cmd}" STREQUAL "" AND
 			NOT CMAKE_VERSION VERSION_LESS "3.16.0")