Consider EXCLUDE_FROM_ALL for CPM to avoid packaging ADS static library/includes

Aperta Adatta ai principianti
  • Qualcosa non va?
#1,189 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
78/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
cmake, cpp
Ambito
build-system

Direzione di ricerca

Leggi thirdparty/CMakeLists.txt e verifica in che modo le chiamate a CPMAddPackage e add_subdirectory influiscono sulla build Linux e sul packaging. Applica le esclusioni descritte nell’issue, quindi verifica che il pacchetto risultante non contenga più la cartella di inclusione ADS né la libreria statica non necessarie.

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

Descrizione

Description

Linux build produces folders include/qtadvancedocking-qt6 and lib/libqtadvancedocking-qt6_static.a which are not required.

Describe the solution you'd like
diff --git a/thirdparty/CMakeLists.txt b/thirdparty/CMakeLists.txt
index 5d128d86..526bf9b6 100644
--- a/thirdparty/CMakeLists.txt
+++ b/thirdparty/CMakeLists.txt
@@ -1,6 +1,7 @@
 include(${CMAKE_SOURCE_DIR}/cmake/CPM.cmake)
 
 CPMAddPackage(
+    EXCLUDE_FROM_ALL YES
     NAME ads
     GITHUB_REPOSITORY githubuser0xFFFF/Qt-Advanced-Docking-System
     GIT_TAG v5.1.1
@@ -10,6 +11,7 @@ CPMAddPackage(
 )
 
 CPMAddPackage(
+    EXCLUDE_FROM_ALL YES
     NAME QSimpleUpdater
     GITHUB_REPOSITORY alex-spataru/QSimpleUpdater
     GIT_TAG c5c136c0f79ca6423b0914a2676ae3d0e3811c5b
@@ -19,20 +21,22 @@ CPMAddPackage(
 
 set(QAPPLICATION_CLASS QApplication CACHE STRING "Inheritance class for SingleApplication" FORCE)
 CPMAddPackage(
+    EXCLUDE_FROM_ALL YES
     NAME SingleApplication
     GITHUB_REPOSITORY itay-grudev/SingleApplication
     GIT_TAG v3.5.6
 )
 
 CPMAddPackage(
+    EXCLUDE_FROM_ALL YES
     NAME editorconfig_core_qt
     GITHUB_REPOSITORY editorconfig/editorconfig-core-qt
     GIT_TAG ab62f0554abf2bbe4d45427b36a8b2f81ca7b4ab
 )
 
-add_subdirectory(lua)
-add_subdirectory(scintilla)
-add_subdirectory(lexilla)
-add_subdirectory(uchardet)
+add_subdirectory(lua EXCLUDE_FROM_ALL)
+add_subdirectory(scintilla EXCLUDE_FROM_ALL)
+add_subdirectory(lexilla EXCLUDE_FROM_ALL)
+add_subdirectory(uchardet EXCLUDE_FROM_ALL)
 
 target_link_libraries(lexilla PRIVATE scintilla)
Describe alternatives you've considered

Arch PKGBUILD automated regex:

perl -pi -e 's/(add_subdirectory.*)(?=\))/\1 EXCLUDE_FROM_ALL/gi ; s/(CPMAddPackage\(.*)/\1\n    EXCLUDE_FROM_ALL YES/gi' "$pkgname"/thirdparty/CMakeLists.txt
Lingua principale
C++
Stelle
14.8k
Fork
943
Merge medio
2g 7h
PR unite (30g)
5

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

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 dail8859/NotepadNext

Tutte le issue di dail8859/NotepadNext

Issue simili

Altre issue su C++

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.