Skip to content

Allow discovering host Xcodes with vendored BUILD - #1878

Open
keith wants to merge 1 commit into
mainfrom
ks/allow-discovering-host-xcodes-with-vendored-build
Open

Allow discovering host Xcodes with vendored BUILD#1878
keith wants to merge 1 commit into
mainfrom
ks/allow-discovering-host-xcodes-with-vendored-build

Conversation

@keith

@keith keith commented Jul 31, 2026

Copy link
Copy Markdown
Member

Previously when you vendored an explicit modules BUILD file, this would
disable all scanning locally. This is still the default behavior but now
you can set an env var to also discover local Xcodes. This is useful for
2 cases:

  1. If you run remote exec builds from Linux, you have to vendor a BUILD
    file, but you might not strictly enforce Xcode versions for local
    devs
  2. You do enforce strict versions for local devs, but also want to test
    betas without having to manually generate these BUILD files before
    testing

Previously when you vendored an explicit modules BUILD file, this would
disable all scanning locally. This is still the default behavior but now
you can set an env var to also discover local Xcodes. This is useful for
2 cases:

1. If you run remote exec builds from Linux, you have to vendor a BUILD
   file, but you might not strictly enforce Xcode versions for local
   devs
2. You do enforce strict versions for local devs, but also want to test
   betas without having to manually generate these BUILD files before
   testing
@keith

keith commented Jul 31, 2026

Copy link
Copy Markdown
Member Author

i went back and forth on the api for something in the MODULE.bazel vs env vs both, I'm up for suggestions!

@adincebic

Copy link
Copy Markdown
Contributor

i went back and forth on the api for something in the MODULE.bazel vs env vs both, I'm up for suggestions!

I would avoid MODULE.bazel. Env seems more flexivle.

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