[PR #13007] [MERGED] Add InteractivityOneCore/RendererWddmCon projects #29327

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/13007
Author: @lhecker
Created: 4/30/2022
Status: ✅ Merged
Merged: 5/4/2022
Merged by: @undefined

Base: main ← Head: dev/lhecker/build-onecore-wddmcon


📝 Commits (3)

  • 598f0ca Add InteractivityOneCore/RendererWddmCon projects
  • 65ff808 Fix AuditMode checks
  • 7540222 Address feedback

📊 Changes

50 files changed (+932 additions, -537 deletions)

View changed files

📝 .github/actions/spelling/expect/expect.txt (+3 -2)
📝 OpenConsole.sln (+78 -2)
➕ dep/Console/ConIoSrv.h (+76 -0)
📝 dep/Console/condrv.h (+10 -2)
➕ dep/Console/csrmsg.h (+24 -0)
➕ dep/Console/ntcsrdll.h (+15 -0)
➕ dep/Console/ntcsrmsg.h (+16 -0)
➕ dep/Console/ntlpcapi.h (+126 -0)
📝 src/host/conddkrefs.h (+43 -6)
📝 src/interactivity/inc/IAccessibilityNotifier.hpp (+1 -9)
📝 src/interactivity/inc/IConsoleControl.hpp (+1 -9)
📝 src/interactivity/inc/IConsoleInputThread.hpp (+4 -6)
📝 src/interactivity/inc/IConsoleWindow.hpp (+1 -3)
📝 src/interactivity/inc/IHighDpiApi.hpp (+1 -9)
📝 src/interactivity/inc/IInteractivityFactory.hpp (+1 -3)
📝 src/interactivity/inc/ISystemConfigurationProvider.hpp (+1 -9)
📝 src/interactivity/inc/IWindowMetrics.hpp (+1 -9)
📝 src/interactivity/onecore/AccessibilityNotifier.cpp (+8 -8)
📝 src/interactivity/onecore/AccessibilityNotifier.hpp (+9 -9)
📝 src/interactivity/onecore/BgfxEngine.cpp (+24 -35)

...and 30 more files

📄 Description

InteractivityOneCore and RendererWddmCon were the last two remaining
projects which are relevant for our internal console builds, but couldn't be
easily compiled publicly by users on GitHub. This commit adds all definitions
required to compile the two projects into dysfunctional libraries at least.
(Since the added definitions are deliberately incorrect.)

Additionally this commit fixes the AuditMode build for the two projects.

Validation Steps Performed

The two new projects compile fine.


🔄 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/13007 **Author:** [@lhecker](https://github.com/lhecker) **Created:** 4/30/2022 **Status:** ✅ Merged **Merged:** 5/4/2022 **Merged by:** [@undefined](undefined) **Base:** `main` ← **Head:** `dev/lhecker/build-onecore-wddmcon` --- ### 📝 Commits (3) - [`598f0ca`](https://github.com/microsoft/terminal/commit/598f0caed6c0fae7d6c2a566a9ba0ed0e2f81656) Add InteractivityOneCore/RendererWddmCon projects - [`65ff808`](https://github.com/microsoft/terminal/commit/65ff8087bdc6d24d167a359dca94cb6eef81a693) Fix AuditMode checks - [`7540222`](https://github.com/microsoft/terminal/commit/7540222d73204a1be68ea8a9354c9f996461c084) Address feedback ### 📊 Changes **50 files changed** (+932 additions, -537 deletions) <details> <summary>View changed files</summary> 📝 `.github/actions/spelling/expect/expect.txt` (+3 -2) 📝 `OpenConsole.sln` (+78 -2) ➕ `dep/Console/ConIoSrv.h` (+76 -0) 📝 `dep/Console/condrv.h` (+10 -2) ➕ `dep/Console/csrmsg.h` (+24 -0) ➕ `dep/Console/ntcsrdll.h` (+15 -0) ➕ `dep/Console/ntcsrmsg.h` (+16 -0) ➕ `dep/Console/ntlpcapi.h` (+126 -0) 📝 `src/host/conddkrefs.h` (+43 -6) 📝 `src/interactivity/inc/IAccessibilityNotifier.hpp` (+1 -9) 📝 `src/interactivity/inc/IConsoleControl.hpp` (+1 -9) 📝 `src/interactivity/inc/IConsoleInputThread.hpp` (+4 -6) 📝 `src/interactivity/inc/IConsoleWindow.hpp` (+1 -3) 📝 `src/interactivity/inc/IHighDpiApi.hpp` (+1 -9) 📝 `src/interactivity/inc/IInteractivityFactory.hpp` (+1 -3) 📝 `src/interactivity/inc/ISystemConfigurationProvider.hpp` (+1 -9) 📝 `src/interactivity/inc/IWindowMetrics.hpp` (+1 -9) 📝 `src/interactivity/onecore/AccessibilityNotifier.cpp` (+8 -8) 📝 `src/interactivity/onecore/AccessibilityNotifier.hpp` (+9 -9) 📝 `src/interactivity/onecore/BgfxEngine.cpp` (+24 -35) _...and 30 more files_ </details> ### 📄 Description `InteractivityOneCore` and `RendererWddmCon` were the last two remaining projects which are relevant for our internal console builds, but couldn't be easily compiled publicly by users on GitHub. This commit adds all definitions required to compile the two projects into dysfunctional libraries at least. (Since the added definitions are deliberately incorrect.) Additionally this commit fixes the AuditMode build for the two projects. ## Validation Steps Performed The two new projects compile fine. --- <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:34:17 +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#29327