-
Notifications
You must be signed in to change notification settings - Fork 39
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
chore: make blst portable #1520
Conversation
Workflow run where the runner doesn't support ADX: https://github.com/axonweb3/axon/actions/runs/6665823425/job/18116195808#step:5:69 |
3a6f00f
to
10a5f19
Compare
Sorry, I didn't catch your meaning.
I don't try to reject your PR, it's pretty good. I'm just curious. I think your PR is a good try, but doesn't be proved to be a final solution from that logs. |
See the build axon step: |
Then, you should paste that link to |
Merge CI failed since failures: tests::system_script::metadata::test_write_functions. (ref: #1374 (comment)) |
What this PR does / why we need it?
This PR enables the portable feature of the blst crate so that axon won't use ADX instructions.
From blst readme:
What is the impact of this PR?
No Breaking Change
PR relation:
CI Settings
CI Usage
Tip: Check the CI you want to run below, and then comment
/run-ci
.CI Switch
CI Description