Skip to content

build: remove SAM3_BUILD_TESTS option - #29

Closed
PABannier wants to merge 1 commit into
cleanup/remove-test-apifrom
cleanup/remove-build-tests-option
Closed

PABannier wants to merge 1 commit into
cleanup/remove-test-apifrom
cleanup/remove-build-tests-option

Conversation

@PABannier

Copy link
Copy Markdown
Owner

After #20 deletes tests/, cmake .. -DSAM3_BUILD_TESTS=ON fails at configure time: add_subdirectory(tests) points at a missing directory. This PR removes the dangling pieces:

  • the SAM3_BUILD_TESTS option block in CMakeLists.txt
  • "To build tests" in README.md and the Tests: line in CLAUDE.md
  • the tests/output/, tests/cpp_*, tests/debug_* and tests/ref*/ patterns in .gitignore

Stacked on #20. Merge #20 first; GitHub will retarget this PR to main.

git diff --shortstat: 4 files changed, 19 deletions(-). git grep SAM3_BUILD_TESTS is now empty.

Part of an over-engineering cleanup series.

🤖 Generated with Claude Code

https://claude.ai/code/session_01M2qLmQ9ag49hT61V6i9qAq

tests/ is gone, so -DSAM3_BUILD_TESTS=ON fails at configure time with a
missing add_subdirectory. Drop the option, the README/CLAUDE.md build
instructions for it, and the tests/ ignore patterns.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01M2qLmQ9ag49hT61V6i9qAq
@PABannier PABannier closed this Sep 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant