Skip to content

Decouple configure/build/test tools from Docker#3661

Merged
tenpercent merged 0 commit intodevelopfrom
decouple-tools-from-docker
Jan 30, 2026
Merged

Decouple configure/build/test tools from Docker#3661
tenpercent merged 0 commit intodevelopfrom
decouple-tools-from-docker

Conversation

@tenpercent
Copy link
Copy Markdown
Contributor

@tenpercent tenpercent commented Jan 27, 2026

Summary

  • Create a two-layer tool architecture separating core build tools from Docker container management
  • Add ck-configure: New CMake configuration tool with preset support and native GPU detection
  • Refactor ck-build: Remove Docker dependency, add --configure and --list options
  • Refactor ck-test: Remove Docker dependency, add CTest integration with --smoke/--regression/--all
  • Enhance common.sh: Add native utilities for GPU detection and build directory management
  • Update ck-docker: Add configure/build/test/exec commands that delegate to core tools

Architecture

User CLI
    │
    ├─── ck-docker start/stop/shell ──► Container Management
    │
    ├─── ck-docker configure/build/test ──► docker exec ck-configure/build/test
    │
    └─── ck-configure / ck-build / ck-test ──► Native execution

Test plan

  • Test native execution on ROCm host
  • Test Docker delegation with ck-docker configure/build/test
  • Verify CTest integration with ck-test --smoke

🤖 Generated with Claude Code

@tenpercent tenpercent force-pushed the decouple-tools-from-docker branch 3 times, most recently from 261106d to 80db50e Compare January 28, 2026 23:38
Copy link
Copy Markdown
Contributor

@ThomasNing ThomasNing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tenpercent tenpercent force-pushed the decouple-tools-from-docker branch from 80db50e to e2e14c1 Compare January 29, 2026 19:02
@tenpercent tenpercent marked this pull request as ready for review January 29, 2026 19:02
@tenpercent tenpercent enabled auto-merge (squash) January 29, 2026 19:13
@tenpercent tenpercent disabled auto-merge January 30, 2026 01:27
@tenpercent tenpercent closed this Jan 30, 2026
@tenpercent tenpercent force-pushed the decouple-tools-from-docker branch from e2e14c1 to 83b6155 Compare January 30, 2026 01:41
@tenpercent tenpercent merged commit 83b6155 into develop Jan 30, 2026
6 of 10 checks passed
@tenpercent tenpercent deleted the decouple-tools-from-docker branch January 30, 2026 01:41
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.

2 participants