-
Notifications
You must be signed in to change notification settings - Fork 4k
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
--ios_multi_cpus
defaults to x86_64 on M1 Macs
#14284
Closed
Labels
Comments
Probably can reopen #13873 instead? Not sure if it's a different issue. |
That Pr Was merged so probably want a new one instead |
We think now this is likely related to this transition bazelbuild/rules_apple#1154 |
If bazelbuild/rules_apple#1154 fixes this, can we mark as such in the description to close this issue when ready? |
Done. When it's merged it will close this. |
Didn't work because of ACL, but that's merged |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description of the problem / feature request:
--ios_multi_cpus
should default tosim_arm64
on M1 Macs, but still defaults tox86_64
.Feature requests: what underlying problem are you trying to solve with this feature?
M1 Mac users shouldn't have to override
--ios_multi_cpus
to get the proper default.Bugs: what's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.
Build an
ios_application
target on an M1 Mac, see that the architecture is for the x86_64 simulator.I think that this might be the issue:
bazel/src/main/starlark/builtins_bzl/common/objc/transitions.bzl
Lines 70 to 73 in e50b203
d7628e1 changed the default in one place already.
What operating system are you running Bazel on?
macOS 11.6.1 (arm64)
What's the output of
bazel info release
?release 6.0.0-pre.20211101.2
The text was updated successfully, but these errors were encountered: