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

Bump nw-builder to latest 3.8.5 supported version. #121

Merged
merged 2 commits into from
Dec 29, 2024

Conversation

andrericardo
Copy link
Contributor

Hello!

I'm trying to get the project to build in a version of NW.js that supports "Mac OS X (arm64)" such as https://nwjs.io/blog/v0.94.0/

For that to happen I believe I need to upgrade nw-builder as well, so this PR is a step into first fixing a change related to to detectCurrentPlatform being moved.

Location of detectCurrentPlatform moved, see:
https://github.com/nwutils/nw-builder/pull/564/files

Tested build with

gulp build  --platforms osx64

@vankasteelj
Copy link
Owner

Seems allright, thanks

@vankasteelj vankasteelj reopened this Dec 29, 2024
@vankasteelj
Copy link
Owner

Sorry, the button "merge & close" has changed to "close with comment" x)

@vankasteelj vankasteelj merged commit d4c31f6 into vankasteelj:master Dec 29, 2024
@andrericardo
Copy link
Contributor Author

andrericardo commented Dec 30, 2024

@vankasteelj thank you for having a look and merging.

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