ufs_release_1.0: cleanup work 2020/01/06

[climbfuji]

• update submodule pointer for cmake, add branch information
• update include/lib flags for nemsio, sigio, sfcio in sorc/ncep_post.fd/CMakeLists.txt
• add REQUIRED attribute to find_package calls

[climbfuji]

@DusanJovic-NOAA do you know if LAPACK is a required dependency for NCEPLIBS-post? If I comment out find_package(LAPACK) it still compiles fine, but I don't know if this is because the system finds the lapack libraries automatically in the system include/library paths. Similar question for NCEPLIBS-grib_util. Thanks!

[DusanJovic-NOAA]

@DusanJovic-NOAA do you know if LAPACK is a required dependency for NCEPLIBS-post? If I comment out find_package(LAPACK) it still compiles fine, but I don't know if this is because the system finds the lapack libraries automatically in the system include/library paths. Similar question for NCEPLIBS-grib_util. Thanks!

I think it does not require LAPACK. I also commented out find_package(LAPACK) and successfully built the EMC_post.

[climbfuji]

Thanks, will comment them out in the PRs.

[climbfuji]

@Hang-Lei-NOAA actually can you merge this PR, please? I do not have write access to EMC_post. Thanks! It is ready to go.

[fossell]

@climbfuji - I'll merge this for you.

[Hang-Lei-NOAA]

I don’t have the write access either. Ask mark Potts, he has super access to all EMC repo. Or ask post people: [email protected] Hang

…

On Wed, Jan 8, 2020 at 11:13 PM Dom Heinzeller ***@***.***> wrote: @Hang-Lei-NOAA <https://github.com/Hang-Lei-NOAA> actually can you merge this PR, please? I do not have write access to EMC_post. Thanks! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#42>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AKWSMFHCM2US5JPDKTGCZSLQ42P77ANCNFSM4KDKVX2Q> .

[Hang-Lei-NOAA]

Thanks Kate!

…

On Wed, Jan 8, 2020 at 11:17 PM Kate Fossell ***@***.***> wrote: Merged #42 <#42> into ufs_release_v1.0. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#42>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AKWSMFGQJYLXNDB7VFYFYQDQ42QPHANCNFSM4KDKVX2Q> .

[climbfuji]

Thanks to both of you, Kate and Hang!

[DusanJovic-NOAA]

${GFSIO_INC} should be ${GFSIO_INC4} in sorc/ncep_post.fd/CMakeLists.txt, here:

https://github.com/NOAA-EMC/EMC_post/blob/9bd8aed6418550842dfae1e0d9fd3a8aa915908f/sorc/ncep_post.fd/CMakeLists.txt#L191

For example, these are environment variables set by gfsio module on wcoss_dell_p3:

$ module show gfsio/1.1.0
--------------------------------------------------------------------------------------------------------------------------------------------
   /gpfs/dell1/nco/ops/nwprod/modulefiles/compiler_prod/ips/18.0.1/gfsio/1.1.0:
--------------------------------------------------------------------------------------------------------------------------------------------
conflict("gfsio")
setenv("COMP","ips")
setenv("GFSIO_SRC","/gpfs/dell1/nco/ops/nwprod/lib/gfsio/v1.1.0/src")
setenv("GFSIO_INC4","/gpfs/dell1/nco/ops/nwprod/lib/gfsio/v1.1.0/ips/18.0.1/include/gfsio_v1.1.0_4")
setenv("GFSIO_LIB4","/gpfs/dell1/nco/ops/nwprod/lib/gfsio/v1.1.0/ips/18.0.1/libgfsio_v1.1.0_4.a")
setenv("GFSIO_VER","v1.1.0")
help([[Set environment variables for gfsio library, including GFSIO_SRC, GFSIO_INC4, and GFSIO_LIB4]])

[climbfuji]

${GFSIO_INC} should be ${GFSIO_INC4} in sorc/ncep_post.fd/CMakeLists.txt, here:

https://github.com/NOAA-EMC/EMC_post/blob/9bd8aed6418550842dfae1e0d9fd3a8aa915908f/sorc/ncep_post.fd/CMakeLists.txt#L191

Thanks, will fix it in a minute.

[climbfuji]

${GFSIO_INC} should be ${GFSIO_INC4} in sorc/ncep_post.fd/CMakeLists.txt, here:
https://github.com/NOAA-EMC/EMC_post/blob/9bd8aed6418550842dfae1e0d9fd3a8aa915908f/sorc/ncep_post.fd/CMakeLists.txt#L191

Thanks, will fix it in a minute.

See here: #43 - will merge right away.