Optional tools are fetched from master, and cmake/ikos.cmake is included nowhere
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 69/100
- Tipo di issue
- Refactoring
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Ambito
- build-system, devops
Direzione di ricerca
Read the three named CMake modules, scripts/release/check-pinned-dependencies.sh, and the relevant section of CMakeLists.txt. Pin flawfinder and TscanCode, update the script exclusions, and remove the unused ikos module and commented include. Run check-pinned-dependencies.sh, ctest, and configure and build with both options enabled to verify the acceptance criteria.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Three build modules fetch GIT_TAG master:
cmake/flawfinder.cmake:7(-DENABLE_FLAWFINDER=ON);cmake/tscancode.cmake:7(-DENABLE_TSCANCODE=ON);cmake/ikos.cmake:55, which nothing includes: itsincludeis commented out inCMakeLists.txt:144, and ikos is found throughPATH.
scripts/release/check-pinned-dependencies.sh skips these three files because no release ships those tools. As a result, two builds of the same commit with the same option can get different versions of flawfinder or TscanCode, and cmake/ikos.cmake is dead code.
To do
- Pin flawfinder and TscanCode to a tag or a commit, like the other dependencies.
- Remove them from the exclusion list of
check-pinned-dependencies.sh. - Delete
cmake/ikos.cmakeand the commentedinclude.
Acceptance
check-pinned-dependencies.shchecks every file incmake/, andctestpasses.- A build with
-DENABLE_FLAWFINDER=ON -DENABLE_TSCANCODE=ONstill configures and builds.
From the technical debt register (docs/quality/technical-debt.md, D7).
- Lingua principale
- C++
- Stelle
- 5
- Fork
- 4
- Merge medio
- 1h 2m
- PR unite (30g)
- 74
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
-
documentation severity/minor
Difficoltà 1/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
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
I maintainer di solito rispondono entro 1 giorno
-
build good first issue severity/minor
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno
-
test: add unit tests for ctrace::RunDirectoryForse già presa @tecnolgd l’ha presa 1 giorno 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
Tutte le issue di CoreTrace/coretrace
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno
-
Ctrl-C can abort a node: the state timer asks for the node names after shutdownForse già presa @fmrico l’ha presa oggi. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
fmrico/cascade_lifecycle#24 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
Deltakit/deltakit-stim#52 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno