Fix building static libs

Port of microsoft/vcpkg#17506
opengisch · May 22, 2021 · 12f3e54 · 12f3e54
1 parent 8ee2236
commit 12f3e54
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 10 deletions.
diff --git a/include/spatialindex/capi/sidx_export.h b/include/spatialindex/capi/sidx_export.h
@@ -29,7 +29,7 @@
 #pragma once
 
 #ifndef SIDX_C_DLL
-#if defined(_MSC_VER)
+#if defined(_MSC_VER) && defined(SIDX_DLL_EXPORT)
 #  define SIDX_C_DLL     __declspec(dllexport)
 #  define SIDX_DLL     __declspec(dllexport)
 #else

diff --git a/include/spatialindex/tools/Tools.h b/include/spatialindex/tools/Tools.h
@@ -46,7 +46,7 @@
   #ifdef SIDX_DLL_EXPORT
     #define SIDX_DLL __declspec(dllexport)
   #else
-    #define SIDX_DLL __declspec(dllimport)
+    #define SIDX_DLL
   #endif
 
   // Nuke this annoying warning.  See http://www.unknownroad.com/rtfm/VisualStudio/warningC4251.html

diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
@@ -178,9 +178,13 @@ set(SIDX_SOURCES
   ${SIDX_HPP}
   ${SIDX_CPP})
 
-add_library(spatialindex ${SIDX_SOURCES})
-
-add_library(spatialindex_c ${SIDX_CAPI_CPP})
+if(BUILD_SHARED_LIBS)
+  add_library(spatialindex SHARED ${SIDX_SOURCES})
+  add_library(spatialindex_c SHARED ${SIDX_CAPI_CPP})
+else(BUILD_SHARED_LIBS)
+  add_library(spatialindex STATIC ${SIDX_SOURCES})
+  add_library(spatialindex_c STATIC ${SIDX_CAPI_CPP})
+endif(BUILD_SHARED_LIBS)
 
 target_link_libraries(spatialindex_c spatialindex)
 
@@ -204,11 +208,12 @@ if(MSVC)
   target_compile_options(spatialindex PRIVATE /wd4068)
   target_compile_options(spatialindex_c PRIVATE /wd4068)
 
-  target_compile_definitions(spatialindex_c PRIVATE -DSIDX_DLL_EXPORT=1)
-  target_compile_definitions(spatialindex PRIVATE -DSIDX_DLL_EXPORT=1)
-  if(NOT WITH_STATIC_SIDX)
-    target_compile_definitions(spatialindex PRIVATE -DSIDX_DLL_IMPORT=1)
-    target_compile_definitions(spatialindex_c PRIVATE -DSIDX_DLL_IMPORT=1)
+ if (BUILD_SHARED_LIBS)
+   target_compile_definitions(spatialindex PRIVATE -DSIDX_DLL_EXPORT=1)
+   target_compile_definitions(spatialindex_c PRIVATE -DSIDX_DLL_EXPORT=1)
+ else()
+  target_compile_definitions(spatialindex PRIVATE -DSIDX_DLL_IMPORT=1)
+  target_compile_definitions(spatialindex_c PRIVATE -DSIDX_DLL_IMPORT=1)
   endif()
 endif()