-
Notifications
You must be signed in to change notification settings - Fork 1.6k
ARM64 Simulator support #6520
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
Comments
@ryanwilson could you help to prioritize this issue? It can be consistently reproduced, even if the version is
|
Hi @imWildCat - thanks for the report. This is definitely top of mind as we're expecting the first Apple silicon Macs to be announced next week, the plan is to have something ASAP and included in the next release or two (likely not next release though at this point). |
@ryanwilson thanks for your prompt reply! Really appreciate your efforts! |
@bcutler94 Great, thank you! Update: this looks like a workaround? |
@ryanwilson sorry if this is slightly off topic for this issue, but do your current plans include support for arm64 for Mac Catalyst apps too? Happy to open a new issue if that would help to track this. |
Same issue here with the new M1 Silicon MacBook. Hope there will be a fix soon |
I have a PR open (#6950) that should land today and we're going to try to package a release as soon as we can for support. I also have an M1 Mac hopefully arriving today that I can test with. Thanks for your patience while we get this resolved! |
Same here, stackOverflow fix didn't work on M1 Mac |
I realize that this is off topic, but does anyone know if GoogleSignIn will get a similar treatment soon? https://developers.google.com/identity/sign-in/ios/sdk |
@asowers1 We've had conversations with the |
Any update on this fix? |
We're were hoping to get it out this week, but ran into a CocoaPods bug that will likely force us to split the Arm64 simulator support into a separate CocoaPod that will probably push it until sometime next week. |
Thanks @paulb777, hopefully CocoaPods will merge the PR soon! |
Yep, hopefully the fix will land soon. Either way, we're planning to do a workaround with product structure and documentation in the meantime and to avoid requiring all Firebase users to update their CocoaPods version. |
I just want to point out that (at least in my case) I had to add Previously, I was using this workaround by just adding it to my main app project, but that wasn't enough now that I'm on an Apple Silicon Mac. Maybe this is obvious, but I wanted to point it out in case anyone else is having the same issue. |
Thanks, that really helped a lot and quite a few Xcode errors disappeared now, but iOS 14 Widgets still do not preview in the canvas on an M1 - Xcode says
Maybe this is not directly related. Those cryptic Xcode error message really don't help at all. |
Adding arm64 to excluded architectures causes conflicts with other Pods I have on my project, so a fix for Firebase is still very much needed.
|
Hi everyone, Thanks for your patience. We ran into a few hiccups along the way, but just published a version of Analytics and Perf that are compatible with Apple Silicon. Perf: update to I'll close this now but if you run into issues with the instructions or running on Apple silicon machines, please file a new bug! |
Agh, looks like the instructions don't quite work as expected for locking to minor versions. I'll get that addressed in the morning. In the meantime, you can lock to the exact version (remove Sorry about that! |
Thank you! That did the job @ryanwilson |
After following the instructions above I am now getting a build error:
|
My project which has Firebase dependencies runs fine on a physical device. To make it run in the simulator, I have to run Xcode with Rosetta 2. |
I'm having the same issue on M1 with Google Sign In. |
I'm unable to reproduce this, can you try cleaning your build folder and trying again? If you can reliably reproduce this, let me know how and I'll see if I can do the same. |
Sorry for the troubles - are you using CocoaPods? If so, did you try the new instructions? If you're still having issues after trying that out, please let us know and we can help further. |
Thanks for the report, Google Sign In still hasn't migrated to XCFrameworks. I'll reach out to the team again to discuss migrating. |
Thank you! |
No luck. I cleaned the build folder, deleted DerivedData, build: same error. I then cleaned the build folder again, deleted DerivedData, closed Xcode. I ran |
|
That was it. Many thanks! :o) |
Maybe a stupid question, but: now that v7.2.0 adds |
Thanks for the hint. yes, I'm using cocoapods. However I can't open that new instructions link, Safari is just showing an empty page. In the meantime, I set up a test project to see if it builds without Rosetta 2 on M1. And it finally did build. Below is the content of my pod file. Important: I did not include the post install command adding the arm64 exclusion for iOS Simulator for each pod. And I did neither add it to the pods project nor to my test project. That seems logical - maybe I misunderstood some remarks in this issue thread. Anyway, thanks for your quick reaction publishing these M1 updates. platform :ios, '14.0' |
yes, that is exactly my experience. You have to remove exclude arm64 for iOS Simulator: not only for each pod but also for your project and the pods project. See my post above. And it makes sense because now iOS simulator can basically run like on the iPhone with the arm64 instructions. |
Where are these new instructions? That link is broken. |
@randyhill apologies, here's the full link: https://github.com/firebase/firebase-ios-sdk/blob/master/AppleSilicon.md |
With SPM same issue, I also tried removing package and re-adding. I'm not able to use ARM64 due to another package breaking. FYI, If I try using 7.2.0-M1 then I get an error installing the package when I get to analytics (because this wasn't in that branch. FYI, if you open Xcode under rosetta things work, obviously slower |
@ryanwilson What's the best way to track status of Apple Silicon Simulator support for the Google Sign-In SDK? |
Find Xcode in Finder and press CMD+I and select the application to open with Rosetta 2. I did the same for my Terminal. This fixed all my issues with my new Mac mini M1 chip! |
This doesn't fix the issue :), its a work around I mentioned above, but this makes Xcode run much slower. For me, I have the beta Xcode running as native M1, and prod Xcode running with Rosetta (I have disabled analytics in most of my projects except one that requires it) |
@jeremyfrancis @darrenpedley No work arounds should be necessary if you follow the instructions at https://github.com/firebase/firebase-ios-sdk/blob/master/AppleSilicon.md |
@paulb777 I have followed the instructions referenced, and get the same issue as described, the work around listed is the only way to compile. FYI I'm using: Analytics causes the error, using excluded arm64 allows compilation, however, ZIP breaks. The workaround above (i.e. using Rosetta) allows compilation without using excluded arm64. |
@darrenpedley What does ZIP 2.1.1 mean? The Analytics included in 7.3.1 includes a slice for the arm64 simulator? Are you trying Catalyst? If so, that's a separate issue - #7043 |
@paulb777 Zip 2.1.1 is a different SPM package, that doesn't support the excluded Arm64, not related to Firebase, but . This is the error I get Undefined symbols for architecture arm64: If I remove Analytics, issue goes away, |
Hi, I'm pretty sure I deleted my DerivedData folder before but looks like cleaning that up, rebooting and clean build has resolved my issue, no longer have to start in Rosetta, etc. Thanks for feedback and assistance. |
Step 0: Are you in the right place?
file a Github issue.
with the
firebase
tag.google group.
of the above categories, reach out to
Firebase Support.
this repository, please delete this section.
[REQUIRED] Step 1: Describe your environment
Version 12.0 (12A7209)
CocoaPods
(select one)[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
Create a minimal demo project on arm Mac, cannot build this project.
Relevant Code:
None.
The text was updated successfully, but these errors were encountered: