[PR #4536] [MERGED] deps: upgrade CppWinRT to 2.0.200316.3, gsl to v2.1.0 #25816

Open
opened 2026-01-31 09:11:58 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/4536
Author: @DHowett-MSFT
Created: 2/11/2020
Status: ✅ Merged
Merged: 3/23/2020
Merged by: @undefined

Base: master ← Head: dev/duhowett/c++++winrt


📝 Commits (7)

  • 799c46f deps: upgrade CppWinRT to 2.0.200203.5
  • 61f551d Fix build error from cppwinrt update
  • 2bef4e4 TerminalAppLib no longer needs to fake its xaml reference
  • 71162e8 Merge remote-tracking branch 'origin/master' into dev/duhowett/c++++winrt
  • 79c7a41 Microsoft.Windows.CppWinRT -> 2.0.200316.3
  • b94572b Add deps to WindowsTerminalUniversal
  • 6214856 gsl -> v2.1.0

📊 Changes

18 files changed (+48 additions, -63 deletions)

View changed files

📝 dep/gsl (+1 -1)
📝 src/cascadia/TerminalApp/TerminalApp.vcxproj (+12 -5)
📝 src/cascadia/TerminalApp/TerminalPage.cpp (+2 -1)
📝 src/cascadia/TerminalApp/lib/TerminalAppLib.vcxproj (+5 -33)
📝 src/cascadia/TerminalApp/packages.config (+1 -1)
📝 src/cascadia/TerminalAzBridge/packages.config (+1 -1)
📝 src/cascadia/TerminalConnection/packages.config (+1 -1)
📝 src/cascadia/TerminalControl/packages.config (+1 -1)
📝 src/cascadia/TerminalCore/packages.config (+1 -1)
📝 src/cascadia/TerminalSettings/packages.config (+1 -1)
📝 src/cascadia/WinRTUtils/packages.config (+1 -1)
📝 src/cascadia/WindowsTerminal/packages.config (+1 -1)
📝 src/cascadia/WindowsTerminalUniversal/WindowsTerminalUniversal.vcxproj (+10 -5)
📝 src/cascadia/WindowsTerminalUniversal/packages.config (+1 -1)
📝 src/cppwinrt.build.post.props (+3 -3)
📝 src/cppwinrt.build.pre.props (+1 -1)
📝 src/tools/U8U16Test/U8U16Test.vcxproj (+4 -4)
📝 src/tools/U8U16Test/packages.config (+1 -1)

📄 Description

This commit upgrades C++/WinRT to 2.0.200316.3 and fixes a couple of the
transitive dependency issues we had in the process.

Because the latest version has better dependency resolution, we're able
to properly depend on Microsoft.UI.Xaml and the Toolkit in TerminalApp
and TerminalAppLib so we no longer need to manually include .dll and
.pri files.

Because of nebulous other changes in dependency resolution,
WindowsTerminalUniversal isn't picking up transitive .winmd dependencies
from TerminalApp, and needs to include them as ProjectReferences
directly. This was already happening transitively, so now it's explicit.

I've also taken the time to upgrade GSL to v2.1.0, the last release
before they removed span::at and blew up our world.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/4536 **Author:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Created:** 2/11/2020 **Status:** ✅ Merged **Merged:** 3/23/2020 **Merged by:** [@undefined](undefined) **Base:** `master` ← **Head:** `dev/duhowett/c++++winrt` --- ### 📝 Commits (7) - [`799c46f`](https://github.com/microsoft/terminal/commit/799c46f2e3993587635c01d58b10744fefa52254) deps: upgrade CppWinRT to 2.0.200203.5 - [`61f551d`](https://github.com/microsoft/terminal/commit/61f551d18b74bd5a866e8e96ea1a59d917736c71) Fix build error from cppwinrt update - [`2bef4e4`](https://github.com/microsoft/terminal/commit/2bef4e484223de3eb89638d59a3c63aafbec50a9) TerminalAppLib no longer needs to fake its xaml reference - [`71162e8`](https://github.com/microsoft/terminal/commit/71162e86ea1be8b86f0d8660ae44759b8790be63) Merge remote-tracking branch 'origin/master' into dev/duhowett/c++++winrt - [`79c7a41`](https://github.com/microsoft/terminal/commit/79c7a41739f0da7c43830dcb61ab5c69610339c3) Microsoft.Windows.CppWinRT -> 2.0.200316.3 - [`b94572b`](https://github.com/microsoft/terminal/commit/b94572bcb32ba93d7f26a2c792c5cb993f7a4427) Add deps to WindowsTerminalUniversal - [`6214856`](https://github.com/microsoft/terminal/commit/621485631eb152cbd1be471fdff44ff4fc43ac5f) gsl -> v2.1.0 ### 📊 Changes **18 files changed** (+48 additions, -63 deletions) <details> <summary>View changed files</summary> 📝 `dep/gsl` (+1 -1) 📝 `src/cascadia/TerminalApp/TerminalApp.vcxproj` (+12 -5) 📝 `src/cascadia/TerminalApp/TerminalPage.cpp` (+2 -1) 📝 `src/cascadia/TerminalApp/lib/TerminalAppLib.vcxproj` (+5 -33) 📝 `src/cascadia/TerminalApp/packages.config` (+1 -1) 📝 `src/cascadia/TerminalAzBridge/packages.config` (+1 -1) 📝 `src/cascadia/TerminalConnection/packages.config` (+1 -1) 📝 `src/cascadia/TerminalControl/packages.config` (+1 -1) 📝 `src/cascadia/TerminalCore/packages.config` (+1 -1) 📝 `src/cascadia/TerminalSettings/packages.config` (+1 -1) 📝 `src/cascadia/WinRTUtils/packages.config` (+1 -1) 📝 `src/cascadia/WindowsTerminal/packages.config` (+1 -1) 📝 `src/cascadia/WindowsTerminalUniversal/WindowsTerminalUniversal.vcxproj` (+10 -5) 📝 `src/cascadia/WindowsTerminalUniversal/packages.config` (+1 -1) 📝 `src/cppwinrt.build.post.props` (+3 -3) 📝 `src/cppwinrt.build.pre.props` (+1 -1) 📝 `src/tools/U8U16Test/U8U16Test.vcxproj` (+4 -4) 📝 `src/tools/U8U16Test/packages.config` (+1 -1) </details> ### 📄 Description This commit upgrades C++/WinRT to 2.0.200316.3 and fixes a couple of the transitive dependency issues we had in the process. Because the latest version has better dependency resolution, we're able to properly depend on Microsoft.UI.Xaml and the Toolkit in TerminalApp and TerminalAppLib so we no longer need to manually include .dll and .pri files. Because of nebulous _other_ changes in dependency resolution, WindowsTerminalUniversal isn't picking up transitive .winmd dependencies from TerminalApp, and needs to include them as ProjectReferences directly. This was already happening transitively, so now it's explicit. I've also taken the time to upgrade GSL to v2.1.0, the last release before they removed span::at and blew up our world. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:11:58 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#25816