[tinyxml2]: do not force export the symbols when building statically

leethomason/tinyxml2#922
microsoft · Oct 28, 2022 · 55179ca · 55179ca
1 parent 48cc0c7
commit 55179ca
Show file tree

Hide file tree

Showing 5 changed files with 54 additions and 2 deletions.
diff --git a/ports/tinyxml2/0001-fix-do-not-force-export-the-symbols-when-building-st.patch b/ports/tinyxml2/0001-fix-do-not-force-export-the-symbols-when-building-st.patch
@@ -0,0 +1,45 @@
+From a512d312db5604efe803a6fe088e7d582e04533f Mon Sep 17 00:00:00 2001
+From: Amin Yahyaabadi <aminyahyaabadi74@gmail.com>
+Date: Wed, 26 Oct 2022 18:21:29 -0700
+Subject: [PATCH] fix: do not force export the symbols when building statically
+
+---
+ CMakeLists.txt | 11 ++++++++---
+ 1 file changed, 8 insertions(+), 3 deletions(-)
+
+diff --git a/CMakeLists.txt b/CMakeLists.txt
+index 8802fb8..87cda90 100644
+--- a/CMakeLists.txt
++++ b/CMakeLists.txt
+@@ -16,8 +16,10 @@ endif ()
+ ## Main library build
+ ##
+
+-set(CMAKE_CXX_VISIBILITY_PRESET hidden)
+-set(CMAKE_VISIBILITY_INLINES_HIDDEN YES)
++if (BUILD_SHARED_LIBS)
++    set(CMAKE_CXX_VISIBILITY_PRESET hidden)
++    set(CMAKE_VISIBILITY_INLINES_HIDDEN YES)
++endif()
+
+ add_library(tinyxml2 tinyxml2.cpp tinyxml2.h)
+ add_library(tinyxml2::tinyxml2 ALIAS tinyxml2)
+@@ -36,11 +38,14 @@ target_compile_definitions(
+ set_target_properties(
+     tinyxml2
+     PROPERTIES
+-    DEFINE_SYMBOL "TINYXML2_EXPORT"
+     VERSION "${tinyxml2_VERSION}"
+     SOVERSION "${tinyxml2_VERSION_MAJOR}"
+ )
+
++if(BUILD_SHARED_LIBS)
++    target_compile_definitions(tinyxml2 PRIVATE "TINYXML2_EXPORT")
++endif()
++
+ if (tinyxml2_BUILD_TESTING)
+     add_executable(xmltest xmltest.cpp)
+     target_link_libraries(xmltest PRIVATE tinyxml2::tinyxml2)
+-- 
+2.37.3.windows.1
+
diff --git a/ports/tinyxml2/portfile.cmake b/ports/tinyxml2/portfile.cmake
@@ -4,6 +4,8 @@ vcpkg_from_github(
     REF 9.0.0
     SHA512 9C5CE8131984690DF302CA3E32314573B137180ED522C92FD631692979C942372A28F697FDB3D5E56BCF2D3DC596262B724D088153F3E1D721C9536F2A883367
     HEAD_REF master
+    PATCHES
+        0001-fix-do-not-force-export-the-symbols-when-building-st.patch
 )
 
 vcpkg_cmake_configure(

diff --git a/ports/tinyxml2/vcpkg.json b/ports/tinyxml2/vcpkg.json
@@ -1,7 +1,7 @@
 {
   "name": "tinyxml2",
   "version-semver": "9.0.0",
-  "port-version": 1,
+  "port-version": 2,
   "description": "A simple, small, efficient, C++ XML parser",
   "homepage": "https://github.com/leethomason/tinyxml2",
   "license": "Zlib",

diff --git a/versions/baseline.json b/versions/baseline.json
@@ -7438,7 +7438,7 @@
     },
     "tinyxml2": {
       "baseline": "9.0.0",
-      "port-version": 1
+      "port-version": 2
     },
     "tl-expected": {
       "baseline": "1.0.0",

diff --git a/versions/t-/tinyxml2.json b/versions/t-/tinyxml2.json
@@ -1,5 +1,10 @@
 {
   "versions": [
+    {
+      "git-tree": "29c1c3aaa87957356e4c83b54767b80e3dd89fa2",
+      "version-semver": "9.0.0",
+      "port-version": 2
+    },
     {
       "git-tree": "5ef0e856167ad66665f51776d78fcffb1fcc1cb1",
       "version-semver": "9.0.0",