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

feat: add visionOS deployment target for Cocoapods #15917

Conversation

YoloMao
Copy link
Contributor

@YoloMao YoloMao commented Feb 22, 2024

add visionOS deployment target for Cocoapods

@thomasvl thomasvl added c++ objective-c 🅰️ safe for tests Mark a commit as safe to run presubmits over labels Feb 22, 2024
@github-actions github-actions bot removed the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Feb 22, 2024
Copy link
Contributor

@thomasvl thomasvl left a comment

Choose a reason for hiding this comment

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

Change for the ObjC podspec is fine

@@ -45,6 +45,7 @@ Pod::Spec.new do |s|
s.osx.deployment_target = '10.12'
s.tvos.deployment_target = '12.0'
s.watchos.deployment_target = '6.0'
s.visionos.deployment_target = '1.0'
Copy link
Contributor

Choose a reason for hiding this comment

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

When was visionOS support added to CocoaPods? i.e. - do we need to raise the s.cocoapods_version = '>= 1.12.0' line or will versions that old be ok with this directive?

ps - can you add back the blank line that was here to keep the deployment_target info grouped?

Copy link
Contributor Author

@YoloMao YoloMao Feb 22, 2024

Choose a reason for hiding this comment

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

added blank line, and I saw Kingfisher and SDWebImage have not limit cocoapods_version. Maybe it is not nessary, I will take a check about it if I have free time.

Copy link
Contributor

Choose a reason for hiding this comment

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

@paulb777 do you happen to know if one should to raise the cocoapods_version when adding a visionos.deployment_target value?

@paulb777
Copy link
Contributor

CocoaPods Support was introduced in 1.13.0 and there were fixes in 1.15.0 - so at least 1.13.0 and maybe 1.15.0.

https://github.com/CocoaPods/CocoaPods/blob/master/CHANGELOG.md

and there are still outstanding issues that may or may not be solved:

https://github.com/CocoaPods/CocoaPods/issues?q=is%3Aissue+is%3Aopen+visionos

Firebase requires Swift PM for visionOS support.

@mkruskal-google mkruskal-google added the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Feb 23, 2024
@github-actions github-actions bot removed the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Feb 23, 2024
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.

None yet

4 participants