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

CI (alpine): Add an emulated RISC-V 64-bit runner #810

Merged
merged 3 commits into from
May 29, 2024

Conversation

mmuetzel
Copy link
Contributor

@mmuetzel mmuetzel commented May 23, 2024

This is a simpler alternative to #801 now that Alpine Linux 3.20 is released that includes packages for RISC-V 64.

Like @suvorovrain already wrote in that PR, valgrind isn't packaged for that architecture. (Upstream valgrind doesn't support that architecture.)
I'm not sure why it is installed in the first place on those runners. Could it be removed entirely?

@DrTimothyAldenDavis
Copy link
Owner

valgrind could be removed, or better yet it could be commented out. I had it there in case I need to debug something through the github CI, where an error was occuring on some platform I didn't have access to except through the github runners. Some of the demos and examples can be run with valgrind, by editting the Makefiles.

@mmuetzel
Copy link
Contributor Author

Thank you for clarifying.
I added a commit that comments out the line that would install valgrind. (I hope I didn't mess up the YAML syntax by that change.)

@DrTimothyAldenDavis
Copy link
Owner

Looks good. Shall I merge it in now or would it best to wait for all the CI's to finish?

@mmuetzel
Copy link
Contributor Author

Looks good. Shall I merge it in now or would it best to wait for all the CI's to finish?

I believe that file is still valid YAML with the last change. But it would probably be best to wait at least until (and if) the CI starts with that change.

@mmuetzel
Copy link
Contributor Author

mmuetzel commented May 24, 2024

It looks like the new runner built and checked without issues. 🎉

(Only slightly related: Still no feedback on the bug report regarding the lapack packages on the aarch64 and ppc64le Alpine Linux runners.)

It looks like with the update to Alpine Linux 3.20, the object files
that are produced by the compiler became smaller or the cache compression
of ccache is more efficient.
Reduce the maximum cache size to approx. twice the size of the ccache
after a single clean run.
@DrTimothyAldenDavis DrTimothyAldenDavis merged commit 14e2699 into DrTimothyAldenDavis:dev2 May 29, 2024
26 checks passed
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