Skip to content
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

Allow users to set NetCDF_[C|Fortran]_PATHS #459

Open
jayeshkrishna opened this issue Feb 24, 2022 · 0 comments
Open

Allow users to set NetCDF_[C|Fortran]_PATHS #459

jayeshkrishna opened this issue Feb 24, 2022 · 0 comments
Assignees

Comments

@jayeshkrishna
Copy link
Contributor

jayeshkrishna commented Feb 24, 2022

Currently users are recommended to use NetCDF_PATH or NetCDF_[C|Fortran]_PATH to set the paths to the NetCDF libraries. Although NetCDF_[C|Fortran]_PATHS is recognized by the find_package_component() cmake function (in cmake/LibFind.cmake), that is used to find the NetCDF library/package, there are assumptions in the code that its never set by the user (the user invoking cmake).

In FindNetCDF.cmake NetCDF_[C|Fortran]_PATHS is initialized (in certain conditions). Instead of initializing NetCDF_[C|Fortran]_PATHS directly a temporary variable could be initialized and appended to NetCDF_[C|Fortran]_PATHS ensuring that the paths set by the user is not overridden.

Also see Issue #456

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants