Remove USE_EXTERNAL_CLI11: it downloads CLI11 2.3.2 while the vendored header is 2.5.0
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 85/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- cmake, cpp
- Área
- build-system
Línea de trabajo
Lee la opción USE_EXTERNAL_CLI11 y el bloque condicional en CMakeLists.txt. Después, inspecciona cmake/CLI11.cmake y las referencias encontradas con los dos comandos git grep del issue. Elimina la opción y el módulo y, a continuación, ejecuta los comandos indicados para la compilación, ctest y la comprobación de formato. La tarea estará terminada cuando se use el encabezado CLI11 incluido en el repositorio y no queden referencias a la opción ni al módulo.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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).
- Lenguaje dominante
- C++
- Estrellas
- 5
- Forks
- 4
- Merge medio
- 1 h 10 min
- PR fusionados (30 d)
- 79
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de CoreTrace/coretrace
-
chore good first issue severity/minor
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
build severity/minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
CoreTrace/coretrace#176 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
test: add unit tests for ctrace::RunDirectoryPosiblemente ocupada @tecnolgd la tomó hace 2 días. Abiertogood first issue test
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
CoreTrace/coretrace#164 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Builds without a build type are unoptimized: the README's commands and CI's main jobPosiblemente ocupada @SizzleUnrlsd la tomó hace 1 día. Abiertobuild severity/minor
CoreTrace/coretrace#192 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
bug severity/minor
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
Los mantenedores suelen responder en 1 día
Todos los issues de CoreTrace/coretrace
Issues similares
-
agent:Windows bug code-review LOW testing
Dificultad 2/5 1-3 horas Aptitud para principiantes 63/100
Los mantenedores suelen responder en 1 día
-
[Cleanup] Include the public header in work_split.cppPosiblemente ocupada @blozano-tt la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
tenstorrent/tt-metal#60850 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
beehive-lab/TornadoVM#1248 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
SimVascular/svZeroDSolver#253 ·
-
core: RAC_LLM_OPTIONS_DEFAULT.min_p disagrees with the IDL's annotated defaultPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertobug core p3
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
RunanywhereAI/runanywhere-sdks#1083 ·
Los mantenedores suelen responder en 5 días