Remove USE_EXTERNAL_CLI11: it downloads CLI11 2.3.2 while the vendored header is 2.5.0
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 85/100
- Tipo de issue
- Refatoração
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Stack de tecnologia
- cmake, cpp
- Domínio
- build-system
Direção de pesquisa
Leia a opção USE_EXTERNAL_CLI11 e o bloco condicional em CMakeLists.txt. Em seguida, inspecione cmake/CLI11.cmake e as referências encontradas com os dois comandos git grep da issue. Remova a opção e o módulo e, depois, execute os comandos listados para a compilação, ctest e a verificação de formatação. A tarefa estará concluída quando o header CLI11 incluído no repositório estiver sendo usado e não houver mais referências à opção ou ao módulo.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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).
- Linguagem predominante
- C++
- Estrelas
- 5
- Forks
- 4
- Merge médio
- 1h 10min
- PRs com merge (30d)
- 79
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de CoreTrace/coretrace
-
chore good first issue severity/minor
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 90/100
Mantenedores costumam responder em até 1 dia
-
build severity/minor
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
CoreTrace/coretrace#176 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
test: add unit tests for ctrace::RunDirectoryTalvez já em andamento @tecnolgd assumiu há 1 dia. Abertagood first issue test
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
CoreTrace/coretrace#164 · 2 comentários · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
Builds without a build type are unoptimized: the README's commands and CI's main jobTalvez já em andamento @SizzleUnrlsd assumiu hoje. Abertabuild severity/minor
CoreTrace/coretrace#192 · 1 responsável ·
Mantenedores costumam responder em até 1 dia
-
bug severity/minor
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 52/100
Mantenedores costumam responder em até 1 dia
Todas as issues de CoreTrace/coretrace
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
FriendsofECCE/ECCE#336 · 1 comentário ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 66/100
bitcoin-core/gui-qml#1030 ·
Mantenedores costumam responder em até 3 dias
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
tesseract-robotics/tesseract#1384 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
NVIDIA/nccl-extensions#26 ·