[Bug] Make CTest self-contained from a clean checkout
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- Half a day
- Newbie friendliness
- 72/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- c, cmake
- Domain
- build-system, testing
Research direction
Start in CMakeLists.txt around lines 61-73, where the elf_loader, disasm and decompiler tests point at the untracked ${CMAKE_CURRENT_SOURCE_DIR}/hello binary, and at the fixture target near lines 83-89, which does not build hello.c. Check .gitignore:15 to see why hello is ignored. Done when a fresh archive of HEAD configures, builds and passes ctest --test-dir build without any pre-existing hello binary, with the fixture shared by all three tests.
Written by the indexing model from the issue text.
Description
Summary
ctest fails from a clean checkout because three tests require an ignored hello executable that no CMake target creates.
Evidence
CMakeLists.txt:61-73 registers elf_loader, disasm, and decompiler with:
"${CMAKE_CURRENT_SOURCE_DIR}/hello"
However, hello is ignored by .gitignore:15, only hello.c is tracked, and the build fixture target at CMakeLists.txt:83-89 does not compile hello.c.
I reproduced this by archiving HEAD into a fresh directory and running the documented workflow:
50% tests passed, 3 tests failed out of 6
elf_loader: could not open file: .../hello
disasm: could not open file: .../hello
decompiler: could not open file: .../hello
The roundtrip test passes because it compiles its own temporary fixture, which makes the inconsistency easy to miss when an old ignored hello binary happens to exist locally.
Impact
The README promises that make -C build test works after a fresh configure/build, but a new clone or CI workspace fails three tests before exercising the loader, disassembler, and project decompiler tests.
Expected behavior
The test suite should provision its ELF fixture as part of the build/test graph, or compile a temporary fixture from hello.c for every test that needs a binary. A clean checkout should pass the documented test command without relying on ignored files left by a previous local build.
Acceptance criteria
- Remove the dependency on an untracked
helloexecutable. - Verify
cmake -S . -B build && cmake --build build && ctest --test-dir buildin a clean checkout. - Keep the fixture setup deterministic and shared by the loader, disassembly, and decompiler tests.
- Dominant language
- C++
- Stars
- 0
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from TiElixir/caraxes
-
bug good first issue
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
enhancement
Difficulty 5/5 Over a week Newbie friendliness 12/100
-
enhancement
Difficulty 4/5 3-5 days Newbie friendliness 35/100
-
enhancement
Difficulty 5/5 Over a week Newbie friendliness 20/100
-
bug good first issue
Difficulty 3/5 Half a day Newbie friendliness 68/100
All issues in TiElixir/caraxes
Similar issues
-
std::fdim drops partial derivative w.r.t second argument in forward and reverse modePossibly taken @ADITYASINGH1206 claimed this today. Open
Difficulty 1/5 Under an hour Newbie friendliness 85/100
vgvassilev/clad#2534 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 61/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 3 days
-
HAP.cpp fixOpen
Difficulty 2/5 1-3 hours Newbie friendliness 68/100