Build NetCDF using ./configure and create only static libs #52
Conversation
There was a problem hiding this comment.
This is probably beyond this PR: regarding zlib, we always had the issue that zlib is a dependency for libpng as well and that we need to use the same for both. By only using static libraries, we can already avoid the problem that at runtime the wrong library is picked. So now we need to make sure that building netcdf and png always have to go together (build both or none), or find a better solution. What if png is available and built against a shared zlib that we cannot use for netcdf in nceplibs-external? Something to discuss.
|
Fixed and run |
|
Do you know about building HDF5? Why do we set the compilers as We set |
I think we never bothered building the parallel version, because the UFS didn't need it. Since it's coming in the UFS weather model, we should probably enable the parallel builds of hdf5 and netcdf. No need for C++ bindings I suppose. |
|
Please do not delete |
|
I'm not actually deleting it. I just can't get either one to work. Mac because its using GCC10 and this version of NetCDF-Fortran has errors building with GCC 10. Linux because NetCDF-Fortran can't find NetCDF-C even though it builds fine on my Mac and on Orion. |
|
Ok, do you want me to take a look and work off of your PR?
… On Jul 7, 2020, at 2:44 PM, Kyle Gerheiser ***@***.***> wrote:
I'm not actually deleting it. I just can't get either one to work. Mac because its using GCC10 and this version of NetCDF-Fortran has errors building with GCC 10. Linux because I don't know why it NetCDF-Fortran can't find NetCDF-C even though it builds fine on my Mac and on Orion.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub <#52 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AB5C2ROI7ALI2LF43NAU2W3R2OCL3ANCNFSM4ONZ5FVQ>.
|
|
If you would. I can't see anything that's wrong and it builds everywhere I try that isn't Github. |
|
Sure, I will look at it for the rest of today. Thanks for all the work so far!
… On Jul 7, 2020, at 2:50 PM, Kyle Gerheiser ***@***.***> wrote:
If you would. I can't see anything that's wrong and it builds everywhere I try that isn't Github.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub <#52 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AB5C2RIFK7VKS4EPT3J235LR2ODCZANCNFSM4ONZ5FVQ>.
|
|
I was able to reproduce the Linux error on an Ubuntu server I have |
|
Think I found the problem. I added |
|
Looks like it's working. zlib, HDF5, NetCDF, and ESMF build and all tests pass. |
That's awesome, I'll test your PR on my Mac as well. You are still deleting the macos travis file, can you please put it back? I'll fix that failed test later. |
|
Ok, just gotta use GCC-9 for Mac and it should work |
Thanks, will do. Good to know - and we should note that in the release notes for MRW App 1.1 - that GNU 10 won't work. |
|
See here |
|
I will merge this and then I can add the new wgrib2 |
I configure some scripts and run
./configureto build NetCDF, creating only static libs (including HDF5 and zlib).