PointCloudLibrary/pcl

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

Open

#2 607 ouverte le 9 nov. 2018

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)C++ (4 506 forks)batch import
good first issuekind: todomodule: tutorialsskill: cmake

Métriques du dépôt

Stars
 (9 023 stars)
Métriques de merge PR
 (Merge moyen 8j 7h) (3 PRs mergées en 30 j)

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

Guide contributeur

Direction de recherche
Identifiez tous les fichiers CMakeLists.txt des didacticiels dans le référentiel et mettez les à jour pour utiliser le CMake moderne (target include directories, target link libraries, etc.) au lieu d'appels obsolètes comme include directories.
Stack technique
cpp
Domaine
build system
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
CMake
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70