Skip to content

Commit abe9e97

Browse files
authored
Remove Interop refs from ZipBuilder (#5779)
1 parent 900e7ed commit abe9e97

File tree

2 files changed

+8
-13
lines changed

2 files changed

+8
-13
lines changed

ZipBuilder/Sources/ZipBuilder/CocoaPodUtils.swift

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -432,16 +432,14 @@ enum CocoaPodUtils {
432432
}
433433

434434
// If we're using local pods, explicitly add FirebaseInstanceID, FirebaseInstallations,
435-
// the Interop pods, and any Google* podspecs if they exist and there are no
436-
// explicit versions in the Podfile. Note there are versions for local podspecs if we're doing
437-
// the secondary install for module map building.
435+
// and any Google* podspecs if they exist and there are no explicit versions in the Podfile.
436+
// Note there are versions for local podspecs if we're doing the secondary install for module
437+
// map building.
438438
if !versionsSpecified, let localURL = LaunchArgs.shared.localPodspecPath {
439439
let podspecs = try! FileManager.default.contentsOfDirectory(atPath: localURL.path)
440440
for podspec in podspecs {
441441
if (podspec == "FirebaseInstanceID.podspec" ||
442442
podspec == "FirebaseInstallations.podspec" ||
443-
podspec == "FirebaseAnalyticsInterop.podspec" ||
444-
podspec == "FirebaseAuthInterop.podspec" ||
445443
podspec == "FirebaseCoreDiagnostics.podspec") ||
446444
podspec.starts(with: "Google"), podspec.hasSuffix(".podspec") {
447445
let podName = podspec.replacingOccurrences(of: ".podspec", with: "")

ZipBuilder/Sources/ZipBuilder/ZipBuilder.swift

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -385,9 +385,8 @@ struct ZipBuilder {
385385
// Loop through each installedPod item and get the name so we can fetch the framework and copy
386386
// it to the destination directory.
387387
for podName in installedPods {
388-
// Skip the Firebase pod, any Interop pods, and specifically ignored frameworks.
388+
// Skip the Firebase pod and specifically ignored frameworks.
389389
guard podName != "Firebase",
390-
!podName.contains("Interop"),
391390
!podsToIgnore.contains(podName) else {
392391
continue
393392
}
@@ -557,9 +556,8 @@ struct ZipBuilder {
557556
.duplicateFrameworksToRemove(pod: podName))
558557

559558
let copiedFrameworks = namedFrameworks.filter {
560-
// Only return the frameworks that aren't contained in the "podsToIgnore" array, aren't an
561-
// interop framework (since they don't compile to frameworks), or the Firebase pod itself.
562-
!(podsToIgnore.contains($0) || $0.hasSuffix("Interop") || $0 == "Firebase")
559+
// Skip frameworks that aren't contained in the "podsToIgnore" array and the Firebase pod.
560+
!(podsToIgnore.contains($0) || $0 == "Firebase")
563561
}
564562

565563
return (productDir, copiedFrameworks)
@@ -688,9 +686,8 @@ struct ZipBuilder {
688686
for (podName, podInfo) in pods {
689687
var frameworks: [URL] = []
690688
var carthageFrameworks: [URL] = []
691-
// Ignore any Interop pods or the Firebase umbrella pod.
692-
guard !podName.contains("Interop"),
693-
podName != "Firebase" else {
689+
// Ignore the Firebase umbrella pod.
690+
guard podName != "Firebase" else {
694691
continue
695692
}
696693

0 commit comments

Comments
 (0)