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

Wails doctor failing without error description #1458

Closed
DazedNConfused- opened this issue Jun 16, 2022 · 5 comments
Closed

Wails doctor failing without error description #1458

DazedNConfused- opened this issue Jun 16, 2022 · 5 comments
Labels
awaiting feedback More information is required from the requestor Bug Something isn't working

Comments

@DazedNConfused-
Copy link

Description

Wails doctor fails to run its diagnostics without any sort of message regarding what caused it to fail:

image

To Reproduce

  1. Download latest beta build (v2.0.0-beta.37 at the time of writing this) by means of go install github.com/wailsapp/wails/v2/cmd/wails@latest
  2. Run wails doctor

Expected behaviour

Expected to see a detailed list of the missing dependencies needed for Wails development.

Screenshots

No response

Attempted Fixes

No response

System Details

I cannot properly fill out this section because this is the very command that is failing.

Additional context

I am running Linux Mint 19.3

Here is the output of uname -vrm

5.4.0-117-generic #132~18.04.1-Ubuntu SMP Thu Jun 2 23:36:48 UTC 2022 x86_64
@DazedNConfused- DazedNConfused- added the Bug Something isn't working label Jun 16, 2022
@abtin
Copy link
Contributor

abtin commented Jun 17, 2022

I noticed the same problem on Linux Mint (20.3), but I don't think it's related to Mint. It's just the apt command syntax. Basically deep in the code we are running apt -qq list instead of apt list -qq and that produces apt usage help and error code 1. I'm going to submit a PR for the fix.

abtin added a commit to abtin/wails that referenced this issue Jun 17, 2022
@leaanthony
Copy link
Member

Thanks for tracking that down. Did that syntax change recently? It was developed on Ubuntu 20.10 and appeared to work at the time.

@leaanthony
Copy link
Member

@DazedNConfused- please let us know if master works for you now 🙏

@leaanthony leaanthony added awaiting feedback More information is required from the requestor Fixed on Master labels Jun 18, 2022
@abtin
Copy link
Contributor

abtin commented Jun 21, 2022

Thanks for tracking that down. Did that syntax change recently? It was developed on Ubuntu 20.10 and appeared to work at the time.

That is a great question. Linux mint is more strict and requires the options after the command. However, ubuntu and debian are more lax and accept the command before or after the options.

@DazedNConfused-
Copy link
Author

Thanks for the fix @abtin! Indeed, master works flawlessly now 🚀

image

Once again, thank you all for the analysis and the expeditious patch. You may go ahead and close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting feedback More information is required from the requestor Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants