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

ci: Linux build + update CI, Wails, Nodejs #4

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

aminya
Copy link

@aminya aminya commented May 17, 2023

Fixes #3

image

@aminya aminya changed the title ci: make the build and release cross-platform + update Wails and Node ci: Linux build + update CI, Wails, Nodejs May 17, 2023
@iDevelopThings
Copy link
Owner

Thank you for this! I couldn't get the build to work for linux, also had no way to test it! let me know when it's good to go :)

@aminya
Copy link
Author

aminya commented May 18, 2023

You're welcome!

I got a successful build after patching Wails. There was a crash happening during startup because of some empty key, which I have patched and am waiting for them to review.

The CI works, but I haven't tested the publishing part yet.

Might be easier if we merge this, and I upload the binary using the patched Wails so that we do not have to wait for them. Up to you

@aminya
Copy link
Author

aminya commented Jun 1, 2023

My fix for linux has merged to Wails
wailsapp/wails#2672

Once Wails releases a new version, I will update this PR

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.

Linux Builds
2 participants