Skip to content

Configure Apple crosstool to return a complete target triple that includes minimum OS version and target environment#14898

Closed
thii wants to merge 1 commit intobazelbuild:masterfrom
thii:configure-apple-crosstool-to-return-a-complete-target-triple-that
Closed

Configure Apple crosstool to return a complete target triple that includes minimum OS version and target environment#14898
thii wants to merge 1 commit intobazelbuild:masterfrom
thii:configure-apple-crosstool-to-return-a-complete-target-triple-that

Conversation

@thii
Copy link
Copy Markdown
Member

@thii thii commented Feb 24, 2022

This is required for bazelbuild/rules_swift#768

includes minimum OS version and target environment

This is required for bazelbuild/rules_swift#768
Comment thread tools/osx/crosstool/cc_toolchain_config.bzl
@thii thii marked this pull request as ready for review February 24, 2022 07:01
Comment thread tools/osx/crosstool/cc_toolchain_config.bzl
Copy link
Copy Markdown
Member

@keith keith left a comment

Choose a reason for hiding this comment

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

this matches what xcode does today

@keith
Copy link
Copy Markdown
Member

keith commented Mar 3, 2022

cc @oquenchil can you review?

@oquenchil oquenchil added platform: apple team-Rules-ObjC Issues for Objective-C maintainers labels Mar 10, 2022
@oquenchil oquenchil requested a review from googlewalt March 10, 2022 15:30
@thii
Copy link
Copy Markdown
Member Author

thii commented Mar 24, 2022

@googlewalt Can you review?

Comment thread tools/osx/crosstool/cc_toolchain_config.bzl
@bazel-io bazel-io closed this in 3442179 Mar 28, 2022
@brentleyjones
Copy link
Copy Markdown
Contributor

@bazel-io fork 5.2

@thii thii deleted the configure-apple-crosstool-to-return-a-complete-target-triple-that branch March 29, 2022 00:35
thii added a commit to thii/bazel that referenced this pull request Apr 15, 2022
…ludes minimum OS version and target environment

This is required for bazelbuild/rules_swift#768

Closes bazelbuild#14898.

PiperOrigin-RevId: 437770046
(cherry picked from commit 3442179)
thii added a commit to thii/bazel that referenced this pull request Apr 18, 2022
…ludes minimum OS version and target environment

This is required for bazelbuild/rules_swift#768

Closes bazelbuild#14898.

PiperOrigin-RevId: 437770046
(cherry picked from commit 3442179)
ckolli5 pushed a commit that referenced this pull request May 9, 2022
…ludes minimum OS version and target environment (#15266)

This is required for bazelbuild/rules_swift#768

Closes #14898.

PiperOrigin-RevId: 437770046
(cherry picked from commit 3442179)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

platform: apple team-Rules-ObjC Issues for Objective-C maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants