[PR #2347] [MERGED] Fix the ut_app build for VS 16.2, 16.3 #24892

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/2347
Author: @zadjii-msft
Created: 8/8/2019
Status: ✅ Merged
Merged: 8/9/2019
Merged by: @DHowett-MSFT

Base: master ← Head: dev/migrie/b/2143-fix-ut_app


📝 Commits (1)

  • 6ef1447 Fix the ut_app build for VS 16.2, 16.3

📊 Changes

3 files changed (+37 additions, -27 deletions)

View changed files

➕ src/cascadia/TerminalApp/FixVisualStudioBug.targets (+30 -0)
📝 src/cascadia/TerminalApp/TerminalApp.vcxproj (+3 -27)
📝 src/cascadia/ut_app/TerminalApp.UnitTests.vcxproj (+4 -0)

📄 Description

Summary of the Pull Request

Move the hack from TerminalApp.vcxproj to a .targets file to be used by the ut_app project too.

References

This might need to go into #2294

PR Checklist


🔄 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/2347 **Author:** [@zadjii-msft](https://github.com/zadjii-msft) **Created:** 8/8/2019 **Status:** ✅ Merged **Merged:** 8/9/2019 **Merged by:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Base:** `master` ← **Head:** `dev/migrie/b/2143-fix-ut_app` --- ### 📝 Commits (1) - [`6ef1447`](https://github.com/microsoft/terminal/commit/6ef14477aa2859d9d2d3c1834423ed24ad4325df) Fix the ut_app build for VS 16.2, 16.3 ### 📊 Changes **3 files changed** (+37 additions, -27 deletions) <details> <summary>View changed files</summary> ➕ `src/cascadia/TerminalApp/FixVisualStudioBug.targets` (+30 -0) 📝 `src/cascadia/TerminalApp/TerminalApp.vcxproj` (+3 -27) 📝 `src/cascadia/ut_app/TerminalApp.UnitTests.vcxproj` (+4 -0) </details> ### 📄 Description <!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? --> ## Summary of the Pull Request Move the hack from TerminalApp.vcxproj to a .targets file to be used by the ut_app project too. <!-- Other than the issue solved, is this relevant to any other issues/existing PRs? --> ## References This might need to go into #2294 <!-- Please review the items on the PR checklist before submitting--> ## PR Checklist * [x] Closes #2143 * [x] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [x] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx --- <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:05:57 +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#24892