-
Notifications
You must be signed in to change notification settings - Fork 3
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
Apple Silicon support #62
Comments
We will build another package for M1 when we release neuron? |
Yes, there will be two packages for MacOS as some apps do, e.g. Or, for Macs with Apple Silicon chips, click here to download on https://zoom.us/download The app could be packaged as a universal one but the size will be doubled because it simply includes one for intel and one for arm64. It might not be a good idea for users(but we can provide this version if possible) I'm using M1 so I can test the package. |
I can test the package also. |
@JeffreyMa597 Do you have time to try to finish it? |
@yanguoyu sure. |
How is it going |
@Keith-CY I've fixed this issue. I'll package the app on my machine and perform a local verification to verify whether the modification is successful. I earned two problems: 1、The appleId is not found. 2、I can't sign when I commit my code. I'm trying to resolve them. |
Cool, I would suggest proposing a draft PR first, so we can have a review. Besides, apple id is not so important to block this issue because we can verify it by CI. For the signature of commit, we can force push the PR once it's been fixed. |
Okay. |
Package for Apple Silicon: https://www.electronjs.org/blog/apple-silicon
Could have a try on building a universal app: electron-userland/electron-builder#5481Pay attention to a bug of auto update: electron-userland/electron-builder#5689 (comment)
The text was updated successfully, but these errors were encountered: