Skip to content

Conversation

@luciansmith
Copy link
Member

Change the 'rapid build' github action to use c++26 when possible. This should allow us to catch any changes (dropped constructs, etc.) sooner rather than later.

Worked out of the box, so no problems as of now.

It causes errors in most builds, for a variety of reasons.
Revert including util in the bindings.
@luciansmith
Copy link
Member Author

Unrelatedly, 'yum' stopped working on the quay build, because it's looking for the mirror URLs:

http://westus2.azure.repo.almalinux.org/almalinux/8.10/BaseOS/x86_64/os/Packages/c/crypto-policies-scripts-20230731-1.git3177e06.el8.noarch.rpm

but there's no (or no longer?) any 'c/' subdirectory; it's actually just available at:

http://westus2.azure.repo.almalinux.org/almalinux/8.10/BaseOS/x86_64/os/Packages/crypto-policies-scripts-20230731-1.git3177e06.el8.noarch.rpm

@luciansmith
Copy link
Member Author

Yum works again; builds now succeeding once more.

@luciansmith luciansmith requested a review from fbergmann April 11, 2025 19:50
@fbergmann
Copy link
Member

that works for me ... though we should not have the changes to libsbml.i / libsbml.h here. After that i think it is good to merge.

@luciansmith
Copy link
Member Author

Done!

Copy link
Member

@fbergmann fbergmann left a comment

Choose a reason for hiding this comment

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

awesome thank you so much

@luciansmith luciansmith merged commit c802b3c into development Apr 14, 2025
23 of 35 checks passed
@luciansmith luciansmith deleted the test-later-cxx branch April 14, 2025 15:55
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.

3 participants