CMake: iconv on OSX (#4851)

Check for iconv_open availability in libiconv on OSX due to differences in ICONV builds on that platform
OSGeo · Nov 18, 2021 · a663c88 · a663c88
1 parent 97a0899
commit a663c88
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/cmake/helpers/CheckDependentLibraries.cmake b/cmake/helpers/CheckDependentLibraries.cmake
@@ -13,6 +13,7 @@ include(CheckFunctionExists)
 include(CMakeDependentOption)
 include(FeatureSummary)
 include(DefineFindPackage2)
+include(CheckSymbolExists)
 
 # Macro to declare a package
 # Accept a CAN_DISABLE option to specify that the package can be disabled

diff --git a/port/CMakeLists.txt b/port/CMakeLists.txt
@@ -179,8 +179,14 @@ if (GDAL_USE_ICONV)
     target_sources(cpl PRIVATE cpl_recode_iconv.cpp)
   else ()
     if (APPLE)
-      target_compile_definitions(cpl PRIVATE -DLIBICONV_PLUG)
-    endif ()
+      # linkage against OSX's system iconv is going to require LIBICONV_PLUG to be defined, but other self-built
+      # situations like CondaForge or Homebrew might not need this
+      set(CMAKE_REQUIRED_LIBRARIES ${Iconv_LIBRARY})
+      check_symbol_exists(_iconv_close "iconv.h" HAVE_ICONV_CLOSE)
+      if (HAVE_ICONV_CLOSE)
+        target_compile_definitions(cpl PRIVATE -DLIBICONV_PLUG)
+      endif (HAVE_ICONV_CLOSE)
+    endif (APPLE)
     target_sources(cpl PRIVATE cpl_recode_iconv.cpp)
     target_include_directories(cpl PRIVATE $<TARGET_PROPERTY:Iconv::Iconv,INTERFACE_INCLUDE_DIRECTORIES>)
     gdal_add_private_link_libraries(Iconv::Iconv)