[PR #1900] [MERGED] Fix the WAP packaging project #24699

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/1900
Author: @DHowett-MSFT
Created: 7/10/2019
Status: Merged
Merged: 7/12/2019
Merged by: @DHowett-MSFT

Base: masterHead: dev/duhowett/fixwap


📝 Commits (3)

  • 0be3224 Fix the WAP packaging project
  • 68ab678 Apply a couple more fixes to the WAP project
  • 70e987c Update src/cascadia/TerminalApp/TerminalApp.vcxproj

📊 Changes

5 files changed (+54 additions, -31 deletions)

View changed files

📝 src/cascadia/CascadiaPackage/CascadiaPackage.wapproj (+26 -16)
📝 src/cascadia/TerminalApp/MinMaxCloseControl.cpp (+1 -2)
📝 src/cascadia/TerminalApp/TerminalApp.vcxproj (+26 -0)
📝 src/cascadia/TerminalApp/TerminalPage.cpp (+1 -4)
📝 src/cascadia/WindowsTerminal/WindowsTerminal.vcxproj (+0 -9)

📄 Description

This commits fixes the centennial package by:

  • Forcing XBF (XAML binary format) files to be embedded in project
    PRI files.
  • Moving package content generation to before PRI generation
  • Collecting all of the package's PRI files to merge into resources.pri
  • Fixing the hardcoded resource paths to reflect the new reality.

It also includes a magic value that fixes the bug where the project is
autodetected as a Mixed (CLR + Native) project.

Fixes #1816.

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/1900 **Author:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Created:** 7/10/2019 **Status:** ✅ Merged **Merged:** 7/12/2019 **Merged by:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Base:** `master` ← **Head:** `dev/duhowett/fixwap` --- ### 📝 Commits (3) - [`0be3224`](https://github.com/microsoft/terminal/commit/0be3224f41f14be5a34fd47adb174c0c6a5a3bc6) Fix the WAP packaging project - [`68ab678`](https://github.com/microsoft/terminal/commit/68ab67867a84bc4e93310435569414713f181a8a) Apply a couple more fixes to the WAP project - [`70e987c`](https://github.com/microsoft/terminal/commit/70e987c7b248b76c0590b26d88bbe6665aa8938e) Update src/cascadia/TerminalApp/TerminalApp.vcxproj ### 📊 Changes **5 files changed** (+54 additions, -31 deletions) <details> <summary>View changed files</summary> 📝 `src/cascadia/CascadiaPackage/CascadiaPackage.wapproj` (+26 -16) 📝 `src/cascadia/TerminalApp/MinMaxCloseControl.cpp` (+1 -2) 📝 `src/cascadia/TerminalApp/TerminalApp.vcxproj` (+26 -0) 📝 `src/cascadia/TerminalApp/TerminalPage.cpp` (+1 -4) 📝 `src/cascadia/WindowsTerminal/WindowsTerminal.vcxproj` (+0 -9) </details> ### 📄 Description This commits fixes the centennial package by: * Forcing XBF (XAML binary format) files to be embedded in project PRI files. * Moving package content generation to before PRI generation * Collecting all of the package's PRI files to merge into resources.pri * Fixing the hardcoded resource paths to reflect the new reality. It also includes a magic value that fixes the bug where the project is autodetected as a Mixed (CLR + Native) project. Fixes #1816. <!-- Please review the items on the PR checklist before submitting--> ## PR Checklist * [x] Closes #1816 * [x] CLA signed * [x] Tests added/passed/na? * [x] Requires documentation to be updated/na? * [x] I've discussed this with core contributors already. --- <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:04:50 +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#24699