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

Update rules_ios to latest and enable bzlmod #2696

Merged
merged 1 commit into from
Oct 27, 2023

Conversation

luispadron
Copy link
Contributor

@luispadron luispadron commented Oct 25, 2023

Updates to use rules_ios 3.1.0 and enables bzlmod.

rules_ios is lacking support for Bazel > 6.x: bazel-ios/rules_ios#795

@luispadron
Copy link
Contributor Author

luispadron commented Oct 25, 2023

Current blocker:

Generating "rules_ios.xcodeproj"
ERROR: /Users/lpadron/Development/rules_xcodeproj/examples/rules_ios/bazel-output-base/rules_xcodeproj.noindex/build_output_base/external/rules_xcodeproj~override~internal~rules_xcodeproj_generated/generator/xcodeproj/BUILD:12:10: error loading package 'iOSApp': Unable to find package for @[unknown repo 'build_bazel_rules_apple' requested from @]//apple:versioning.bzl: The repository '@[unknown repo 'build_bazel_rules_apple' requested from @]' could not be resolved: No repository visible as '@build_bazel_rules_apple' from main repository. and referenced by '@rules_xcodeproj~override~internal~rules_xcodeproj_generated//generator/xcodeproj:xcodeproj'
ERROR: Analysis of target '@rules_xcodeproj~override~internal~rules_xcodeproj_generated//generator/xcodeproj:xcodeproj' failed; build aborted: Analysis failed

✅ fixed

@luispadron luispadron force-pushed the luis/update-rules_ios-use-bzlmod branch 7 times, most recently from 537d100 to 8358120 Compare October 26, 2023 20:59
@luispadron luispadron changed the title Update rules_ios to latest and bzlmod Update rules_ios to latest and enable bzlmod Oct 26, 2023
@luispadron
Copy link
Contributor Author

@brentleyjones Any tips for testing the failing job locally? I cant seem to get a failure here locally. The error is:

ERROR: /Users/administrator/buildbuddy.noindex/remote_build/df09ef84-c2fb-4a69-83e9-b4757b8a721a/repo-root/examples/rules_ios/iOSApp/Source/BUILD:32:26: //iOSApp/Source:iOSApp: no such attribute 'output_discriminator' in 'ios_application' rule
ERROR: /Users/administrator/buildbuddy.noindex/remote_build/df09ef84-c2fb-4a69-83e9-b4757b8a721a/repo-root/examples/rules_ios/bazel-output-base/rules_xcodeproj.noindex/build_output_base/external/rules_ios~3.1.0/rules/transition_support.bzl:125:36: transition inputs [//command_line_option:apple_compiler, //command_line_option:apple_grte_top] do not correspond to valid settings
ERROR: /Users/administrator/buildbuddy.noindex/remote_build/df09ef84-c2fb-4a69-83e9-b4757b8a721a/repo-root/examples/rules_ios/Lib/BUILD:30:26: Errors encountered while applying Starlark transition
ERROR: Analysis of target '@rules_xcodeproj~override~internal~rules_xcodeproj_generated//generator/test/fixtures/xcodeproj_bwb:xcodeproj_bwb' failed; build aborted: Analysis failed

@brentleyjones
Copy link
Contributor

I can look at this Monday.

@luispadron luispadron force-pushed the luis/update-rules_ios-use-bzlmod branch 2 times, most recently from eb925eb to 217179c Compare October 27, 2023 16:23
@luispadron luispadron marked this pull request as ready for review October 27, 2023 16:23
@luispadron luispadron requested review from a team as code owners October 27, 2023 16:23
@luispadron luispadron force-pushed the luis/update-rules_ios-use-bzlmod branch from 217179c to 265d86f Compare October 27, 2023 16:25
@brentleyjones
Copy link
Contributor

Per offline, this is good to go in as is (when others approve it).

Signed-off-by: Luis Padron <lpadron@squareup.com>
@luispadron luispadron force-pushed the luis/update-rules_ios-use-bzlmod branch from 265d86f to bac6f49 Compare October 27, 2023 16:31
@luispadron
Copy link
Contributor Author

So it now has two rules_ios CI jobs, both using rules_ios 3.1.0:

  • One with bzlmod enabled using Bazel 6.4.0
  • One without bzlmod enabled using Bazel 6.4.0

@brentleyjones brentleyjones merged commit 06366ad into main Oct 27, 2023
13 checks passed
@brentleyjones brentleyjones deleted the luis/update-rules_ios-use-bzlmod branch October 27, 2023 17:33
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