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

Use bazel-ios/rules_apple #634

Merged
merged 1 commit into from
Jan 26, 2023
Merged

Conversation

dostrander
Copy link
Collaborator

Uses the bazel-ios/rules_apple fork which has updates that need to be upstreamed before we can go back to using bazel's rules_apple

@mattrobmattrob
Copy link
Collaborator

Might be useful to get #633 merged first. Not sure how much you're pulling the rules_apple reference forward in this one.

@jerrymarino
Copy link
Contributor

@mattrobmattrob are you planning to push forward 2.0 sometime soon?

I think you are right - there is probably some breaking changes in this sha if we just pull HEAD we might need to roll it back to the current version.

@mattrobmattrob
Copy link
Collaborator

@mattrobmattrob are you planning to push forward 2.0 sometime soon?

I think you are right - there is probably some breaking changes in this sha if we just pull HEAD we might need to roll it back to the current version.

Yeah, likely switching back to focus on it tomorrow. Can make it my priority to unblock y'all.

@jerrymarino
Copy link
Contributor

Cool thanks a lot! We can also roll back our fork as well / ensure this commit just advances on the current rules_apple sha in the repo - to give the 2.0 bump a bit more breathing room / time

@dostrander dostrander force-pushed the dostrander/use-bazel-ios-rules_apple branch from d94b411 to dd76183 Compare January 4, 2023 22:16
@jerrymarino
Copy link
Contributor

INFO: From Testing //tests/ios/frameworks/mixed-source/custom-module-map:CustomModuleMapTests:
                          [--launch_options_json_path LAUNCH_OPTIONS_JSON_PATH]
                          [--signing_options_json_path SIGNING_OPTIONS_JSON_PATH]
                          [--test_type TEST_TYPE] [--work_dir WORK_DIR]
                          [--output_dir OUTPUT_DIR]
                          {prepare,test,simulator_test} ...
ios_test_runner.py: error: unrecognized arguments: --product_module_name=CustomModuleMapTests

That looks like a legit failure ☝️

@dostrander dostrander force-pushed the dostrander/use-bazel-ios-rules_apple branch 2 times, most recently from 4e7a677 to a3e1e3c Compare January 5, 2023 22:37
Copy link
Contributor

@jerrymarino jerrymarino left a comment

Choose a reason for hiding this comment

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

LGTM: @mattrobmattrob once you bump to 2.0.0 we can take care of rebasing the fork for our test sharding fixes

@mattrobmattrob
Copy link
Collaborator

LGTM: @mattrobmattrob once you bump to 2.0.0 we can take care of rebasing the fork for our test sharding fixes

FWIW, #633 is ready now. 👍

@dostrander dostrander force-pushed the dostrander/use-bazel-ios-rules_apple branch 4 times, most recently from a524e94 to fec4827 Compare January 12, 2023 14:11
@dostrander dostrander force-pushed the dostrander/use-bazel-ios-rules_apple branch from fec4827 to 1923e27 Compare January 26, 2023 21:17
@jerrymarino jerrymarino enabled auto-merge (squash) January 26, 2023 21:23
@jerrymarino jerrymarino merged commit 985d578 into master Jan 26, 2023
@jerrymarino jerrymarino deleted the dostrander/use-bazel-ios-rules_apple branch January 26, 2023 22:00
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.

4 participants