Firebase Performance warns me to add Analytics, which is not available on Catalyst

[steipete]

[REQUIRED] Step 1: Describe your environment

• Xcode version: 12.4
• Firebase SDK version: 7.8
• Installation method: Zip file
• Firebase Component: Performance

[REQUIRED] Step 2: Describe the problem

When building, I see a warning that asks me to add Firebase Analytics when using Performance, however we build for Catalyst and Analytics is not available there.

Seems related to #4563

Steps to reproduce:

Integrate from Zip. Add Performance. Archive Catalyst app.

[steipete]

Side note - we can finally ship Crashlytics for our Catalyst app, super excited about this. Thanks for handling my last bug report so quickly!

[paulb777]

@steipete Thanks for the report. This issue - #7487 - was just fixed in the 7.8.0 release. Did you update the Firebase.h from the the 7.8.0 zip download?

[steipete]

I indeed forgot to update it, but updating didn't resolve the issue.

If I look at the header it's clear why. See the selected part for the fix:

[steipete]

Ah - or am I responsible for setting FIREBASE_ANALYTICS_SUPPRESS_WARNING somewhere?

[paulb777]

Oops! Thanks for pointing that out. We'll get it updated for the next release.

FIREBASE_ANALYTICS_SUPPRESS_WARNING should only be needed by iOS Remote Config users wanting to suppress the warning.

We'll need to discuss with the Performance and RemoteConfig teams the right fix for iOS since Performance doesn't need Analytics, but its dependency RemoteConfig might want the warning? But either way, there shouldn't be the need for Catalyst and your suggestion should be applied.

[steipete]

Thanks for looking into this! I'd also take a Catalyst build of Analytics, would make things simpler :)

[paulb777]

Sent #7719 to fix and reopened #7487 for the related iOS issue