-
Notifications
You must be signed in to change notification settings - Fork 630
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 the build system for the Apple Silicon macOS build #2762
Comments
Now, there has two builds, osx64 & arm64, and haven't the universal build, may be supported in the future after the chromium and nodejs release the same. |
Hi, has there been any progress on this task? |
I'm totally into this!!!!!!!! fingers crossed! |
So.. the way the macOS arm64 build was included in the release of version 0.5.0 was by basically tricking our workflows/build system to think that it was building the x64 one (renaming the NW.js sdk, yes I know..) then manually attaching the output to the release. Obviously this cannot remain like this so changing the title to fit the issue and tagging this as a bug (even if its with the build system and our current junky solution produces the exact same .zip & .pkg's). Hopefully someone can help fix the build files, gulpfile etc to process this the same way they do with everything else. (also no arm64 build for NW.js 0.44.5 so any solution will have to take that into account too) P.s the current NW.js 0.82.0 location that's in the gulpfile includes the arm64 sdk, also patched with the ffmpeg prebuild libraries like the rest of the sdks there so that's good to go. |
And "xattr -cr /path/to/application.app" before you open .. for now? Edit: or as I've since found ... use a Download Manager (doesn't add flags to .zip file). |
I went down the rabbit hole, so the version of the nw-builder that is associated to this build, cannot detect arm64 macs, it detects them as osx32 and that doesn't build because nwbuilder defaults to v0.82.0, not taking the osx32 into account, that's easy to do, a simple change in the gulp file to fix the build for osx32. the bigger problem is that, nwbuilder 3.7.4 does not even recognize arm64 as a platform, here's the old one's page |
You must have been using the non-arm64 version before? Both Popcorntime 0.5.0 and 0.5.1 (Silicon) versions show that warning notice? A Silicon app package has to be signed/notarised (using an Apple developer account), or it throws a flag when it downloads .. and Gatekeeper won't pass it. It shows this "warning" notice to deter users from using unsigned apps. Just strip the quarantine flag/s .. if you want to continue? |
run this in terminal to remove the quarantine flag example: |
Don't put Just put a space after |
Now that NW.js supports Apple Silicon, it would be great if Popcorn Time could have a macOS universal build.
The text was updated successfully, but these errors were encountered: