PointCloudLibrary/pcl

Update CMake scripts in tutorials (remove superfluous include_directories et al. calls)

Open

#2.607 aberto em 9 de nov. de 2018

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)C++ (4.506 forks)batch import
good first issuekind: todomodule: tutorialsskill: cmake

Métricas do repositório

Stars
 (9.023 stars)
Métricas de merge de PR
 (Mesclagem média 8d 7h) (3 fundiu PRs em 30d)

Description

I was thinking about the CMakeLists in tutorials, do they need to be updated as well? My conclusion was no, not yet. It should still be possible to use PCL in downstream projects with old CMake. But once we switch to C++14 we should consider bumping version there and requesting CXX_14 explicitly.

Originally posted by @taketwo in https://github.com/PointCloudLibrary/pcl/pull/2605#issuecomment-437301261

Guia do colaborador

Direção de pesquisa
Identifique todos os arquivos CMakeLists.txt de tutoriais no repositório e atualize os para usar o CMake moderno (target include directories, target link libraries, etc.) em vez de chamadas obsoletas como include directories.
Pilha de tecnologia
cpp
Domain
build system
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
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.
70