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

build/pkgs/cbc 2.10.11, sage_numerical_backends_coin #38312

Draft
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

mkoeppe
Copy link
Member

@mkoeppe mkoeppe commented Jun 30, 2024

  • Our cbc package is ancient and fails to build on various platforms
  • the current sage_numerical_backends_coin package has not been updated for Cython 3
  • current cbc tarballs no longer seem to contain dependencies such as Clp.

Switching cbc to install via coinbrew.

📝 Checklist

  • The title is concise and informative.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation and checked the documentation preview.

⌛ Dependencies

@mkoeppe mkoeppe added this to the sage-10.4 milestone Jun 30, 2024
@mkoeppe mkoeppe self-assigned this Jun 30, 2024
mkoeppe added a commit to sagemath/sage-numerical-backends-coin that referenced this pull request Jun 30, 2024
Copy link

github-actions bot commented Jun 30, 2024

Documentation preview for this PR (built with commit 5516158; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

@mkoeppe
Copy link
Member Author

mkoeppe commented Jun 30, 2024

@culler Here is a tentative fix for sage_numerical_backends_coin, but on my macOS machine I'm getting segfaults in the Sage test suite when it is installed. I'll check what it looks like on the other platforms here in the CI.

@culler
Copy link
Contributor

culler commented Jun 30, 2024

Wow. That does much more than add a few noexcept qualifiers. It adds yet another build system, with its own spkg! I see that lots of jobs were killed due to abort, but I don't see a stack trace.

@mkoeppe mkoeppe removed this from the sage-10.4 milestone Jun 30, 2024
@mkoeppe
Copy link
Member Author

mkoeppe commented Jun 30, 2024

Yes... Unfortunately our ancient version of cbc fails to build on many platforms, including the "manylinux" wheel building environment that I'm trying to set up. Hence the upgrade. But it looks like this won't be ready for 10.4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants