Merge pull request #2912 from rouault/cmake_static_build

CMake: revise how we deal with symbol export and static builds

10 files changed, 17 insertions, 52 deletions

diff --git a/src/bin_cct.cmake b/src/bin_cct.cmake
index 39dfb2bc..85ce490c 100644
--- a/src/bin_cct.cmake
+++ b/src/bin_cct.cmake
@@ -13,7 +13,3 @@ target_compile_options(cct PRIVATE ${PROJ_CXX_WARN_FLAGS})
 
 install(TARGETS cct
   DESTINATION ${BINDIR})
-
-if(MSVC AND BUILD_SHARED_LIBS)
-  target_compile_definitions(cct PRIVATE PROJ_MSVC_DLL_IMPORT=1)
-endif()
diff --git a/src/bin_cs2cs.cmake b/src/bin_cs2cs.cmake
index 2fc8a6ad..bffc6503 100644
--- a/src/bin_cs2cs.cmake
+++ b/src/bin_cs2cs.cmake
@@ -12,7 +12,3 @@ target_compile_options(cs2cs PRIVATE ${PROJ_CXX_WARN_FLAGS})
 
 install(TARGETS cs2cs
   DESTINATION ${BINDIR})
-
-if(MSVC AND BUILD_SHARED_LIBS)
-  target_compile_definitions(cs2cs PRIVATE PROJ_MSVC_DLL_IMPORT=1)
-endif()
diff --git a/src/bin_geod.cmake b/src/bin_geod.cmake
index 3f3f9bf5..6f081634 100644
--- a/src/bin_geod.cmake
+++ b/src/bin_geod.cmake
@@ -15,10 +15,6 @@ target_compile_options(geod PRIVATE ${PROJ_CXX_WARN_FLAGS})
 install(TARGETS geod
   DESTINATION ${BINDIR})
 
-if(MSVC AND BUILD_SHARED_LIBS)
-  target_compile_definitions(geod PRIVATE PROJ_MSVC_DLL_IMPORT=1)
-endif()
-
 # invgeod target: symlink or copy of geod executable
 
 if(UNIX)
@@ -48,8 +44,4 @@ else()
     install(TARGETS invgeod
       DESTINATION ${BINDIR})
 
-    if(MSVC AND BUILD_SHARED_LIBS)
-      target_compile_definitions(invgeod PRIVATE PROJ_MSVC_DLL_IMPORT=1)
-    endif()
-
 endif()
diff --git a/src/bin_geodtest.cmake b/src/bin_geodtest.cmake
index ab366144..8687a2bc 100644
--- a/src/bin_geodtest.cmake
+++ b/src/bin_geodtest.cmake
@@ -9,7 +9,3 @@ target_compile_options(geodtest PRIVATE ${PROJ_CXX_WARN_FLAGS})
 
 # Do not install, instead run as a test
 add_test(NAME geodesic-test COMMAND geodtest)
-
-if(MSVC AND BUILD_SHARED_LIBS)
-  target_compile_definitions(geodtest PRIVATE PROJ_MSVC_DLL_IMPORT=1)
-endif()
diff --git a/src/bin_gie.cmake b/src/bin_gie.cmake
index 6ad7d4ab..853163f5 100644
--- a/src/bin_gie.cmake
+++ b/src/bin_gie.cmake
@@ -15,7 +15,3 @@ if(BUILD_GIE)
   install(TARGETS gie
     DESTINATION ${BINDIR})
 endif()
-
-if(MSVC AND BUILD_SHARED_LIBS)
-  target_compile_definitions(gie PRIVATE PROJ_MSVC_DLL_IMPORT=1)
-endif()
diff --git a/src/bin_proj.cmake b/src/bin_proj.cmake
index 2df467af..931f3f12 100644
--- a/src/bin_proj.cmake
+++ b/src/bin_proj.cmake
@@ -16,10 +16,6 @@ target_compile_options(binproj PRIVATE ${PROJ_CXX_WARN_FLAGS})
 install(TARGETS binproj
   DESTINATION ${BINDIR})
 
