PointCloudLibrary/pcl

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

Open

#2607 aperta il 9 nov 2018

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C++ (4506 fork)batch import
good first issuekind: todomodule: tutorialsskill: cmake

Metriche repository

Star
 (9023 star)
Metriche merge PR
 (Merge medio 8g 7h) (3 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Identifica tutti i file CMakeLists.txt dei tutorial nel repository e aggiornali per utilizzare CMake moderno (target include directories, target link libraries, ecc.) invece di chiamate deprecate come include directories.
Tech stack
cpp
Dominio
build system
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
CMake
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70