Skip to content

CLI: fixed adding cli and completion into docker containers, improved docs#18003

Merged
ngxson merged 1 commit intoggml-org:masterfrom
andrew-aladev:feature/fix-devops-cli-and-completion-and-related-docs
Dec 16, 2025
Merged

CLI: fixed adding cli and completion into docker containers, improved docs#18003
ngxson merged 1 commit intoggml-org:masterfrom
andrew-aladev:feature/fix-devops-cli-and-completion-and-related-docs

Conversation

@andrew-aladev
Copy link
Contributor

@andrew-aladev andrew-aladev commented Dec 13, 2025

Hello @CISC I've read your PR #17964 and caught the idea: you've added both cli and completion into :light image. This is a good decision; we just need to improve this idea a bit:

  1. Add completion to llama-cli-cann.Dockerfile, llama-cpp-cuda.srpm.spec, and llama-cpp.srpm.spec.
  2. Replace main executables with clear executable lists in docs/docker.md.
  3. Add examples of using cli and completion in docs/docker.md.

Related PRs: #17964, #17824.

Please review; thank you.

@CISC
Copy link
Member

CISC commented Dec 13, 2025

Honestly, since I don't know if nix/rpm stuff is still usable (I don't think we have any maintainers who know anything about those any more) I didn't want to touch them.

@andrew-aladev
Copy link
Contributor Author

Ok, then I am going to revert the change to the nix directory. Thank you.

@andrew-aladev andrew-aladev force-pushed the feature/fix-devops-cli-and-completion-and-related-docs branch from e67fdd7 to e52c4a4 Compare December 13, 2025 18:45
@andrew-aladev andrew-aladev force-pushed the feature/fix-devops-cli-and-completion-and-related-docs branch from e52c4a4 to 6fc076e Compare December 13, 2025 19:15
@andrew-aladev andrew-aladev force-pushed the feature/fix-devops-cli-and-completion-and-related-docs branch from 6fc076e to 99ead70 Compare December 13, 2025 20:56
Copy link
Member

@CISC CISC left a comment

Choose a reason for hiding this comment

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

LGTM, final approval left to @ngxson

@github-actions github-actions bot added documentation Improvements or additions to documentation devops improvements to build systems and github actions labels Dec 13, 2025
@ngxson ngxson merged commit fb64424 into ggml-org:master Dec 16, 2025
2 checks passed
Anico2 added a commit to Anico2/llama.cpp that referenced this pull request Jan 15, 2026
blime4 referenced this pull request in blime4/llama.cpp Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops improvements to build systems and github actions documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants