-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[Infra] Update Package.swift for Firebase 11 #13301
Conversation
Generated by 🚫 Danger |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Addressable by #13303 |
Another failure similar to the objc-import-test failure: |
Sorry I saw that in the review and thought I was missing something about how the objc tests would need to change but not the swift ones. |
|
Still trying to figure out why some test targets are running and others are failing to boot. |
hint:
and likewise for Analytics Still trying to determine if this is a bug or if the Package.swift structure should change following the switch to dylibs |
Latest failure is reproducible by opening Package.swift on this branch and testing the |
This may be more serious than I thought. Also reproducible when:
|
Narrowing down the issue: running |
It's likely that the wrapper targets we use for the binary targets are no longer necessary when using dylibs. I'm going to try:
|
No description provided.