Skip to content

[ObjC] Upgrade podspec xx.deployment_target#33091

Merged
HannahShiSFB merged 1 commit intogrpc:masterfrom
HannahShiSFB:upgrade-podspec-target
May 20, 2023
Merged

[ObjC] Upgrade podspec xx.deployment_target#33091
HannahShiSFB merged 1 commit intogrpc:masterfrom
HannahShiSFB:upgrade-podspec-target

Conversation

@HannahShiSFB
Copy link
Collaborator

@HannahShiSFB HannahShiSFB commented May 12, 2023

Upgrade apple platform deployment_target versions to fix the cocoapods push of BoringSSL-GRPC about the following error:

ld: file not found: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/arc/libarclite_macosx.a
clang: error: linker command failed with exit code 1 (use -v to see invocation)

ref: https://developer.apple.com/forums/thread/725300

This also aligns with the versions required by protobuf

ios.deployment_target = '10.0'
osx.deployment_target = '10.12'
tvos.deployment_target = '12.0'
watchos.deployment_target = '6.0'

@HannahShiSFB HannahShiSFB force-pushed the upgrade-podspec-target branch from c951b1e to fb74235 Compare May 12, 2023 15:12
@HannahShiSFB HannahShiSFB force-pushed the upgrade-podspec-target branch from fb74235 to bc4f9e6 Compare May 14, 2023 00:24
@HannahShiSFB HannahShiSFB force-pushed the upgrade-podspec-target branch from bc4f9e6 to efc9eae Compare May 17, 2023 02:47
@HannahShiSFB HannahShiSFB force-pushed the upgrade-podspec-target branch 2 times, most recently from d0017b8 to b03444a Compare May 18, 2023 01:46
@HannahShiSFB HannahShiSFB marked this pull request as ready for review May 18, 2023 05:01
@HannahShiSFB HannahShiSFB requested a review from sampajano May 18, 2023 05:02
Copy link
Contributor

@sampajano sampajano left a comment

Choose a reason for hiding this comment

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

Wow.. that's a lot of podspecs that we have..... 😂

Thanks so much for figuring this out, Hannah.. Great investigative work... 😄

@mikefan1991 FYI

@sampajano
Copy link
Contributor

@HannahShiSFB can you fix the "Mergeable" CI failure too? I think you can modify some labels and it should work :)

@HannahShiSFB HannahShiSFB force-pushed the upgrade-podspec-target branch from b03444a to 5cce36e Compare May 19, 2023 03:23
@HannahShiSFB HannahShiSFB enabled auto-merge (squash) May 20, 2023 20:37
@HannahShiSFB HannahShiSFB added the release notes: no Indicates if PR should not be in release notes label May 20, 2023
@HannahShiSFB HannahShiSFB merged commit 988c855 into grpc:master May 20, 2023
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label May 22, 2023
eugeneo pushed a commit to eugeneo/grpc that referenced this pull request Jun 1, 2023
Upgrade apple platform deployment_target versions to fix the cocoapods
push of BoringSSL-GRPC about the following error:
```
ld: file not found: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/arc/libarclite_macosx.a
clang: error: linker command failed with exit code 1 (use -v to see invocation)
```
ref: https://developer.apple.com/forums/thread/725300

This also aligns with the versions required by
[protobuf](protocolbuffers/protobuf#10652)

```
ios.deployment_target = '10.0'
osx.deployment_target = '10.12'
tvos.deployment_target = '12.0'
watchos.deployment_target = '6.0'
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bloat/none imported Specifies if the PR has been imported to the internal repository lang/c++ lang/core lang/ObjC per-call-memory/neutral per-channel-memory/neutral release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants