Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Empty/debug conhost shells created from shims #5505

Closed
brian6932 opened this issue May 7, 2023 · 2 comments
Closed

[Bug] Empty/debug conhost shells created from shims #5505

brian6932 opened this issue May 7, 2023 · 2 comments
Labels

Comments

@brian6932
Copy link

brian6932 commented May 7, 2023

Bug Report

Current Behavior

When shims are opened from explorer/start menu/run, they have to spawn an extra conhost shell, then the program itself as a sub-process.

Expected Behavior

This is not ideal, you end up with either an empty shell, or a debug shell, and it looks very ugly for programs like terminal emulators, or browsers. Especially for use cases like wanting to spawn a shell from a path when you're already in explorer, normally you'd just alt+d->program_name.

Possible Solution

Instead there should be some way to contextually start shells. I know there are some Windows API limitations here, but maybe some way toggle CREATE_NO_WINDOW as a shim prop, there are prob smarter ways of handling this.

The alternative is to append $env:PATH with env_add_path, often times this is not ideal, and can lead to path pollution. Another challenge to a solution here would be whether or not it would be possible to launch from some type of symlink-like link to the bin, and having it work as a nested dir, then you could accomplish this effectively without polluting path (I don't believe such a solution is easily possible on Windows, as symlinks and junctions don't behave this way, but maybe there is some type of solution like this).

Before anyone says anything like just & (scoop which shim | cvpa), often times your native shell on Windows will not be pwsh/powershell, so this is simply not feasible.

Screenshots

image
image
image

System details

Windows version: 10

OS architecture: 64bit

PowerShell version: 7.4.0-preview.3

@brian6932 brian6932 added the bug label May 7, 2023
@rashil2000
Copy link
Member

Duplicate of #1606

@rashil2000 rashil2000 marked this as a duplicate of #1606 May 10, 2023
@brian6932
Copy link
Author

That's a bit general, but sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants