Remove USE_EXTERNAL_CLI11: it downloads CLI11 2.3.2 while the vendored header is 2.5.0
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
- 85/100
- Tipo di issue
- Refactoring
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- cmake, cpp
- Ambito
- build-system
Direzione di ricerca
Leggi l’opzione USE_EXTERNAL_CLI11 e il blocco condizionale in CMakeLists.txt, quindi esamina cmake/CLI11.cmake e i riferimenti trovati con i due comandi git grep riportati nell’issue. Rimuovi l’opzione e il modulo, poi esegui i comandi elencati per la compilazione, ctest e il controllo della formattazione. Il lavoro è completato quando viene usato l’header CLI11 incluso nel repository e non rimangono riferimenti all’opzione o al modulo.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
ctrace parses its command line with the CLI11 header vendored in external/CLI11.hpp, version 2.5.0. With -DUSE_EXTERNAL_CLI11=ON (CMakeLists.txt:115-120), CMake downloads CLI11 v2.3.2 instead (cmake/CLI11.cmake): an older version that CI never builds, while the release license check reads the vendored header (CMakeLists.txt:209-214). The option gives two possible versions of the same dependency, and nothing needs it.
To do: remove the option and keep the vendored header.
How:
- delete
cmake/CLI11.cmake; - delete the
USE_EXTERNAL_CLI11option and itsifblock inCMakeLists.txt(lines 115 to 120); - run
git grep -n USE_EXTERNAL_CLI11andgit grep -n CLI11.cmaketo check that nothing else refers to them.
The wiki's build page also lists the option; a maintainer will update it.
Check:
cmake --build build --parallel 4
ctest --test-dir build --output-on-failure
./scripts/format-check.sh
Acceptance
cmake/CLI11.cmakeis gone, andgit grep USE_EXTERNAL_CLI11finds nothing.- The README build configures and builds, and
ctestpasses.
From the technical debt register (docs/quality/technical-debt.md, D8).
- Lingua principale
- C++
- Stelle
- 5
- Fork
- 4
- Merge medio
- 1h 9m
- PR unite (30g)
- 81
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di CoreTrace/coretrace
-
chore good first issue severity/minor
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
I maintainer di solito rispondono entro 1 giorno
-
build severity/minor
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
CoreTrace/coretrace#176 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
test: add unit tests for ctrace::RunDirectoryForse già presa @tecnolgd l’ha presa 2 giorni fa. Apertagood first issue test
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
CoreTrace/coretrace#164 · 2 commenti · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Release a macOS disk image (.dmg) for Apple siliconForse già presa @SizzleUnrlsd l’ha presa oggi. Apertabuild enhancement
CoreTrace/coretrace#198 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Builds without a build type are unoptimized: the README's commands and CI's main jobForse già presa @SizzleUnrlsd l’ha presa 1 giorno fa. Apertabuild severity/minor
CoreTrace/coretrace#192 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di CoreTrace/coretrace
Issue simili
-
bot-triaged module: correctness (silent) module: mps module: NaNs and Infs triage review
Difficoltà 2/5 Mezza giornata Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno
-
README: CI link/badge checkAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
SCIInstitute/SCIRun#2842 ·
I maintainer di solito rispondono entro 3 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
agent:Windows bug code-review LOW testing
Difficoltà 2/5 1-3 ore Idoneità per principianti 63/100
I maintainer di solito rispondono entro 1 giorno