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

Update to current CM3 versions of CMEPS and CICE #269

Open
wants to merge 1 commit into
base: cm3
Choose a base branch
from

Conversation

blimlim
Copy link

@blimlim blimlim commented Feb 11, 2025

This PR updates the CMEPs and CICE submodules on the cm3 branch to the latest CM3-specific versions:

These updates are temporary, and each component will shortly be updated to the versions from #250

@MartinDix
Copy link

Checks suggest that Configure Cmake crashed with an illegal instruction which is impressively broken. Is this expected to work?

@blimlim
Copy link
Author

blimlim commented Feb 11, 2025

Error happens when using the current cm3 branch with no changes: #271

The changes here update the submodules to the versions that are picked up when following the dev-coupling build instructions, which work when building locally.

Most of the CMake files in the main branch have changed since the cm3 branch split off. I wonder if these differences are why the CI is failing? When updating to the latest versions, the CMake files will also be updated.

Copy link
Collaborator

@kieranricardo kieranricardo left a comment

Choose a reason for hiding this comment

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

Looks good, very strange CI failures though! @blimlim I think you're right, and we worry about them once everything is updated. Probably the way om3 is built has changed quite a bit since these branches diverged.

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