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.

Sign up for GitHub

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

Jump to bottom

Pin a newer @platforms in the Bazel workspace to fix Mac ARM builds. #10164

Merged
merged 1 commit into from
Apr 6, 2022
Merged

Pin a newer @platforms in the Bazel workspace to fix Mac ARM builds. #10164

merged 1 commit into from
Apr 6, 2022

Conversation

hawkinsp
Copy link
Collaborator

@hawkinsp hawkinsp commented Apr 6, 2022

Fixes #10132

@hawkinsp hawkinsp added the pull ready Ready for copybara import and testing label Apr 6, 2022
@nicholasjng
Copy link
Contributor

Nice, thanks for the help!

@hawkinsp
Copy link
Collaborator Author

hawkinsp commented Apr 6, 2022

@nicholasjng No, thank you! Please verify this works for you!

@hawkinsp hawkinsp mentioned this pull request Apr 6, 2022
Closed
@nicholasjng
Copy link
Contributor

Ran bazel clean before testing, jaxlib builds successfully with the currently pinned TF version.

I will retest with my local TF snapshot from yesterday, but the current one is fine. This looks perfect to me.

@copybara-service copybara-service bot merged commit be64d8b into jax-ml:main Apr 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
pull ready Ready for copybara import and testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Build failures at HEAD on Apple ARM64
2 participants
@hawkinsp @nicholasjng