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

Testing: Recurse target submodules, LDFLAGS hook #1355

Merged
merged 1 commit into from
Mar 23, 2021

Conversation

marshallward
Copy link
Collaborator

Two minor changes to the .testing build:

  • We now apply --recurse-submodules to the target build in the
    regression test. This is required after an update to the submodules,
    when the target submodule is out of sync with the main branch (e.g.
    dev/gfdl at NOAA-GFDL).

  • A LDFLAGS_USER hook was added to the .testing/Makefile
    configuration, similar to the FCFLAGS_* hooks. This is required if
    the library dependencies do not reside in the default directores.

    For example, this may be needed for a custom netCDF or MPI library.

Two minor changes to the .testing build:

- We now apply `--recurse-submodules` to the target build in the
  regression test.  This is required after an update to the submodules,
  when the target submodule is out of sync with the main branch (e.g.
  dev/gfdl at NOAA-GFDL).

- A LDFLAGS_USER hook was added to the `.testing/Makefile`
  configuration, similar to the FCFLAGS_* hooks.  This is required if
  the library dependencies do not reside in the default directores.

  For example, this may be needed for a custom netCDF or MPI library.
@marshallward
Copy link
Collaborator Author

marshallward commented Mar 23, 2021

Now that Gaea is running .testing, it probably makes sense to start testing these:

https://gitlab.gfdl.noaa.gov/ogrp/MOM6/-/pipelines/12324 ✔️

@adcroft adcroft merged commit 4fd897d into mom-ocean:dev/gfdl Mar 23, 2021
@marshallward marshallward deleted the test_submod_update branch May 7, 2021 02:53
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.

2 participants