Skip to content

Commit a2320db

Browse files
authored
Fix analytics warnings building for macOS/tvOS with SPM (#9030)
1 parent 276e35f commit a2320db

File tree

3 files changed

+5
-4
lines changed
  • .github/workflows
  • SwiftPM-PlatformExclude
    • FirebaseAnalyticsWithoutAdIdSupportWrap
    • FirebaseAnalyticsWrap

3 files changed

+5
-4
lines changed

.github/workflows/spm.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ on:
88
- '.swiftpm/*'
99
- 'scripts/build.sh'
1010
- 'SwiftPMTests/*'
11+
- 'SwiftPM-PlatformExclude'
1112
- 'Gemfile*'
1213
schedule:
1314
# Run every day at 12am (PST) - cron uses UTC times

SwiftPM-PlatformExclude/FirebaseAnalyticsWithoutAdIdSupportWrap/dummy.m

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,6 @@
1313
// limitations under the License.
1414

1515
#import <TargetConditionals.h>
16-
#if !TARGET_OS_IOS
17-
#warning "Firebase Analytics only supports the iOS platform"
16+
#if TARGET_OS_WATCH
17+
#warning "Firebase Analytics does not support the watchOS platform"
1818
#endif

SwiftPM-PlatformExclude/FirebaseAnalyticsWrap/dummy.m

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,6 @@
1313
// limitations under the License.
1414

1515
#import <TargetConditionals.h>
16-
#if !TARGET_OS_IOS
17-
#warning "Firebase Analytics only supports the iOS platform"
16+
#if TARGET_OS_WATCH
17+
#warning "Firebase Analytics does not support the watchOS platform"
1818
#endif

0 commit comments

Comments
 (0)