Skip to content

Commit 0fe89cb

Browse files
committed
temporarily fix issue with avro by disabling hidden visibility
1 parent 0186975 commit 0fe89cb

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

cmake_modules/IcebergBuildUtils.cmake

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -152,10 +152,6 @@ function(add_iceberg_lib LIB_NAME)
152152

153153
string(TOUPPER ${LIB_NAME} VISIBILITY_NAME)
154154
target_compile_definitions(${LIB_NAME}_shared PRIVATE ${VISIBILITY_NAME}_EXPORTING)
155-
set_target_properties(${LIB_NAME}_shared
156-
PROPERTIES C_VISIBILITY_PRESET hidden CXX_VISIBILITY_PRESET
157-
hidden
158-
VISIBILITY_INLINES_HIDDEN 1)
159155

160156
install(TARGETS ${LIB_NAME}_shared
161157
EXPORT iceberg_targets

src/iceberg/CMakeLists.txt

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -135,6 +135,13 @@ add_iceberg_lib(iceberg
135135
OUTPUTS
136136
ICEBERG_LIBRARIES)
137137

138+
if(ICEBERG_BUILD_SHARED)
139+
set_target_properties(iceberg_shared
140+
PROPERTIES C_VISIBILITY_PRESET hidden
141+
CXX_VISIBILITY_PRESET hidden VISIBILITY_INLINES_HIDDEN
142+
1)
143+
endif()
144+
138145
iceberg_install_all_headers(iceberg)
139146

140147
add_subdirectory(catalog)

0 commit comments

Comments
 (0)