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

Added s390x arch support #1265

Merged
merged 5 commits into from
Jul 30, 2024
Merged

Added s390x arch support #1265

merged 5 commits into from
Jul 30, 2024

Conversation

Repana-Chowdappa
Copy link
Contributor

Currently buildtools repo doesn't a downloadable s390x binary yet for buildifier, buildozer. I tried to build buildtools on s390x and it actually builds fine. Need to include s390x arch binaries in https://github.com/bazelbuild/buildtools/releases/.

@Repana-Chowdappa
Copy link
Contributor Author

@vladmos @pmbethe09 @larsrc-google - Can you please check and approve this PR.

@Repana-Chowdappa
Copy link
Contributor Author

@vladmos @pmbethe09 @larsrc-google - This is blocking rules_rust and kong CI completion for s390x arch to add s390x support. Buildifier is dependency for rust_rules and kong. Please help us in reviewing this PR.

Copy link

@lysliu lysliu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@vladmos vladmos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@vladmos vladmos merged commit ef13530 into bazelbuild:main Jul 30, 2024
2 checks passed
@rchowdap
Copy link

@vladmos @lysliu - Thanks for merging the PR to code base. May i know when will be next release for buildtools and when s390x buildifier binary available for us.

@vladmos
Copy link
Member

vladmos commented Aug 27, 2024

I'll cut a new release in an hour.

@Repana-Chowdappa
Copy link
Contributor Author

@vladmos - I don't see s390x binaries on https://github.com/bazelbuild/buildtools/releases/download/v7.3.1/ after new release. Please let us know whether s390x binaries are built or not.

@vladmos
Copy link
Member

vladmos commented Aug 28, 2024

The binaries were not configured for Github releases. I uploaded them this time manually, #1289 is a permanent fix.

@Repana-Chowdappa
Copy link
Contributor Author

Thanks @vladmos for quick help. may i know from where I can get sha256 values for these binaries?

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