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

Windows Terminal "Open in Terminal" explorer.exe right-click context menu not working properly #752

Open
eggbean opened this issue Sep 9, 2022 · 1 comment
Assignees
Labels
Bug This is a problem. To Implement This is something we should implement.

Comments

@eggbean
Copy link

eggbean commented Sep 9, 2022

In Windows Terminal settings when the Command line setting it set to: %ProgramFiles%\WindowsApps\WhitewaterFoundryLtd.Co.16571368D6CFF_22.3.2.0_x64__kd1vv0z0vy70w\DistroLauncher\pengwin.exe the explorer.exe right-click context menu item to "Open in Terminal" doesn't work, as "does not respect the starting directory".

To get around this I have changed Command line: to wsl -d WLinux and Starting directory to ~.

Apparently this problem also occurs in WSL Ubuntu.

Ah, this is the same root cause as the Ubuntu startingDirectory bug. Pengwin has included a JSON file that overrides the commandline to point to pengwin.exe, but pengwin.exe does not respect the starting directory. Any executable that ignores its working directory will exhibit this behavior.

microsoft/terminal#13952

I don't yet know if there are any problems caused by this workaround. I'll report them here if I find any.

@eggbean eggbean added the Bug This is a problem. label Sep 9, 2022
@crramirez
Copy link
Collaborator

Probably because we needed to put a Starting Directory in the Json fragment as a workaround for an issue that Windows Terminal had.

Another workaround would be installing the Open with Pengwin option in pengwin-setup.

Thanks for reporting, we will figure out how to fix it.

Regards,
Carlos

@crramirez crramirez moved this to 📋 Backlog in Pengwin Mar 31, 2023
@crramirez crramirez self-assigned this Nov 14, 2023
@crramirez crramirez added the To Implement This is something we should implement. label Nov 14, 2023
@crramirez crramirez moved this from 📋 Backlog to 🏗 In progress in Pengwin Mar 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a problem. To Implement This is something we should implement.
Projects
Status: 🏗 In progress
Development

No branches or pull requests

2 participants