Skip to content

Commit

Permalink
Made helixpublishwitharcade's TargetOS property lowercase. (#85357)
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
ivdiazsa committed Apr 26, 2023
1 parent 4e5c9de commit 004437e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/tests/Common/helixpublishwitharcade.proj
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@

<_PropertiesToPass>
TargetArchitecture=$(TargetArchitecture);
TargetOS=$(TargetOS);
TargetOS=$(TargetOS.ToLowerInvariant());
Configuration=$(Configuration);
Creator=$(_Creator);
HelixAccessToken=$(_HelixAccessToken);
Expand Down

0 comments on commit 004437e

Please sign in to comment.