Skip to content

Code changes to support automatic debug compilation#10

Merged
DeniseWorthen merged 2 commits into
NOAA-EMC:developfrom
MinsukJi-NOAA:feature/cice5_debug_flag
Jan 21, 2020
Merged

Code changes to support automatic debug compilation#10
DeniseWorthen merged 2 commits into
NOAA-EMC:developfrom
MinsukJi-NOAA:feature/cice5_debug_flag

Conversation

@MinsukJi-NOAA
Copy link
Copy Markdown
Contributor

  1. comp_ice.backend receives DEBUG as an environment variable, and passes it down to the makefile.
  2. Macros.Linux.NEMS.hera selects FFLAGS based on the DEBUG flag it receives.

This resolves issue #9

1. comp_ice.backend receives DEBUG as an environment variable, and passes it down to the makefile.

2. Macros.Linux.NEMS.hera selects FFLAGS based on the DEBUG flag it receives.
@MinsukJi-NOAA
Copy link
Copy Markdown
Contributor Author

  • Note that all regression tests for ufs-s2s-model passed.

@DeniseWorthen
Copy link
Copy Markdown
Collaborator

Minsuk,

Could you please add the required changes to the Macros.Linux.NEMS.cheyenne and then add it to your pull request (committing the changes to your branch should add the change to your PR). I had added information here stating that we would provide these for Hera and Cheyenne initially. Thanks!

Copy link
Copy Markdown
Collaborator

@DeniseWorthen DeniseWorthen left a comment

Choose a reason for hiding this comment

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

I confirmed that building using the app level debug option (make app=coupledFV3_MOM6_CICE_debug build) builds CICE5 on hera with the correct fflags _debug

@DeniseWorthen DeniseWorthen merged commit aafac32 into NOAA-EMC:develop Jan 21, 2020
@MinsukJi-NOAA MinsukJi-NOAA deleted the feature/cice5_debug_flag branch January 21, 2020 15:13
DeniseWorthen referenced this pull request in DeniseWorthen/CICE5 Jan 30, 2020
Code changes to support automatic debug compilation (#10)
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