[custom] Version-independent installation layout?

Aperta
  • Qualcosa non va?
#6,335 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 3 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
35/100
Tipo di issue
Funzionalità
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
cmake, cpp
Ambito
build-system

Direzione di ricerca

Inizia esaminando pcl_utils.cmake e i file .cmake e .cmake.in coinvolti per individuare i percorsi che contengono la directory di inclusione versionata. Verifica come viene generata la struttura di installazione e se tutti i percorsi correlati sono coperti. Il lavoro è completato quando gli header vengono installati senza la versione di PCL nel percorso di inclusione e il compromesso di compatibilità della proposta è stato risolto.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

status: triage

Hello,

We just want to suggest a small improvement. Currently, the installation layout creates a folder

${CMAKE_INSTALL_PREFIX}/include/pcl-${PCL_VERSION_MAJOR}.${PCL_VERSION_MINOR}/pcl

to install the PCL headers. Surprisingly, nothing else depends on the actual PCL version as the libraries and binaries are simply named pcl_${MODULE}.${EXT}. This is useful to keep depending projects independent of the actual PCL version (assuming there are no API breaking changes) - particularly useful in combination with disabling VTK's VTK_VERSIONED_INSTALL flag, too.

However, PCL's include directory depends on the version, such that depending projects need to be reconfigured (if they are not CMake-based) according to the new include path (e.g. replace pcl-1.4 by pcl-1.5), even though there are no breaking changes, and there seems to be no option to disable this. Is there any particular reason for this setup?

If not, we suggest to remove the versioned include directory as defined in pcl_utils.cmake. A patch is rather easy, we just have to remove the /${PROJECT_NAME_LOWER}-${PCL_VERSION_MAJOR}.${PCL_VERSION_MINOR} parts from the paths in all involved .cmake and .cmake.in files. Maybe this is useful for everyone who wants to manually patch the file... we can prepare a PR for this if you agree to our proposal.

If you want to keep the current layout, feel free to simply close the issue, we are fine with patching the file ourselves, too. Still, we wanted at least to suggest a respective patch.

Lingua principale
C++
Stelle
11.2k
Fork
4.7k
Merge medio
4g 9h
PR unite (30g)
9

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di PointCloudLibrary/pcl

Tutte le issue di PointCloudLibrary/pcl

Issue simili

Altre issue su C++

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.