GitLiveApp / firebase-kotlin-sdk

A Kotlin-first SDK for Firebase
https://gitliveapp.github.io/firebase-kotlin-sdk/
Apache License 2.0
1.17k stars 155 forks source link

chore(deps): update dependency firebase/firebase-ios-sdk to from: `10.23.0` #481

Closed tomu28 closed 7 months ago

tomu28 commented 7 months ago

update dependency firebase/firebase-ios-sdk to from: 10.23.0


Resolves:

nbransby commented 7 months ago

Hi @tomu28, the ios build appears to fail with Error: Exception in thread "main" org.jetbrains.kotlin.native.interop.gen.CInteropPrettyException: It seems that library is using clang modules. Try adding-compiler-option -fmodulesto cinterop. Do you get the same result locally?

tomu28 commented 7 months ago

@nbransby I apologize for the delay in confirming this locally. 🙇

Environment being used:

As of commit 33956f549e380b732da84ec1eba7a8f9ce9ed615, the following error occurred locally as well:

> Task :firebase-crashlytics:cinteropFirebaseCrashlyticsIosX64
Exception in thread "main" org.jetbrains.kotlin.native.interop.gen.CInteropPrettyException: It seems that library is using clang modules. Try adding `-compiler-option -fmodules` to cinterop.
For example, in case of cocoapods plugin:

pod("PodName") {
    // Add these lines
    extraOpts += listOf("-compiler-option", "-fmodules")
}
    at org.jetbrains.kotlin.native.interop.gen.ClangModulesDisabledPrettifier.prettify(ExceptionPrettifiers.kt:36)
    at org.jetbrains.kotlin.native.interop.gen.jvm.MainKt.processCLib(main.kt:713)
    at org.jetbrains.kotlin.native.interop.gen.jvm.MainKt.processCLibSafe(main.kt:243)
    at org.jetbrains.kotlin.native.interop.gen.jvm.MainKt.access$processCLibSafe(main.kt:1)
    at org.jetbrains.kotlin.native.interop.gen.jvm.Interop.interop(main.kt:101)
    at org.jetbrains.kotlin.cli.utilities.InteropCompilerKt.invokeInterop(InteropCompiler.kt:47)
    at org.jetbrains.kotlin.cli.utilities.MainKt.mainImpl(main.kt:23)
    at org.jetbrains.kotlin.cli.utilities.MainKt.main(main.kt:45)

After adding the specification of extraOpts += listOf("-compiler-option", "-fmodules") as in commit 46df8f2dfbccf027d9a4a4a91983766f93eaf0cf, I have confirmed that the issue has been resolved.

Thank you for making the additional commit!