Skip to content

Add support for compile buck2 at riscv64 #947

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

xizheyin
Copy link
Contributor

@xizheyin xizheyin commented May 7, 2025

In order to support riscv, I realized that there are some dependencies that don't support riscv.

  1. The dependency webpki v0.21 in hyper-proxy relies on ring v0.16.* which does not support riscv, so I updated it to webpki v0.22. see PR Update webpki from 0.21 to 0.22 get9/hyper-proxy#3 and this commit.
  2. For perf-event, this fork version is too old, so I copied PR and rebase it at the latest version, thus ensuring compatibility. See https://github.com/xizheyin/perf-event/tree/buck2.

I replaced the dependencies with my fixed version that I forked.

cc @get9 @krallin

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 7, 2025
@facebook-github-bot
Copy link
Contributor

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. (Because this pull request was imported automatically, there will not be any future comments.)

@alexlian
Copy link
Contributor

alexlian commented May 7, 2025

Could you please rebase up to the latest to get the fix for the broken CI actions?

@xizheyin
Copy link
Contributor Author

xizheyin commented May 7, 2025

Sure, I rebased it. Waiting for passing CI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants