Skip to content

Commit

Permalink
fix(ml_downloader): fixed an issue where macOS builds failed due to b…
Browse files Browse the repository at this point in the history
…ug with missing pod subspec in Firebase SDK (added a workaround until issue fixed upstream).
  • Loading branch information
Salakar committed Jan 10, 2022
1 parent 7bd14e0 commit acc6afa
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
Expand Up @@ -10,7 +10,6 @@

import FirebaseCore
import FirebaseMLModelDownloader
import UIKit

import firebase_core

Expand Down Expand Up @@ -78,7 +77,7 @@ public class FirebaseModelDownloaderPluginSwift: FLTFirebasePlugin, FlutterPlugi
andOptionalNSError: nil))
}

var result = FLTFirebaseMethodCallResult.create(success: result, andErrorBlock: errorBlock)
let result = FLTFirebaseMethodCallResult.create(success: result, andErrorBlock: errorBlock)
if call.method == "FirebaseModelDownloader#getModel" {
getModel(arguments: call.arguments as! [String: Any], result: result)
}
Expand Down
Expand Up @@ -52,7 +52,9 @@ Pod::Spec.new do |s|

# Firebase dependencies
s.dependency 'Firebase/CoreOnly', "~> #{firebase_sdk_version}"
s.dependency 'Firebase/MLModelDownloader', "~> #{firebase_sdk_version}"
# TODO(Salakar): Direct pod depency here instead of the Firebase subspec due to:
# https://github.com/firebase/firebase-ios-sdk/pull/9186 (pending acceptance / merge)
s.dependency 'FirebaseMLModelDownloader', '~> 8.10.0-beta'
s.static_framework = true
s.swift_version = '5.0'
s.pod_target_xcconfig = {
Expand Down

0 comments on commit acc6afa

Please sign in to comment.