Skip to content

Add darwin_x86_64 CPU handling#4329

Closed
keith wants to merge 1 commit intogoogle:masterfrom
keith:ks/add-darwin_x86_64-cpu-handling
Closed

Add darwin_x86_64 CPU handling#4329
keith wants to merge 1 commit intogoogle:masterfrom
keith:ks/add-darwin_x86_64-cpu-handling

Conversation

@keith
Copy link
Copy Markdown

@keith keith commented Feb 22, 2023

Currently bazel supports darwin and darwin_x86_64 as meaning the same thing. The fully qualified version is normally used if you're cross compiling from M1 machines to intel machines. I'm also hoping to remove the unqualified version to reduce confusion. This change makes it compatible with both for now.

Currently bazel supports `darwin` and `darwin_x86_64` as meaning the
same thing. The fully qualified version is normally used if you're cross
compiling from M1 machines to intel machines. I'm also hoping to remove
the unqualified version to reduce confusion. This change makes it
compatible with both for now.
@keith
Copy link
Copy Markdown
Author

keith commented Jul 10, 2023

@fbarchard @alankelly could one of you take a look?

copybara-service bot pushed a commit that referenced this pull request Aug 21, 2023
PiperOrigin-RevId: 558837158
copybara-service bot pushed a commit that referenced this pull request Aug 21, 2023
PiperOrigin-RevId: 558837158
copybara-service bot pushed a commit that referenced this pull request Aug 21, 2023
PiperOrigin-RevId: 558837158
copybara-service bot pushed a commit that referenced this pull request Aug 21, 2023
PiperOrigin-RevId: 558837158
@Maratyszcza
Copy link
Copy Markdown
Contributor

This is fixed in 16bb890

@keith keith deleted the ks/add-darwin_x86_64-cpu-handling branch August 21, 2023 20:41
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.

2 participants