MiniTerm: ProcessFactory.Start is hard-coded to run "cmd.exe" and ignores intended parameter #449

Open
opened 2026-01-30 21:52:40 +00:00 by claunia · 2 comments
Owner

Originally created by @oising on GitHub (Nov 10, 2018).

See:

34ff272cfa/samples/ConPTY/MiniTerm/MiniTerm/Processes/ProcessFactory.cs (L21)

Originally created by @oising on GitHub (Nov 10, 2018). See: https://github.com/Microsoft/console/blob/34ff272cfa5b49dcb0bbe7aaec41797d9c285e5e/samples/ConPTY/MiniTerm/MiniTerm/Processes/ProcessFactory.cs#L21
claunia added the Help WantedWork-ItemArea-InteropProduct-ConptyIssue-Samples labels 2026-01-30 21:52:40 +00:00
Author
Owner

@zadjii-msft commented on GitHub (Nov 12, 2018):

Huh. That does look wrong to me. Would love to review a PR :)

@zadjii-msft commented on GitHub (Nov 12, 2018): Huh. That does look wrong to me. Would love to review a PR :)
Author
Owner

@oising commented on GitHub (Nov 13, 2018):

There are a few other things that need reshaping. I'll see what I can do.

@oising commented on GitHub (Nov 13, 2018): There are a few other things that need reshaping. I'll see what I can do.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#449