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

Fixes build failure in gitlab pipeline #325

Merged
merged 1 commit into from
Feb 10, 2023

Conversation

adcroft
Copy link
Member

@adcroft adcroft commented Feb 10, 2023

After a recent update to how the build environment is defined within MOM6-examples, the "no libraries" build test (unique to MOM6) is failing because the Makefile now no longer contains the environment. This commit overrides a CPP macro that points to the bash script that is needed.

I tested the pipeline at https://gitlab.gfdl.noaa.gov/ogrp/MOM6/-/pipelines/18205

- After a recent update to how the build environment is defined within
MOM6-examples, the "no libraries" build test (unique to MOM6) is failing
because the Makefile now no longer contains the environment. This commit
overrides a CPP macro that points to the bash script that is needed.
@adcroft adcroft requested a review from marshallward February 10, 2023 19:54
@codecov
Copy link

codecov bot commented Feb 10, 2023

Codecov Report

Merging #325 (8f568c0) into dev/gfdl (168305f) will decrease coverage by 0.01%.
The diff coverage is n/a.

❗ Current head 8f568c0 differs from pull request most recent head 7d9327f. Consider uploading reports for the commit 7d9327f to get more accurate results

@@             Coverage Diff              @@
##           dev/gfdl     #325      +/-   ##
============================================
- Coverage     37.17%   37.17%   -0.01%     
============================================
  Files           265      265              
  Lines         74424    74424              
  Branches      13822    13822              
============================================
- Hits          27666    27665       -1     
- Misses        41672    41674       +2     
+ Partials       5086     5085       -1     
Impacted Files Coverage Δ
src/framework/MOM_document.F90 72.92% <0.00%> (-0.22%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@marshallward
Copy link
Member

Gaea regression: https://gitlab.gfdl.noaa.gov/ogrp/MOM6/-/pipelines/18206 ✔️

@marshallward marshallward merged commit 9412c9a into NOAA-GFDL:dev/gfdl Feb 10, 2023
@adcroft adcroft deleted the fix-ci-nolibs-build branch June 26, 2023 17:50
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