Skip to content

Commit

Permalink
Added namespace prefix to netCDF_LIBRARIES in CMake in support of #1607
Browse files Browse the repository at this point in the history
  • Loading branch information
@WardF
WardF committed Jan 24, 2020
1 parent 13a4207 commit f71434f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion netCDFConfig.cmake.in
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ set_and_check(netCDF_INSTALL_PREFIX "@PACKAGE_CMAKE_INSTALL_PREFIX@")
set_and_check(netCDF_INCLUDE_DIR "@PACKAGE_CMAKE_INSTALL_INCLUDEDIR@")
set_and_check(netCDF_LIB_DIR "@PACKAGE_CMAKE_INSTALL_LIBDIR@")

set(netCDF_LIBRARIES netcdf)
set(netCDF_LIBRARIES netcdf::netcdf)

This comment has been minimized.

Sign in to view

Copy link
@raovgarimella

raovgarimella Jan 28, 2020

I think this should be netCDF::netcdf to be consistent with the namespace used on

netcdf-c/CMakeLists.txt

Line 2038 in 2a34eb2

NAMESPACE netCDF::


# include target information
include("${CMAKE_CURRENT_LIST_DIR}/netCDFTargets.cmake")
Expand Down

0 comments on commit f71434f

Please sign in to comment.