From 004437e1cfaa1673bd5fdfb916c782dc1ee3028e Mon Sep 17 00:00:00 2001 From: Ivan Diaz Sanchez Date: Wed, 26 Apr 2023 07:16:19 -0700 Subject: [PATCH] Made helixpublishwitharcade's TargetOS property lowercase. (#85357) Fixes #81141. In PR #80164, the build scripts were updated to always require and/or convert the MSBuild `TargetOS` property to lowercase. However, the _helixpublishwitharcade.proj_ file was not included in these efforts, which led to some confusing behavior and misplaced files, as described in issue #81141. This PR addresses and fixes that. --- src/tests/Common/helixpublishwitharcade.proj | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/tests/Common/helixpublishwitharcade.proj b/src/tests/Common/helixpublishwitharcade.proj index a9bc1b1..3a223bc 100644 --- a/src/tests/Common/helixpublishwitharcade.proj +++ b/src/tests/Common/helixpublishwitharcade.proj @@ -84,7 +84,7 @@ <_PropertiesToPass> TargetArchitecture=$(TargetArchitecture); - TargetOS=$(TargetOS); + TargetOS=$(TargetOS.ToLowerInvariant()); Configuration=$(Configuration); Creator=$(_Creator); HelixAccessToken=$(_HelixAccessToken); -- 2.7.4