search

Unidata / netcdf-fortran

Official GitHub repository for netCDF-Fortran libraries, which depend on the netCDF C library. Install the netCDF C library first.
Other
240 stars 97 forks source link

Still unable to build correctly with GNU10 #296

Open afernandezody opened 3 years ago

afernandezody commented 3 years ago

Hello, Unfortunately I'm still facing issues when compiling with GNU10. I was following the thread #212 and have tried the master branch, but make check is still outputting:

...
make[3]: Entering directory '/opt/praetorium/netcdf-fortran/nf_test'
make[4]: Entering directory '/opt/praetorium/netcdf-fortran/nf_test'
FAIL: nf_test
FAIL: tst_int64
FAIL: tst_f77_v2
FAIL: ftest
FAIL: nf03_test
FAIL: f03test
FAIL: tst03_f77_v2
============================================================================
Testsuite summary for netCDF-Fortran 4.5.4-development
============================================================================
# TOTAL: 7
# PASS:  0
# SKIP:  0
# XFAIL: 0
# FAIL:  7
# XPASS: 0
# ERROR: 0
============================================================================
See nf_test/test-suite.log
Please report to support-netcdf@unidata.ucar.edu
============================================================================
make[4]: *** [Makefile:982: test-suite.log] Error 1
make[4]: Leaving directory '/opt/praetorium/netcdf-fortran/nf_test'
make[3]: *** [Makefile:1090: check-TESTS] Error 2
make[3]: Leaving directory '/opt/praetorium/netcdf-fortran/nf_test'
make[2]: *** [Makefile:1206: check-am] Error 2
make[2]: Leaving directory '/opt/praetorium/netcdf-fortran/nf_test'
make[1]: *** [Makefile:1208: check] Error 2
make[1]: Leaving directory '/opt/praetorium/netcdf-fortran/nf_test'
make: *** [Makefile:580: check-recursive] Error 1

Thanks.

edwardhartnett commented 3 years ago

There is a merge coming shortly that we hope and believe will address many or all of the GNU10 issues. This will be an extension of the work presented in #294 which was discussed today. Can you wait a few weeks until this is complete, and try again?

afernandezody commented 3 years ago

Closing until merge is available. Will retest then. Thanks.

edwardhartnett commented 3 years ago

OK, you should leave this open until it is fixed.

This will be fixed by #294 (I believe).

However, in the meantime, simply use flag: -fallow-argument-mismatch

Many fortran codes are having to use this flag since the gfortran team introduced this warning.

afernandezody commented 3 years ago

Hi @edwardhartnett, I use the flag "-fallow-argument-mismatch" while compiling a couple of apps and it does its job (of a workaround). However, one app that has netcdf-fortran as a dependency was still complaining about the netcdf-fortran internals (cannot recall the exact error message) and refusing to compile even with the said flag. That's why we hope that the PR will fix the issue. Thanks.