[PR #5673] [MERGED] On second thought, embed the third-party notices in the package #26418

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/5673
Author: @DHowett-MSFT
Created: 4/30/2020
Status: ✅ Merged
Merged: 4/30/2020
Merged by: @DHowett-MSFT

Base: master ← Head: dev/duhowett/notice_me_senpai


📝 Commits (2)

  • 5ce760d On second thought, embed the third-party notices in the package
  • 870fc38 Update src/cascadia/CascadiaPackage/NOTICE.html

📊 Changes

10 files changed (+42 additions, -34 deletions)

View changed files

📝 NOTICE.md (+1 -1)
➕ build/scripts/Generate-ThirdPartyNotices.ps1 (+14 -0)
➕ src/cascadia/CascadiaPackage/NOTICE.html (+16 -0)
📝 src/cascadia/CascadiaResources.build.items (+5 -0)
📝 src/cascadia/TerminalApp/TerminalPage.cpp (+4 -5)
📝 src/cascadia/TerminalApp/TerminalPage.h (+1 -0)
📝 src/cascadia/TerminalApp/TerminalPage.idl (+0 -2)
📝 src/cascadia/TerminalApp/TerminalPage.xaml (+1 -1)
📝 src/cascadia/TerminalApp/lib/TerminalAppLib.vcxproj (+0 -4)
➖ tools/GenerateCommitHashHeader.ps1 (+0 -21)

📄 Description

This commit introduces a NOTICE.html file that will be embedded into the
package. It will be stamped down with the real notices during a branded
release build (as part of the build pipeline.)

It, in part, reverts some of the really good work in determining the
commit hash at build time. That work will be preserved in history.

This is more compliant with our duties to the OSS we consume. 😄


🔄 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/5673 **Author:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Created:** 4/30/2020 **Status:** ✅ Merged **Merged:** 4/30/2020 **Merged by:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Base:** `master` ← **Head:** `dev/duhowett/notice_me_senpai` --- ### 📝 Commits (2) - [`5ce760d`](https://github.com/microsoft/terminal/commit/5ce760dff8e22505ea5e8d1ec3ca4042f94c1005) On second thought, embed the third-party notices in the package - [`870fc38`](https://github.com/microsoft/terminal/commit/870fc380075030fc23de1eadd336a8175c682f59) Update src/cascadia/CascadiaPackage/NOTICE.html ### 📊 Changes **10 files changed** (+42 additions, -34 deletions) <details> <summary>View changed files</summary> 📝 `NOTICE.md` (+1 -1) ➕ `build/scripts/Generate-ThirdPartyNotices.ps1` (+14 -0) ➕ `src/cascadia/CascadiaPackage/NOTICE.html` (+16 -0) 📝 `src/cascadia/CascadiaResources.build.items` (+5 -0) 📝 `src/cascadia/TerminalApp/TerminalPage.cpp` (+4 -5) 📝 `src/cascadia/TerminalApp/TerminalPage.h` (+1 -0) 📝 `src/cascadia/TerminalApp/TerminalPage.idl` (+0 -2) 📝 `src/cascadia/TerminalApp/TerminalPage.xaml` (+1 -1) 📝 `src/cascadia/TerminalApp/lib/TerminalAppLib.vcxproj` (+0 -4) ➖ `tools/GenerateCommitHashHeader.ps1` (+0 -21) </details> ### 📄 Description This commit introduces a NOTICE.html file that will be embedded into the package. It will be stamped down with the real notices during a branded release build (as part of the build pipeline.) It, in part, reverts some of the really good work in determining the commit hash at build time. That work will be preserved in history. This is more compliant with our duties to the OSS we consume. :smile: --- <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:15:56 +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#26418