search

MetOffice / opsinputs

JEDI library generating VarObs and Cx files
BSD 3-Clause "New" or "Revised" License
4 stars 1 forks source link

Use jedi-cmake. #136

Closed DJDavies2 closed 2 years ago

DJDavies2 commented 2 years ago

I have added a find_package for jedi-cmake. The reason for this is to use the jedi-cmake version of FindNetCDF instead of the ecbuild version. This is beneficial for situations where NetCDF C and NetCDF Fortran are installed in separate locations; the ecbuild version doesn't handle this very well and the jedi-cmake version is better. It makes no difference for situations where NetCDF C, Fortran and C++ are installed in the same location, such as for bb.

DJDavies2 commented 2 years ago

The CI failure is unrelated.

I don't think I can merge at the moment.

DJDavies2 commented 2 years ago

Does someone know how to progress this? I don't think the failure in the CI that is blocking this is related to the change.

ctgh commented 2 years ago

I forced the merge into develop. Please let me know if there are any problems.