Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Core] Fix iOS 12 spec lint failure from incomplete FirebaseCoreInternal implementation #13952

Merged
merged 14 commits into from
Oct 22, 2024

Conversation

ncooke3
Copy link
Member

@ncooke3 ncooke3 commented Oct 22, 2024

Skipping the version bump in FirebaseCoreExtension.podspec because the changed .m file is not listed in that spec's sources.

@paulb777
Copy link
Member

Also, probably worth touching Gemfile and making sure everything else builds ok

FirebaseCore/CHANGELOG.md Outdated Show resolved Hide resolved
ncooke3 added a commit that referenced this pull request Oct 22, 2024
@ncooke3 ncooke3 changed the title [Core] Add availability gate on implementation [Core] Fix iOS 12 spec lint failure from incomplete FirebaseCoreInternal implementation Oct 22, 2024
ncooke3 added a commit that referenced this pull request Oct 22, 2024
@ncooke3 ncooke3 requested a review from paulb777 October 22, 2024 21:34
Copy link
Member

@paulb777 paulb777 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the thorough fix!

FirebaseCore.podspec Show resolved Hide resolved
@ncooke3
Copy link
Member Author

ncooke3 commented Oct 22, 2024

Green CI on this PR. Merging and will create CocoaPods-11.4.2 tag at tip of release-11.4 branch.

@ncooke3 ncooke3 merged commit 1333530 into release-11.4 Oct 22, 2024
100 checks passed
@ncooke3 ncooke3 deleted the nc/hotfix-2 branch October 22, 2024 21:50
@ncooke3
Copy link
Member Author

ncooke3 commented Oct 22, 2024

Global CI running in #13955

@ncooke3
Copy link
Member Author

ncooke3 commented Oct 22, 2024

Tag created: https://github.com/firebase/firebase-ios-sdk/releases/tag/CocoaPods-11.4.2

Staging to SpecsStaging

@ncooke3
Copy link
Member Author

ncooke3 commented Oct 22, 2024

FirebaseCore 11.4.0 will float to pick up the new FirebaseCoreInternal 11.4.2. I confirmed this in an app with the following Podfile:

source 'https://github.com/firebase/SpecsStaging.git'
source 'https://cdn.cocoapods.org/'

platform :ios, '13.0'

target 'FlutterIssue' do
  # Comment the next line if you don't want to use dynamic frameworks
  use_frameworks!

  # Pods for FlutterIssue
  pod 'FirebaseAuth', '11.4.0'
  
  target 'FlutterIssueTests' do
    inherit! :search_paths
    # Pods for testing
  end

  target 'FlutterIssueUITests' do
    # Pods for testing
  end

end

While a public API in FirebaseCoreInternal changed, the signature matches the completion handler API used by FirebaseCore. Since there should be no other usage of the API, this shouldn't cause issue.

@paulb777
Copy link
Member

For completeness, would you also test adding pod 'FirebaseCore', '11.4.0' to the Podfile.

@ncooke3
Copy link
Member Author

ncooke3 commented Oct 22, 2024

Done. Looks good. Podfile.lock:

PODS:
  - FirebaseCore (11.4.0):
    - FirebaseCoreInternal (~> 11.0)
    - GoogleUtilities/Environment (~> 8.0)
    - GoogleUtilities/Logger (~> 8.0)
  - FirebaseCoreInternal (11.4.2):
    - "GoogleUtilities/NSData+zlib (~> 8.0)"
  - GoogleUtilities/Environment (8.0.2):
    - GoogleUtilities/Privacy
  - GoogleUtilities/Logger (8.0.2):
    - GoogleUtilities/Environment
    - GoogleUtilities/Privacy
  - "GoogleUtilities/NSData+zlib (8.0.2)":
    - GoogleUtilities/Privacy
  - GoogleUtilities/Privacy (8.0.2)

DEPENDENCIES:
  - FirebaseCore (= 11.4.0)

SPEC REPOS:
  https://github.com/firebase/SpecsStaging.git:
    - FirebaseCoreInternal
  trunk:
    - FirebaseCore
    - GoogleUtilities

SPEC CHECKSUMS:
  FirebaseCore: e0510f1523bc0eb21653cac00792e1e2bd6f1771
  FirebaseCoreInternal: 35731192cab10797b88411be84940d2beb33a238
  GoogleUtilities: 26a3abef001b6533cf678d3eb38fd3f614b7872d

PODFILE CHECKSUM: 3676495040c7d08dcf76b3612ae88622969a1669

COCOAPODS: 1.15.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants