Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ public let shared = Manifest(
version: "9.6.0",
pods: [
Pod("FirebaseSharedSwift"),
Pod("FirebaseCoreDiagnostics"),
Pod("FirebaseCoreInternal"),
Pod("FirebaseCore"),
Pod("FirebaseCoreExtension"),
Expand Down
4 changes: 2 additions & 2 deletions ReleaseTooling/Sources/ZipBuilder/CarthageUtils.swift
Original file line number Diff line number Diff line change
Expand Up @@ -72,9 +72,9 @@ extension CarthageUtils {
/// - packagedDir: The packaged directory assembled for the Carthage distribution.
/// - templateDir: The template project directory, contains the dummy Firebase library.
/// - jsonDir: Location of directory containing all JSON Carthage manifests.
/// - firebaseVersion: The version of the Firebase pod.
/// - coreDiagnosticsPath: The path to the Core Diagnostics framework built for Carthage.
/// - artifacts: Build artifacts.
/// - outputDir: The directory where all artifacts should be created.
/// - versionCheckEnabled: Checking if Carthage version already exists.

private static func generateCarthageRelease(fromPackagedDir packagedDir: URL,
templateDir: URL,
Expand Down
1 change: 0 additions & 1 deletion ReleaseTooling/Sources/ZipBuilder/CocoaPodUtils.swift
Original file line number Diff line number Diff line change
Expand Up @@ -530,7 +530,6 @@ enum CocoaPodUtils {
let podspecs = try! FileManager.default.contentsOfDirectory(atPath: localURL.path)
for podspec in podspecs {
if podspec == "FirebaseInstallations.podspec" ||
podspec == "FirebaseCoreDiagnostics.podspec" ||
podspec == "FirebaseCore.podspec" ||
podspec == "FirebaseCoreExtension.podspec" ||
podspec == "FirebaseCoreInternal.podspec" ||
Expand Down
4 changes: 2 additions & 2 deletions ReleaseTooling/Sources/ZipBuilder/FrameworkBuilder.swift
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ struct FrameworkBuilder {
/// Build all thin slices for an open source pod.
/// - Parameter framework: The name of the framework to be built.
/// - Parameter logsDir: The path to the directory to place build logs.
/// - Parameter setCarthage: Set Carthage flag in CoreDiagnostics for metrics.
/// - Parameter setCarthage: Set Carthage flag in GoogleUtilities for metrics.
/// - Returns: A dictionary of URLs to the built thin libraries keyed by platform.
private func buildFrameworksForAllPlatforms(withName framework: String,
logsDir: URL,
Expand All @@ -173,7 +173,7 @@ struct FrameworkBuilder {
/// - targetPlatform: The target platform to target for the build.
/// - buildDir: Location where the project should be built.
/// - logRoot: Root directory where all logs should be written.
/// - setCarthage: Set Carthage flag in CoreDiagnostics for metrics.
/// - setCarthage: Set Carthage flag in GoogleUtilities for metrics.
/// - Returns: A URL to the framework that was built.
private func buildSlicedFramework(withName framework: String,
targetPlatform: TargetPlatform,
Expand Down
22 changes: 11 additions & 11 deletions ReleaseTooling/Sources/ZipBuilder/ZipBuilder.swift
Original file line number Diff line number Diff line change
Expand Up @@ -179,7 +179,7 @@ struct ZipBuilder {
// the folders in each product directory.
let linkage: CocoaPodUtils.LinkageType = dynamicFrameworks ? .dynamic : .standardStatic
var groupedFrameworks: [String: [URL]] = [:]
var carthageCoreDiagnosticsFrameworks: [URL] = []
var carthageGoogleUtilitiesFrameworks: [URL] = []
var podsBuilt: [String: CocoaPodUtils.PodInfo] = [:]
var xcframeworks: [String: [URL]] = [:]
var resources: [String: URL] = [:]
Expand Down Expand Up @@ -241,13 +241,13 @@ struct ZipBuilder {
podInfo: podInfo)
groupedFrameworks[podName] = (groupedFrameworks[podName] ?? []) + frameworks

if includeCarthage, podName == "FirebaseCoreDiagnostics" {
if includeCarthage, podName == "GoogleUtilities" {
let (cdFrameworks, _) = builder.compileFrameworkAndResources(withName: podName,
logsOutputDir: paths
.logsOutputDir,
setCarthage: true,
podInfo: podInfo)
carthageCoreDiagnosticsFrameworks += cdFrameworks
carthageGoogleUtilitiesFrameworks += cdFrameworks
}
if resourceContents != nil {
resources[podName] = resourceContents
Expand Down Expand Up @@ -299,13 +299,13 @@ struct ZipBuilder {
fatalError("Could not create XCFrameworks Carthage directory: \(error)")
}

let carthageCoreDiagnosticsXcframework = FrameworkBuilder.makeXCFramework(
withName: "FirebaseCoreDiagnostics",
frameworks: carthageCoreDiagnosticsFrameworks,
let carthageGoogleUtilitiesXcframework = FrameworkBuilder.makeXCFramework(
withName: "GoogleUtilities",
frameworks: carthageGoogleUtilitiesFrameworks,
xcframeworksDir: xcframeworksCarthageDir,
resourceContents: nil
)
return (podsBuilt, xcframeworks, carthageCoreDiagnosticsXcframework)
return (podsBuilt, xcframeworks, carthageGoogleUtilitiesXcframework)
}

/// Try to build and package the contents of the Zip file. This will throw an error as soon as it
Expand Down Expand Up @@ -333,7 +333,7 @@ struct ZipBuilder {
platforms: ["ios"]))

print("Final expected versions for the Zip file: \(podsToInstall)")
let (installedPods, frameworks, carthageCoreDiagnosticsXcframeworkFirebase) =
let (installedPods, frameworks, carthageGoogleUtilitiesXcframeworkFirebase) =
buildAndAssembleZip(podsToInstall: podsToInstall,
includeCarthage: true,
// Always include dependencies for Firebase zips.
Expand All @@ -346,8 +346,8 @@ struct ZipBuilder {
"installed: \(installedPods)")
}

guard let carthageCoreDiagnosticsXcframework = carthageCoreDiagnosticsXcframeworkFirebase else {
fatalError("CoreDiagnosticsXcframework is missing")
guard let carthageGoogleUtilitiesXcframework = carthageGoogleUtilitiesXcframeworkFirebase else {
fatalError("GoogleUtilitiesXcframework is missing")
}

let zipDir = try assembleDistributions(withPackageKind: "Firebase",
Expand All @@ -357,7 +357,7 @@ struct ZipBuilder {
firebasePod: firebasePod)
// Replace Core Diagnostics
var carthageFrameworks = frameworks
carthageFrameworks["FirebaseCoreDiagnostics"] = [carthageCoreDiagnosticsXcframework]
carthageFrameworks["GoogleUtilities"] = [carthageGoogleUtilitiesXcframework]
let carthageDir = try assembleDistributions(withPackageKind: "CarthageFirebase",
podsToInstall: podsToInstall,
installedPods: installedPods,
Expand Down