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

Correct spack-stack path for Gaea-C6 #1083

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

DavidBurrows-NCO
Copy link
Contributor

Updates the spack-stack path for Gaea-C6 to follow ufs-wx-model.
Resolves #1082
Refs NOAA-EMC/global-workflow#3011

@WenMeng-NOAA WenMeng-NOAA added enhancement New feature or request Ready for Review This PR is ready for code review. No Baseline Change No baseline of the UPP regression tests are made. labels Nov 5, 2024
@WenMeng-NOAA
Copy link
Collaborator

@DavidBurrows-NCO Have you tested on building UPP executable on Gaea-C6 given that Gaea is not included in the UPP RTs?

@DavidBurrows-NCO
Copy link
Contributor Author

@WenMeng-NOAA That's correct; I retested after correcting the spack-stack path...here's a snippet from that:

./compile_upp.sh 
Running "module reset". Resetting modules to system default. The following $MODULEPATH directories have been removed: /ncrc/proj/epic/rocoto/modulefiles
0x00007f3869eeabe6 in wait4 () from /lib64/libc.so.6
$1 = 0
$2 = -1
[Inferior 1 (process 2366730) detached]
Building for machine gaeac6, compiler intel
...
-- Installing: /gpfs/f6/bil-fire8/scratch/David.Burrows/double_check_upp_build/UPP/tests/install/include/gtg_ctlblk.mod
-- Installing: /gpfs/f6/bil-fire8/scratch/David.Burrows/double_check_upp_build/UPP/tests/install/include/icingpotential.mod
-- Installing: /gpfs/f6/bil-fire8/scratch/David.Burrows/double_check_upp_build/UPP/tests/install/include/gtg_config.mod
-- Installing: /gpfs/f6/bil-fire8/scratch/David.Burrows/double_check_upp_build/UPP/tests/install/include/native_endianness.mod
-- Installing: /gpfs/f6/bil-fire8/scratch/David.Burrows/double_check_upp_build/UPP/tests/install/lib/libupp.a
-- Installing: /gpfs/f6/bil-fire8/scratch/David.Burrows/double_check_upp_build/UPP/tests/install/lib/cmake/upp/upp-config.cmake
-- Installing: /gpfs/f6/bil-fire8/scratch/David.Burrows/double_check_upp_build/UPP/tests/install/lib/cmake/upp/upp-config-version.cmake
-- Installing: /gpfs/f6/bil-fire8/scratch/David.Burrows/double_check_upp_build/UPP/tests/install/lib/cmake/upp/upp-targets.cmake
-- Installing: /gpfs/f6/bil-fire8/scratch/David.Burrows/double_check_upp_build/UPP/tests/install/lib/cmake/upp/upp-targets-release.cmake
+ rm -rf /gpfs/f6/bil-fire8/scratch/David.Burrows/double_check_upp_build/UPP/exec
+ mkdir -p /gpfs/f6/bil-fire8/scratch/David.Burrows/double_check_upp_build/UPP/exec
+ cp ../install/bin/upp.x /gpfs/f6/bil-fire8/scratch/David.Burrows/double_check_upp_build/UPP/exec/.
+ [[ NO == YES ]]

@WenMeng-NOAA
Copy link
Collaborator

No need for the UPP RTs. This PR is ready for merging.

@WenMeng-NOAA WenMeng-NOAA merged commit f697316 into NOAA-EMC:develop Nov 7, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request No Baseline Change No baseline of the UPP regression tests are made. Ready for Review This PR is ready for code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Gaea-C6 update to spack-stack path to C6 install
2 participants