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

Conditionally handle --apple_compiler & --apple_grte_top #798

Merged
merged 2 commits into from
Oct 30, 2023

Conversation

mattrobmattrob
Copy link
Collaborator

@mattrobmattrob mattrobmattrob commented Oct 30, 2023

  • Create a new repo rule that contains the Bazel version for access in the analysis phase (rules_ios_bazel_version). This will just write the version from getattr(native, "bazel_version", “”) into a variable in @rules_ios_bazel_version//:version.bzl
  • Use the Bazel version to conditionally handle the (now removed in Bazel 7) --apple_compiler & --apple_grte_top command line options.

Work towards #795.

- Create a new repo rule that contains the Bazel version for access in the analysis phase (`rules_ios_bazel_version`). This will just write the version from `getattr(native, "bazel_version", “”)` into a variable in `@rules_ios_bazel_version//:version.bzl`
- Use the Bazel version to conditionally handle the (now removed in Bazel 7) `--apple_compiler` & `--apple_grte_top` command line options.

Signed-off-by: Matt Robinson <mattrob@hey.com>
Signed-off-by: Matt Robinson <mattrob@hey.com>
@mattrobmattrob mattrobmattrob marked this pull request as ready for review October 30, 2023 21:31
Copy link
Collaborator

@luispadron luispadron left a comment

Choose a reason for hiding this comment

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

Thanks!

@mattrobmattrob mattrobmattrob merged commit 5ca1255 into master Oct 30, 2023
9 checks passed
@mattrobmattrob mattrobmattrob deleted the mr/bazel7/handle.removed.command.line.options branch October 30, 2023 22:34
@jerrymarino
Copy link
Contributor

Nice one @mattrobmattrob 👍

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.

3 participants