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

Add build files using autotools to Example package #666

Merged
merged 3 commits into from
Dec 31, 2023

Conversation

mmuetzel
Copy link
Contributor

This adds files to the Example package that lay out how downstream projects that use autotools could integrate SuiteSparse.
This leaves the CMake build rules of the the example (mostly) untouched.

Additionally, this adds a step to the CI which builds the example using autotools. This can serve as a test of whether the .pc files are working. (This test would probably have revealed issues like #656 earlier.)

We already knew that flags were missing from the .pc files to statically link with CUDA. I still don't know enough about CUDA to determine the necessary linker flags generally. So, I resorted to hard-coding what I found on their forum. I hope that is good enough for now.

I chose autotools over other build systems (like meson, scons, ...) because I know how to use autotools but not the others. Additionally, autotools (albeit being "old") is still one of the major established build systems.
If others would like to contribute examples that show how to integrate SuiteSparse in other build systems, that might also be nice.

Simple example for downstream projects using autotools.
That checks if the pkg-config files are working correctly.
@DrTimothyAldenDavis DrTimothyAldenDavis merged commit 78b2d94 into DrTimothyAldenDavis:dev2 Dec 31, 2023
23 checks passed
@DrTimothyAldenDavis
Copy link
Owner

Julia is having trouble: JuliaLang/julia#52577 (comment)

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