Skip to content

Commit

Permalink
Add option YAML_ENABLE_PIC=ON
Browse files Browse the repository at this point in the history
  • Loading branch information
@humbertodias @jbeder
humbertodias authored and jbeder committed Feb 22, 2024
1 parent 5c1bbbd commit 76dc671
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions CMakeLists.txt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ option(YAML_CPP_INSTALL "Enable generation of yaml-cpp install targets" ${YAML_C
option(YAML_CPP_FORMAT_SOURCE "Format source" ${YAML_CPP_MAIN_PROJECT})
option(YAML_CPP_DISABLE_UNINSTALL "Disable uninstallation of yaml-cpp" OFF)
option(YAML_USE_SYSTEM_GTEST "Use system googletest if found" OFF)
option(YAML_ENABLE_PIC "Use Position-Independent Code " ON)

cmake_dependent_option(YAML_CPP_BUILD_TESTS
"Enable yaml-cpp tests" OFF
Expand Down Expand Up @@ -91,9 +92,7 @@ set_property(TARGET yaml-cpp
CXX_STANDARD_REQUIRED ON)

if (NOT YAML_BUILD_SHARED_LIBS)
if (NOT PSP AND NOT PS3)
set_property(TARGET yaml-cpp PROPERTY POSITION_INDEPENDENT_CODE ON)
endif()
set_property(TARGET yaml-cpp PROPERTY POSITION_INDEPENDENT_CODE ${YAML_ENABLE_PIC})
endif()

target_include_directories(yaml-cpp
Expand Down

1 comment on commit 76dc671

@lorrden
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the canonical CMAKE way for handling this not the variable CMAKE_POSITION_INDEPENDENT_CODE?

Please sign in to comment.