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 (macos): Query installation prefix of Homebrew #807

Closed mmuetzel closed 1 month ago

mmuetzel commented 1 month ago

The macos-latest label was changed to point to macos-14 runners somewhat recently: https://github.com/github/roadmap/issues/926 Those runners are on Apple Silicon. For some reason, Homebrew decided to use a different installation prefix on Intel and on Apple Silicon: https://docs.brew.sh/FAQ#why-should-i-install-homebrew-in-the-default-location

Query the installation prefix of Homebrew and use it where necessary.

Also, add a schedule to trigger the workflow weekly. This can be useful not only to detect possibly breaking changes in SuiteSparse itself. But it can also be useful to detect upstream changes (e.g., in some Homebrew package or GitHub runner configuration) that require changes of the build rules.

mmuetzel commented 1 month ago

The build issue on the Arch Linux runners is most likely unrelated. Arch Linux 3.20 has just been tagged a couple of hours ago. That might be a hick-up in the upgrade process that the Arch Linux maintainers will hopefully address soon.

mmuetzel commented 1 month ago

I triggered the macos runner on my fork of the repository, and the OpenMP implementation from Homebrew is detected again on Apple Silicon: https://github.com/mmuetzel/SuiteSparse/actions/runs/9203934509/job/25316416990

mmuetzel commented 1 month ago

I opened an issue at the tracker of Alpine Linux for the package installation error on Alpine Linux 3.20: https://gitlab.alpinelinux.org/alpine/aports/-/issues/16136