protocolbuffers/protobuf

Define cmake target as PIC if BUILD_SHARED_LIBS

Open

#10.828 aberto em 27 de out. de 2022

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)C++ (16.128 forks)batch import
c++cmakehelp wanted

Métricas do repositório

Stars
 (71.223 stars)
Métricas de merge de PR
 (Mesclagem média 2d 11h) (185 fundiu PRs em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Examine o CMakeLists.txt do protobuf para localizar onde o target cmake gerado é definido (provavelmente no diretório cmake/). Adicione condicionalmente set property(TARGET ... PROPERTY POSITION INDEPENDENT CODE TRUE) quando BUILD SHARED LIBS estiver habilitado. Verifique se a alteração não entra em conflito com o tratamento de PIC específico da plataforma existente.
Pilha de tecnologia
cpp
Domain
build system
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
cmake
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80