Skip to content

Objective-C: update macOS and iOS versions to match grpc core config#553

Merged
chalin merged 1 commit intomainfrom
chalin-macos-ios-vers-2020-12-03
Jan 14, 2021
Merged

Objective-C: update macOS and iOS versions to match grpc core config#553
chalin merged 1 commit intomainfrom
chalin-macos-ios-vers-2020-12-03

Conversation

@chalin
Copy link
Copy Markdown
Collaborator

@chalin chalin commented Dec 3, 2020

Match verions of grpc/grpc#24282:

Supported platform changes:

  • iOS 9 or later (from 7)
  • OSX 10.10 or later (from 10.9)

@veblush @yulin-liang

@chalin chalin requested a review from jtattermusch December 3, 2020 17:17
compilers: [Node v4+]
- language: Objective-C
os: [Mac OS X 10.11+, iOS 7.0+]
os: [macOS 10.10+, iOS 9.0+]
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@veblush @jtattermusch @yulin-liang: notice how this used to be 10.11+ (rather than 10.9+). Hence, this PR downgrades the macOS version constraint to match the min version specified by grpc/grpc#24282. Just pointing this out to ensure that it is ok.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is weird to me because it's different from ones in the gRPC repository. I think having 10.10+ would be fine since our CI test are using that version.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This is weird to me because it's different from ones in the gRPC repository

Just let me know which versions (for macOS and iOS) you'd like to see in the docs, and I'll make the changes. I just used 10.10+ because that is what grpc/grpc#24282 seemed to suggest.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

10.10+ sounds reasonable to me!

@chalin chalin force-pushed the chalin-macos-ios-vers-2020-12-03 branch 3 times, most recently from b137b8e to c008830 Compare December 7, 2020 21:17
@chalin chalin force-pushed the chalin-macos-ios-vers-2020-12-03 branch from c008830 to 08009ed Compare January 14, 2021 15:16
@chalin
Copy link
Copy Markdown
Collaborator Author

chalin commented Jan 14, 2021

Thanks for the review @veblush

@chalin chalin merged commit df8761d into main Jan 14, 2021
@chalin chalin deleted the chalin-macos-ios-vers-2020-12-03 branch January 14, 2021 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants