Skip to content

CI: Add a CI to build ELF #606

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: master
Choose a base branch
from

Conversation

Arielfoever
Copy link
Contributor

@trdthg
Copy link

trdthg commented Jun 2, 2025

@aswaterman Any suggestions?

Our starting point is that we want to integrate riscv-tests tests into sail-riscv. Our options are as follows:

  • download test binary from ci release
    • This is what this PR does. Downloading test files in CI maybe is not a good idea. Although we could cache it in CI.
  • using riscv-tests as submodule, and build it in ci
    • I personally think this method is the best, although I don't think git submodule is particularly easy to use.
  • build or download binary and keep it in reposity.
    • Easiest method for users, they don't need to configure riscv toolchain or anything else locally, but the disadvantage is also obvious, it takes up a lot of disk space

But no matter which option, I think there is no harm in distributing binaries in this repository.

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