You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
All of these were copied from Axom, including cases where +fortran. The reason for this was -lpgmath is certainly needed, but it was in Axom's +fortran case. So, I (mistakenly) just added all the fortran-related libs and didn't think much of it.
chapman39
changed the title
Try to remove -lflang -lflangrti from Serac Spack Recipe when +rocm
Try to remove ENABLE_FORTRAN in case of toss4_cray builds
Nov 25, 2024
During the process building Serac on tioga/ rzvernal, I likely added more libs than we need in the Serac Spack package here: https://github.com/LLNL/serac/blob/develop/scripts/spack/packages/serac/package.py#L390
All of these were copied from Axom, including cases where
+fortran
. The reason for this was-lpgmath
is certainly needed, but it was in Axom's+fortran
case. So, I (mistakenly) just added all the fortran-related libs and didn't think much of it.This has possibly caused another issue which required a band-aid fix of enabling fortran in tioga/ rzvernal builds. https://github.com/LLNL/serac/blob/develop/CMakeLists.txt#L52
I should try to remove the
-lflang -lflangrti
libs during the next TPL build and see if that allows us to leave fortran disabled on tioga/ rzvernal.EDIT: Removing these flags does not appear to resolve the issue.
The text was updated successfully, but these errors were encountered: