Skip to content

Conversation

jedwards4b
Copy link
Contributor

We want to move the hardcoding of support libraries out of cime build.py with this PR. I am using mom to demonstrate what the change requires, it is backward compatible.

@jedwards4b jedwards4b self-assigned this Aug 21, 2025
@jedwards4b jedwards4b marked this pull request as draft August 21, 2025 14:14
@jedwards4b jedwards4b marked this pull request as ready for review August 28, 2025 13:58
if CIMEROOT is None:
raise SystemExit("ERROR: must set CIMEROOT environment variable")
sys.path.append(os.path.join(CIMEROOT, "scripts", "Tools"))
sys.path.append(os.path.join(CIMEROOT, "CIME", "Tools"))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updates path to cime/CIME/Tools, not related to the issue of the PR.

@alperaltuntas
Copy link
Member

Running the aux_mom suite, I received `ERROR: No variable CASE_SUPPORT_LIBRARIES found in case'. My sandbox is based on beta06.

Finished SETUP for test SMS.TL319_t232.G1850MARBL_JRA.derecho_intel in 5.807936 seconds (FAIL). [COMPLETED 10 of 14]
Case dir: /glade/derecho/scratch/altuntas/cesm.tests/aux_mom/30beta06/SMS.TL319_t232.G1850MARBL_JRA.derecho_intel.C.20250828_094125_668qgj
Errors were:
Test 'SMS.TL319_t232.G1850MARBL_JRA.derecho_intel' failed in phase 'SETUP' with exception 'ERROR: Fatal error in case.cmpgen_namelists: 2025-08-28 09:41:36 atm
Create namelist for component datm
Calling /glade/work/altuntas/cesm.sandboxes/cesm3_0_beta06.sbx/components/cdeps/datm/cime_config/buildnml
2025-08-28 09:41:36 lnd
Create namelist for component slnd
Calling /glade/work/altuntas/cesm.sandboxes/cesm3_0_beta06.sbx/cime/CIME/non_py/src/components/stub_comps_nuopc/slnd/cime_config/buildnml
2025-08-28 09:41:36 ice
Create namelist for component cice
Calling /glade/work/altuntas/cesm.sandboxes/cesm3_0_beta06.sbx/components/cice//cime_config/buildnml
RUN: /glade/work/altuntas/cesm.sandboxes/cesm3_0_beta06.sbx/components/cice/bld/generate_cice_decomp.pl -ccsmroot /glade/work/altuntas/cesm.sandboxes/cesm3_0_beta06.sbx -res tx2_3v2 -nx 540 -ny 480 -nproc 128 -thrds 1 -output all
FROM: /glade/derecho/scratch/altuntas/cesm.tests/aux_mom/30beta06/SMS.TL319_t232.G1850MARBL_JRA.derecho_intel.C.20250828_094125_668qgj
output: 540 480 18 16 8 sectrobin square-ice

2025-08-28 09:41:36 ocn
Create namelist for component mom
Calling /glade/work/altuntas/cesm.sandboxes/cesm3_0_beta06.sbx/components/mom//cime_config/buildnml
ERROR: No variable CASE_SUPPORT_LIBRARIES found in case'
File "/glade/work/altuntas/cesm.sandboxes/cesm3_0_beta06.sbx/cime/CIME/test_scheduler.py", line 1126, in _run_catch_exceptions
return run(test)
File "/glade/work/altuntas/cesm.sandboxes/cesm3_0_beta06.sbx/cime/CIME/test_scheduler.py", line 1012, in _setup_phase
expect(
File "/glade/work/altuntas/cesm.sandboxes/cesm3_0_beta06.sbx/cime/CIME/utils.py", line 176, in expect
raise exc_type(msg)

@alperaltuntas
Copy link
Member

@jedwards4b If CASE_SUPPORT_LIBRARIES will have been introduced in alpha07d (or already introduced since beta06), then maybe backwards compatibility isn't needed. Otherwise, we probably want something like a try except block around that.

@jedwards4b
Copy link
Contributor Author

Yes, I've added the try except block and we will remove the backward compatibility once all components are updated in a tag. Thanks

@alperaltuntas alperaltuntas merged commit 5b984da into ESCOMP:main Sep 2, 2025
5 checks passed
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