protocolbuffers/protobuf

Define cmake target as PIC if BUILD_SHARED_LIBS

Open

#10.828 aperta il 27 ott 2022

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C++ (16.128 fork)batch import
c++cmakehelp wanted

Metriche repository

Star
 (71.223 star)
Metriche merge PR
 (Merge medio 2g 11h) (185 PR mergiate in 30 g)

Descrizione

What language does this apply to? Generated C/C++ code and the specific cmake target.

Describe the problem you are trying to solve. When I link to the generated cmake target and set the BUILD_SHARED_LIBS variable globally I get linker errors because the protobuf created code is not relocateable.

Describe the solution you'd like In my user code I use something like the following:

# ensure relocateable code
if (BUILD_SHARED_LIBS)
  set_property(TARGET ${PROTO_TARGET} PROPERTY POSITION_INDEPENDENT_CODE TRUE)
endif ()

But this could be integrated in the cmake part of this project directly.

Describe alternatives you've considered It's not a big problem. But may be helpful for people that do not have that much cmake experience.

Additional context Nothing.

Guida contributor

Direzione di ricerca
Esamina il CMakeLists.txt di protobuf per individuare dove è definito il target cmake generato (probabilmente nella directory cmake/). Aggiungi condizionalmente set property(TARGET ... PROPERTY POSITION INDEPENDENT CODE TRUE) quando BUILD SHARED LIBS è abilitato. Verifica che la modifica non sia in conflitto con la gestione PIC specifica della piattaforma esistente.
Tech stack
cpp
Dominio
build system
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
cmake
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80