Things we can fix when we can safely rev the DefTerm interface #17661

Closed
opened 2026-01-31 05:49:18 +00:00 by claunia · 1 comment
Owner

Originally created by @zadjii-msft on GitHub (Jun 8, 2022).

We're in a world where we can't really change much of the delegation console interface. We can't really change how OpenConsole communicates with conhost, cause there are newer OpenConsoles that run on older conhosts, and vice-versa. Should we ever find ourselves in a place where we can rev that boundary, we should make sure to also fix the following:

  • #13247 - We do a lot of wild stuff with duplicating handles. If we could just pass the PID to conhost, dang would that be easier.
    • we actually didn't do the crazy thing we were going to do
Originally created by @zadjii-msft on GitHub (Jun 8, 2022). We're in a world where we can't _really_ change much of the delegation console interface. We can't really change how OpenConsole communicates with conhost, cause there are newer OpenConsoles that run on older conhosts, and vice-versa. Should we ever find ourselves in a place where we _can_ rev that boundary, we should make sure to also fix the following: * [x] #13247 - We do a lot of wild stuff with duplicating handles. If we could just pass the PID to conhost, dang would that be easier. - we actually didn't do the crazy thing we were going to do
Author
Owner

@zadjii-msft commented on GitHub (Jun 10, 2022):

You know what, we actually didn't do the crazy thing we were going to do in #13247 that prompted this issue in the first place. We're closing this for now. If we ever need it again, we know where to find it.

@zadjii-msft commented on GitHub (Jun 10, 2022): You know what, we actually didn't do the crazy thing we were going to do in #13247 that prompted this issue in the first place. We're closing this for now. If we ever need it again, we know where to find it.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#17661