Commit 7897c65f authored by Léonard Gérard's avatar Léonard Gérard Committed by Facebook Github Bot

Remove implied POSITION_INDEPENDENT_CODE property

Summary:
PIC is implied by BUILD_SHARED_LIBS in CMake, so the test is confusing.
Note that if one wants to build static and use it to create a shared library with it, he will need to compile either shared
or static with CMake option -DCMAKE_POSITION_INDEPENDENT_CODE
Closes https://github.com/facebook/folly/pull/833

Reviewed By: Orvid

Differential Revision: D7890111

Pulled By: yfeldblum

fbshipit-source-id: e3996cb83d30ae368dc5fdd4f55b721f7f89d0a0
parent 54ee5225
...@@ -194,9 +194,6 @@ add_library(folly_base OBJECT ...@@ -194,9 +194,6 @@ add_library(folly_base OBJECT
${files} ${hfiles} ${files} ${hfiles}
${CMAKE_CURRENT_BINARY_DIR}/folly/folly-config.h ${CMAKE_CURRENT_BINARY_DIR}/folly/folly-config.h
) )
if (BUILD_SHARED_LIBS)
set_property(TARGET folly_base PROPERTY POSITION_INDEPENDENT_CODE ON)
endif()
auto_source_group(folly ${FOLLY_DIR} ${files} ${hfiles}) auto_source_group(folly ${FOLLY_DIR} ${files} ${hfiles})
apply_folly_compile_options_to_target(folly_base) apply_folly_compile_options_to_target(folly_base)
# Add the generated files to the correct source group. # Add the generated files to the correct source group.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment