Skip to content

Fix SuiteSparse options for CVXOPT 1.2.6 w/ foss/2021a#23864

Merged
Micket merged 1 commit intoeasybuilders:developfrom
Flamefire:20250915125321_new_pr_CVXOPT126
Sep 15, 2025
Merged

Fix SuiteSparse options for CVXOPT 1.2.6 w/ foss/2021a#23864
Micket merged 1 commit intoeasybuilders:developfrom
Flamefire:20250915125321_new_pr_CVXOPT126

Conversation

@Flamefire
Copy link
Copy Markdown
Contributor

@Flamefire Flamefire commented Sep 15, 2025 •
edited
Loading

(created using eb --new-pr)

I don't see how #13533 could work that sets CVXOPT_SUITESPARSE_SRC_DIR. If that is set the build will try build the suitesparse files from source.

If it isn't set it will just link the library.

See https://github.com/cvxopt/cvxopt/blob/1.2.6/setup.py#L181-L202

Use the same approach as used in the newer easyconfigs, see #20232

Error looks like:

    gcc: error: /software/SuiteSparse/5.10.1-foss-2021a-METIS-5.1.0/SuiteSparse_config/SuiteSparse_config.c: No such file or directory
    gcc: fatal error: no input files
    compilation terminated.
    error: command '/software/GCCcore/10.3.0/bin/gcc' failed with exit code 1

@Flamefire
Copy link
Copy Markdown
Contributor Author

Flamefire commented Sep 15, 2025

Test report by @Flamefire
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
i7179 - Linux Rocky Linux 9.6, x86_64, AMD EPYC 7702 64-Core Processor (zen2), Python 3.9.21
See https://gist.github.com/Flamefire/8a6f27bd684c2b549e948cfaa99ccef9 for a full test report.

@Thyre Thyre added the 2021a label Sep 15, 2025
@Flamefire
Copy link
Copy Markdown
Contributor Author

Flamefire commented Sep 15, 2025

Test report by @Flamefire
SUCCESS
Build succeeded for 7 out of 7 (1 easyconfigs in total)
c144 - Linux AlmaLinux 9.4, x86_64, AMD EPYC 9334 32-Core Processor (zen4), 4 x NVIDIA NVIDIA H100, 560.35.03, Python 3.9.18
See https://gist.github.com/Flamefire/8a65d25903f0658e7b115b2fdcf3cdf3 for a full test report.

Micket
Micket approved these changes Sep 15, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Micket Micket added this to the release after 5.1.2 (5.2.0?) milestone Sep 15, 2025
@Micket Micket merged commit 86a719b into easybuilders:develop Sep 15, 2025
8 checks passed
@Flamefire Flamefire deleted the 20250915125321_new_pr_CVXOPT126 branch September 16, 2025 07:15
@boegel boegel added bug fix and removed change labels Sep 18, 2025
@boegel boegel changed the title Fix SuiteSparse options for CVXOPT-1.2.6-foss-2021a. Fix SuiteSparse options for CVXOPT 1.2.6 w/ foss/2021a Sep 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Micket Micket Micket approved these changes

Assignees

No one assigned

Labels

2021a bug fix

Projects

None yet

Milestone

5.1.2

Development

Successfully merging this pull request may close these issues.

4 participants

@Flamefire @Micket @boegel @Thyre