-
Notifications
You must be signed in to change notification settings - Fork 400
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suspect we would like to add the arm build to build.yml
as well (for master branch).
Yes, we would want to add the arm build to the |
This will be a godsend since Intel binaries under Rosetta 2 are currently eating ram A LOT... |
I've managed to run Bitwarden as arm64 build locally and based on light testing, there's no apparent breakage on my M1 MBP. Here's how I do it: The resulting build is arm64 only unfortunately, I tried setting it to universal but CMIIW electron hasn't supported that yet. |
I have been unable to get it to compile on my M1 due to some dependencies missing, mainly LZMA-native. I can't compile it any further.
|
You may want to try building everything from source
|
It still fails.
|
@lbibass Try compiling with Node 14. The last error log only provides a stacktrace of node-gyp calling make, there are probably more detailed errors somewhere in the build logs which should help you trace down the root cause. |
I managed to get it working, but now it fails at notarizing. Any ideas on how to get around that? |
It depends what error you're getting, however you likely won't be able to notarize it because your machine identifier and Apple developer profile/certificates are not authorized to do so for our package identifier (which is a good thing 😉 ). Are you getting a technical error or authorization error? You could also locally modify the build action to exclude notarization. You can also use the |
I've now done the masdev build, but now I get this error when trying to open the built app. |
And if you use an |
To sign and run The easiest way to just run it with |
Any update from the team on this PR? Anything blocking from preventing this to be work on? |
The app store already has a Universal Binary |
Hi @nitrocode, Since 1.30 we now publish a universal dmg build on GitHub and our website. I believe this solves the original issue raised in this PR, and I will therefore close it. Thanks for the contribution, and apologies that it took a while for us to address this. |
Original PR #793 and ticket #792
In the original PR, I added
--universal
topack:mac:mas
andpack:mac:masdev
and added targetpack:mac:arm64
but I forgot to publish these binaries.After looking at the windows arm pr #667, it looks like updating the
release.yml
and maybe thebuild.yml
github actions ?Would appreciate help on this if time permits.
cc: @Hinton @cscharf