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

SPEX: Account for SUITESPARSE_USE_STRICT when using SPEX_USE_PYTHON #817

Merged
merged 2 commits into from
May 29, 2024

Conversation

mmuetzel
Copy link
Contributor

Handle SPEX_USE_PYTHON similar to SPEX_USE_OPENMP when SUITESPARSE_USE_STRICT is ON.

Another addendum to #805.

Handle SPEX_USE_PYTHON similar to SPEX_USE_OPENMP when
SUITESPARSE_USE_STRICT is ON.
Do not default to building Python interface if not building shared libraries.
@mmuetzel
Copy link
Contributor Author

The latest commit uses a default value of OFF for SPEX_USE_PYTHON if BUILD_SHARED_LIBS evaluates to false.
I'm wondering if it would be better to make the default value of SUITESPARSE_USE_PYTHON dependent on BUILD_SHARED_LIBS instead...

What do you think?

@mmuetzel mmuetzel closed this May 26, 2024
@mmuetzel mmuetzel reopened this May 26, 2024
@mmuetzel
Copy link
Contributor Author

Re-opening to trigger the CI again. The issue with the package signatures from Alpine Linux should be fixed now:
https://gitlab.alpinelinux.org/alpine/aports/-/issues/16136

@DrTimothyAldenDavis
Copy link
Owner

Looks great -- thanks!

@DrTimothyAldenDavis DrTimothyAldenDavis merged commit 43cdafc into DrTimothyAldenDavis:dev2 May 29, 2024
47 of 50 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