Skip to content

Conversation

@erinethomas
Copy link
Contributor

@erinethomas erinethomas commented Oct 3, 2024

WW3 code currently compiles in the source code repo. This is severely problematic and creates several issues including:
a) cluttering the source code directory
b) creating problematic 'race-conditions' when more than one case with WW3 need to be compiled (such as running test suites)

This PR simply copies the required directories from the WW3 source code repo (the aux, ftn, and bin directories) to the "build/wav" directory and compiles all WW3 code there.
The compiled .F90 and executable tar files are now located here: "/build/wav/tmp/"

The E3SM commands ./case.build and ./case.build --clean-all correctly compile and clean all WW3 code.

[BFB]

Copy link
Contributor

@sbrus89 sbrus89 left a comment

Choose a reason for hiding this comment

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

PASSES:

SMS_D_Ln3.TL319_EC30to60E2r2_wQU225EC30to60E2r2.GMPAS-JRA1p5-WW3.chrysalis_intel.ww3-jra_1958
ERP_Ld3.ne30pg2_r05_IcoswISC30E3r5.WCYCL1850.chrysalis_intel.allactive-pioroot1

sbrus89 added a commit that referenced this pull request Oct 10, 2024
…6663)

WW3 code currently compiles in the source code repo. This is
severely problematic and creates several issues including:
  a) cluttering the source code directory
  b) creating problematic 'race-conditions' when more than
     one case with WW3 need to be compiled (such as running test suites)

This PR simply copies the required directories from the WW3 source code
repo (the aux, ftn, and bin directories) to the "build/wav" directory
and compiles all WW3 code there.
The compiled .F90 and executable tar files are now located
here: "/build/wav/tmp/"

The E3SM commands ./case.build and ./case.build --clean-all
correctly compile and clean all WW3 code.

[BFB]
@sbrus89
Copy link
Contributor

sbrus89 commented Oct 10, 2024

Merged to next

@sbrus89 sbrus89 merged commit 57f919f into E3SM-Project:master Oct 14, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BFB PR leaves answers BFB Wave

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants