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.
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.The text was updated successfully, but these errors were encountered: