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

Fix apt command syntax (#1458) #1461

Merged
merged 1 commit into from
Jun 17, 2022
Merged

Fix apt command syntax (#1458) #1461

merged 1 commit into from
Jun 17, 2022

Conversation

abtin
Copy link
Contributor

@abtin abtin commented Jun 17, 2022

A minor change to correct the order of parameters to the apt command.

Proof of fix:
Before:

$ wails doctor
Wails CLI v2.0.0-beta.37


Scanning system - Please wait (this may take a long time)...Failed.


ERROR: exit status 1

If Wails is useful to you or your company, please consider sponsoring the project:
https://github.com/sponsors/leaanthony

After:

Wails CLI v2.0.0-beta.37


Scanning system - Please wait (this may take a long time)...Done.

System
------
OS:		Linux Mint
Version: 	20.3
ID:		linuxmint
Go Version:	go1.18
Platform:	linux
Architecture:	amd64

Wails
------
Version: 		v2.0.0-beta.37
Package Manager: 	apt

Dependency	Package Name		Status		Version
----------	------------		------		-------
*docker		docker.io		Installed	20.10.17
gcc		build-essential		Installed	12.8ubuntu1.1
libgtk-3	libgtk-3-dev		Installed	3.24.20-0ubuntu1.1
libwebkit	libwebkit2gtk-4.0-dev	Available	2.36.3-0ubuntu0.20.04.1
npm		npm			Available	6.14.4+ds-1ubuntu2
*nsis		nsis			Available	3.05-2
pkg-config	pkg-config		Installed	0.29.1-0ubuntu4

* - Optional Dependency

Diagnosis
---------
Your system has missing dependencies!

Required package(s) installation details: 
  - libwebkit: sudo apt install libwebkit2gtk-4.0-dev
  - npm: sudo apt install npm

Optional package(s) installation details: 
  - nsis: sudo apt install nsis



If Wails is useful to you or your company, please consider sponsoring the project:
https://github.com/sponsors/leaanthony

@leaanthony leaanthony merged commit 9a01c75 into wailsapp:master Jun 17, 2022
@leaanthony
Copy link
Member

Thanks for this!! 🙏

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

Successfully merging this pull request may close these issues.

2 participants