Update to CMEPS with CM3 changes#42
Conversation
|
@kieranricardo - see gcc build failures |
|
Thanks @kieranricardo - please update this for the commit hash post merging the change into CMEPS |
|
@anton-seaice I've updated to the head of |
| if (cdeps IN_LIST _required_components) | ||
| find_dependency(FoX) | ||
| endif() | ||
| find_dependency(FoX) |
There was a problem hiding this comment.
Fox isn't required by CMEPS or share:
e.g. it's not in
https://github.com/ACCESS-NRI/access3-share/blob/cm3-rc-2026.01.000/CMEPS/CMakeLists.txt
or
https://github.com/ACCESS-NRI/access3-share/blob/cm3-rc-2026.01.000/share/CMakeLists.txt
There was a problem hiding this comment.
I think something breaks without FoX,maybe CICE and MOM depend on CDEPS share but I forget how this all works exactly. Do you remember @blimlim?
If not I'll just test it tomorrow when Gadis back
There was a problem hiding this comment.
I think were running into an issue with CICE linking to cdeps-common which in turn linked to fox
In our old build we'd been adding CDEPs as a component as a workaround
There was a problem hiding this comment.
cdeps is in the list of required components for CICE, so this should be fine as is
I think just retest without this part of the change ?
There was a problem hiding this comment.
yep this works nicely! ACCESS-NRI/ACCESS-OM3#225
3 participants
Companion to ACCESS-NRI/CMEPS#12