-if(MSVC AND BUILD_SHARED_LIBS)
-  target_compile_definitions(binproj PRIVATE PROJ_MSVC_DLL_IMPORT=1)
-endif()
-
 # invproj target: symlink or copy of proj executable
 
 if(UNIX)
@@ -49,8 +45,4 @@ else()
     install(TARGETS invproj
       DESTINATION ${BINDIR})
 
-    if(MSVC AND BUILD_SHARED_LIBS)
-      target_compile_definitions(invproj PRIVATE PROJ_MSVC_DLL_IMPORT=1)
-    endif()
-
 endif()
diff --git a/src/bin_projinfo.cmake b/src/bin_projinfo.cmake
index 6d123436..c2449049 100644
--- a/src/bin_projinfo.cmake
+++ b/src/bin_projinfo.cmake
@@ -12,10 +12,6 @@ target_compile_options(binprojinfo PRIVATE ${PROJ_CXX_WARN_FLAGS})
 install(TARGETS binprojinfo
   DESTINATION ${BINDIR})
 
-if(MSVC AND BUILD_SHARED_LIBS)
-  target_compile_definitions(binprojinfo PRIVATE PROJ_MSVC_DLL_IMPORT=1)
-endif()
-
 if(CURL_ENABLED)
   target_compile_definitions(binprojinfo PRIVATE -DCURL_ENABLED)
 endif()
diff --git a/src/bin_projsync.cmake b/src/bin_projsync.cmake
index 6486b5d4..7ab7c43c 100644
--- a/src/bin_projsync.cmake
+++ b/src/bin_projsync.cmake
@@ -11,7 +11,3 @@ target_compile_options(bin_projsync PRIVATE ${PROJ_CXX_WARN_FLAGS})
 
 install(TARGETS bin_projsync
   DESTINATION ${BINDIR})
-
-if(MSVC AND BUILD_SHARED_LIBS)
-  target_compile_definitions(bin_projsync PRIVATE PROJ_MSVC_DLL_IMPORT=1)
-endif()
diff --git a/src/lib_proj.cmake b/src/lib_proj.cmake
index 4ece9094..efad5552 100644
--- a/src/lib_proj.cmake
+++ b/src/lib_proj.cmake
@@ -477,9 +477,12 @@ if(CURL_ENABLED)
       $<$<CXX_COMPILER_ID:MSVC>:normaliz>)
 endif()
 
-if(MSVC AND BUILD_SHARED_LIBS)
-  target_compile_definitions(proj
-    PRIVATE PROJ_MSVC_DLL_EXPORT=1)
+if(BUILD_SHARED_LIBS)
+  if(MSVC)
+    target_compile_definitions(proj PRIVATE PROJ_MSVC_DLL_EXPORT=1)
+  endif()
+else()
+  target_compile_definitions(proj PUBLIC PROJ_DLL=)
 endif()
 
 ##############################################
diff --git a/src/proj.h b/src/proj.h
index 92515131..a09ae35c 100644
--- a/src/proj.h
+++ b/src/proj.h
@@ -140,15 +140,17 @@ extern "C" {
 /*! @cond Doxygen_Suppress */
 
 #ifndef PROJ_DLL
-#ifdef PROJ_MSVC_DLL_EXPORT
-#define PROJ_DLL __declspec(dllexport)
-#elif defined(PROJ_MSVC_DLL_IMPORT)
-#define PROJ_DLL __declspec(dllimport)
-#elif defined(__GNUC__)
-#define PROJ_DLL __attribute__ ((visibility("default")))
-#else
-#define PROJ_DLL
-#endif
+#  if defined(_MSC_VER)
+#    ifdef PROJ_MSVC_DLL_EXPORT
+#      define PROJ_DLL __declspec(dllexport)
+#    else
+#      define PROJ_DLL __declspec(dllimport)
+#    endif
+#  elif defined(__GNUC__)
+#    define PROJ_DLL __attribute__ ((visibility("default")))
+#  else
+#    define PROJ_DLL
+#  endif
 #endif
 
 #ifdef PROJ_SUPPRESS_DEPRECATION_MESSAGE