Remove USE_EXTERNAL_CLI11: it downloads CLI11 2.3.2 while the vendored header is 2.5.0
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 85/100
- Type d'issue
- Refactorisation
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- cmake, cpp
- Domaine
- build-system
Piste de recherche
Lisez l’option USE_EXTERNAL_CLI11 et le bloc conditionnel dans CMakeLists.txt, puis examinez cmake/CLI11.cmake et les références trouvées avec les deux commandes git grep de l’issue. Supprimez l’option et le module, puis exécutez les commandes indiquées pour la compilation, ctest et la vérification du format. Le travail est terminé lorsque l’en-tête CLI11 fourni dans le dépôt est utilisé et qu’il ne reste aucune référence à l’option ni au module.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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).
- Langage dominant
- C++
- Étoiles
- 5
- Forks
- 4
- Merge moyen
- 5 h 56 min
- PR mergées (30 j)
- 65
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de CoreTrace/coretrace
-
chore good first issue severity/minor
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
Les mainteneurs répondent en général sous 1 jour
-
build severity/minor
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
CoreTrace/coretrace#176 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 56/100
Les mainteneurs répondent en général sous 1 jour
-
Merged SARIF log drops rule descriptionsPeut-être pris @SizzleUnrlsd l’a pris il y a 3 jours. Ouverteenhancement severity/minor
CoreTrace/coretrace#207 · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
Inputs: a file name starting with @ may be read as a response file by the compilerPeut-être pris @SizzleUnrlsd l’a pris il y a 6 jours. Ouvertebug severity/minor
CoreTrace/coretrace#203 · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de CoreTrace/coretrace
Issues similaires
-
Plugin storage: send reason codes, not English sentences, so the overview can translate themOuverteenhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
prestonbrown/helixscreen#1867 ·
Les mainteneurs répondent en général sous 2 jours
-
bug P5 XS
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
open-astro/AlpacaBridge#1550 ·
Les mainteneurs répondent en général sous 1 jour
-
Component: Packaging Type: bug
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
LizardByte/libvirtualhid#204 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
Niko1221/Strata#3785 · 2 commentaires ·
Les mainteneurs répondent en général sous 3 jours