[PR #2611] [MERGED] Add a project that produces conpty.dll #25012

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

📋 Pull Request Information

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

Base: master ← Head: dev/duhowett/winconpty


📝 Commits (5)

  • 80c481e conpty: add winconpty (conpty.dll)!
  • aa209e4 Add support for conpty.dll launching OpenConsole.exe
  • 60b123c feedback, plus moving closer to inbox
  • 39596da Add a dependency from ft-host -> conpty
  • c61690a feedback part II: the feedbackening

📊 Changes

10 files changed (+577 additions, -1 deletions)

View changed files

📝 OpenConsole.sln (+18 -0)
📝 consolegit2gitfilters.json (+1 -0)
📝 src/host/ft_host/Host.FeatureTests.vcxproj (+10 -1)
➕ src/winconpty/device.h (+27 -0)
➕ src/winconpty/precomp.cpp (+4 -0)
➕ src/winconpty/precomp.h (+49 -0)
➕ src/winconpty/winconpty.cpp (+384 -0)
➕ src/winconpty/winconpty.def (+4 -0)
➕ src/winconpty/winconpty.h (+43 -0)
➕ src/winconpty/winconpty.vcxproj (+37 -0)

📄 Description

Summary of the Pull Request

This pull request introduces a copy of the code from kernel32.dll that implements CreatePseudoConsole, ClosePseudoConsole and ResizePseudoConsole. Apart from some light modifications to fit into the infrastructure in this project and support launching OpenConsole.exe, it is intended to be 1:1 with the code that ships in Windows.

Any guideline violations in this code are likely intentional. Since this was built into kernel32, it used the STL only very sparingly.

kernel32 is on the default link line, so we need to figure out how to get included before it. Either that, or we should export different function names.

References

#1130

PR Checklist

  • Closes #xxx
  • Tests added/passed
  • Requires documentation to be updated
  • I've discussed this with core contributors already.

Validation Steps Performed

Launched vtpipeterm.


🔄 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/2611 **Author:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Created:** 8/30/2019 **Status:** ✅ Merged **Merged:** 9/4/2019 **Merged by:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Base:** `master` ← **Head:** `dev/duhowett/winconpty` --- ### 📝 Commits (5) - [`80c481e`](https://github.com/microsoft/terminal/commit/80c481e1847a43b771ead5f51cb0587ade4b3556) conpty: add winconpty (conpty.dll)! - [`aa209e4`](https://github.com/microsoft/terminal/commit/aa209e403d3a9770caee81765d086d83192930e4) Add support for conpty.dll launching OpenConsole.exe - [`60b123c`](https://github.com/microsoft/terminal/commit/60b123c257618976c27191719a2f953440c27ce1) feedback, plus moving closer to inbox - [`39596da`](https://github.com/microsoft/terminal/commit/39596dab63543abacb59f02976bfddec60afe0ee) Add a dependency from ft-host -> conpty - [`c61690a`](https://github.com/microsoft/terminal/commit/c61690abb6afca9c197c179583bef947c2099dd2) feedback part II: the feedbackening ### 📊 Changes **10 files changed** (+577 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `OpenConsole.sln` (+18 -0) 📝 `consolegit2gitfilters.json` (+1 -0) 📝 `src/host/ft_host/Host.FeatureTests.vcxproj` (+10 -1) ➕ `src/winconpty/device.h` (+27 -0) ➕ `src/winconpty/precomp.cpp` (+4 -0) ➕ `src/winconpty/precomp.h` (+49 -0) ➕ `src/winconpty/winconpty.cpp` (+384 -0) ➕ `src/winconpty/winconpty.def` (+4 -0) ➕ `src/winconpty/winconpty.h` (+43 -0) ➕ `src/winconpty/winconpty.vcxproj` (+37 -0) </details> ### 📄 Description ## Summary of the Pull Request This pull request introduces a copy of the code from kernel32.dll that implements `CreatePseudoConsole`, `ClosePseudoConsole` and `ResizePseudoConsole`. Apart from some light modifications to fit into the infrastructure in this project and support launching `OpenConsole.exe`, it is intended to be 1:1 with the code that ships in Windows. Any guideline violations in this code are likely intentional. Since this was built into kernel32, it used the STL only _very sparingly._ `kernel32` is on the default link line, so we need to figure out how to get included before it. Either that, or we should export different function names. ## References #1130 ## PR Checklist * [ ] Closes #xxx * [ ] Tests added/passed * [ ] Requires documentation to be updated * [x] I've discussed this with core contributors already. ## Validation Steps Performed Launched vtpipeterm. --- <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:06:42 +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#25012