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

[webviewloader] Use go implementation to retrieve the version of a fixed runtime #1790

Merged
merged 3 commits into from
Aug 26, 2022

Conversation

stffabi
Copy link
Collaborator

@stffabi stffabi commented Aug 25, 2022

Fixes #1569

…xed runtime

This fixes a problem with the go-winloader and using GetAvailableCoreWebView2BrowserVersionString

Fixes wailsapp#1569
@leaanthony
Copy link
Member

LGTM! Thanks for taking the time to do this 🙏

@leaanthony leaanthony marked this pull request as ready for review August 26, 2022 21:19
@leaanthony
Copy link
Member

Just checking you're happy for this to be merged?

@stffabi
Copy link
Collaborator Author

stffabi commented Aug 26, 2022

Yeah, I'm happy with the state of the PR and fine merging it. 👍

@stffabi
Copy link
Collaborator Author

stffabi commented Aug 26, 2022

Thanks for taking your time looking into it. 🙏

@leaanthony leaanthony merged commit 3632ef9 into wailsapp:master Aug 26, 2022
@stffabi stffabi deleted the feature/webview2-loader-cgo-fix branch August 27, 2022 06:20
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.

Crash when CGO_ENABLED=1 while using WebviewBrowserPath option.
3 participants