-
Notifications
You must be signed in to change notification settings - Fork 7
Add options for building libraries for CM3 #325
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
Conversation
|
Is there a corresponding CMake change - the bigger part of this work is linking the UM into the access3 cmake right ? |
Definitely! At the moment I don't think the UM is ready to add, and so I'm hoping to start off just by cleaning up our current build process for CM3 in a way that shouldn't interfere with adding the UM in the future! I think the corresponding CMake change for this will be to add a placeholder |
Co-authored-by: Anton Steketee <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good - thanks @blimlim
Please squash merge and give a clear commit description
Closes #324
This PR adds a
drivervariant to theaccess-cicespack package. This allows a custom relative path to be set for theCICE_DRIVERoption in the CICE CMake build.It also adds a
MOM6-CICE6-UM13build configuration to theaccess3package, which addsgcomas a dependency when active, as suggested here. If we decide to it this way, this build configuration will also need to be added to the known configurations in the cmake build