search

DrTimothyAldenDavis / SuiteSparse

The official SuiteSparse library: a suite of sparse matrix algorithms authored or co-authored by Tim Davis, Texas A&M University.
https://people.engr.tamu.edu/davis/suitesparse.html
Other
1.11k stars 256 forks source link

CI: Add riscv64 arch #801

Closed suvorovrain closed 3 weeks ago

suvorovrain commented 1 month ago

CI for riscv64 runs on alpine edge branch. Valgrind packages are not available on this architecture, so I had to remove it from the dependency list. The directory for saving the ccahce cache is different on the edge branch, so I also changed it in riscv64 arch case. Also i had to disable static libs flag in Cmake examples. The problem is that the package versions in the edge branch are different from the stable one, and when i tried to statically link i got following: lto1: fatal error: bytecode stream in file '/usr/lib/libgmp.a' generated with LTO version 13.0 instead of the expected 13.1 (on other architectures in the edge branch the problem is same)

mmuetzel commented 1 month ago

Thank you for your PR.

IIUC, Alpine Linux aims to release a new stable version each May and November [1]. According to their (stub) Wiki page, they are planning to include support for RISCV64 in 3.20 which will probably be released soon.

Maybe, we should just wait a couple of weeks for that release. That would probably make it a lot easier to add CI for that platform.

Edit: Fwiw, a first release candidate for Alpine Linux 3.20 has been tagged just an hour ago: https://gitlab.alpinelinux.org/alpine/aports/-/commit/86d5892796fb5c817aab0c07a56931bc08d0af61 Release checklist: https://gitlab.alpinelinux.org/alpine/aports/-/issues/16119

DrTimothyAldenDavis commented 1 month ago

Is this superseded by https://github.com/DrTimothyAldenDavis/SuiteSparse/pull/810 ?

DrTimothyAldenDavis commented 3 weeks ago

See #810 for the alternative option taken.