search

bostrot / wsl2-distro-manager

A GUI to quickly manage your WSL2 instances
https://apps.microsoft.com/store/detail/wsl-manager/9NWS9K95NMJB
Other
1.7k stars 106 forks source link

[BUG] Exception when WT (Terminal) not installed #252

Open RedyAu opened 3 weeks ago

RedyAu commented 3 weeks ago

Is there an existing issue for this?

Current Behavior

When I click "Open with Windows Terminal", an error message appears (as I don't have Terminal installed):

Windows cannot find 'wt'. Make sure you've typed the name correctly, then try again.

Expected Behavior

Definitely make a more useful error message than this: "Windows Terminal is not installed, please install it for this feature to work".

Or you could even prompt the user to download Windows Terminal.

But the best would be to fall back to cmd. I, for one, don't like the bloat and slowness of WT, and would prefer plain cmd, or even PowerShell.

Steps To Reproduce

  1. Have no WT installed
  2. Click Open with Windows Terminal

Environment

No response

Anything else?

Thank you for this great software, it feels like it should be part of Windows <3

github-actions[bot] commented 3 weeks ago

Hello there! As you are a first time Issuer please read our Code of Conduct. Please also check whether your issue description is reasonable and complete. If you are having an issue always include the version you are using (WSL, this app, Windows) as well as a way to reproduce it.

RedyAu commented 3 weeks ago

https://github.com/bostrot/wsl2-distro-manager/blob/4572f4e6c7ca8e7df9874d3af394b65ee731089b/lib/api/wsl.dart#L205

RedyAu commented 3 weeks ago

@bostrot I just realized, that this is Flutter :P Might be able to submit a quick patch!