-
Notifications
You must be signed in to change notification settings - Fork 263
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Question about whether CMake build provides proper package config file... #1733
Comments
I would really second this!! It would be super fantastic if netcdf installed a cmake and pkg-config file for end users! |
What would be helpful is if some cmake-expert netCDF users would submit a PR with these fixes... |
Hi all, We're using NetCDF with the CMake config file, one difference is the name is |
Good to know about the case-sensitivity re: |
For the record, netCDF does install a pkg-config file, whether you are building via The weakest part of the install are the files used to pull netCDF into other cmake-based projects. As @edwardhartnett said, a PR would be great, but I'll also come up to speed on what it will take to incorporate more of the modern functionality (as opposed to the cmake v2.8 functionality we used when cmake was originally integrated into netcdf). Thanks :) |
Hi @ibaned, are the nitpicks that the transitive dependence is present in the first place? Or is there something we can clean up there to make it more functional? |
@WardF I believe the canonical advice in CMake is that find_dependency(HDF5) if NetCDF is built with HDF5 support. I'm happy to create a PR for this eventually, I have good expertise with CMake but am just short on time to work on it. I appreciate the work done to date on CMake in NetCDF, thank you. |
See #2751 for at least getting the HDF5 bit covered. |
I have this question from a fellow NOAA programmer about the CMake config file for netcdf-c:
The text was updated successfully, but these errors were encountered: