Skip to content

Commit

Permalink
Merge #303: cmake: Drop extraneous -fvisibility-inlines-hidden
Browse files Browse the repository at this point in the history
4dd3928 fixup! cmake: Add `REDUCE_EXPORTS` option (Hennadii Stepanov)

Pull request description:

  This PR mirrors the master branch behavior and addresses bitcoin#30454 (comment).

ACKs for top commit:
  m3dwards:
    ACK 4dd3928

Tree-SHA512: 329800301803c86e40eed9aa356ed7c0cd17207f0dc4a481e72705d45bf72cf12a0b5ec48de8133f9760f23930528b93440f38e2e47e4740eb1a811ba3c964e6
  • Loading branch information
hebasto committed Aug 5, 2024
2 parents 874e21e + 4dd3928 commit 2feccd4
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -528,7 +528,6 @@ endif()

if(REDUCE_EXPORTS)
set(CMAKE_CXX_VISIBILITY_PRESET hidden)
set(CMAKE_VISIBILITY_INLINES_HIDDEN 1)
try_append_linker_flag("-Wl,--exclude-libs,ALL" TARGET core_interface)
try_append_linker_flag("-Wl,-no_exported_symbols" VAR CMAKE_EXE_LINKER_FLAGS)
endif()
Expand Down

0 comments on commit 2feccd4

Please sign in to comment.