Skip to content

singularity: add man-db to container image#55

Open
wwood wants to merge 1 commit into
mainfrom
claude/mqyolo-container-man-qlknck
Open

singularity: add man-db to container image#55
wwood wants to merge 1 commit into
mainfrom
claude/mqyolo-container-man-qlknck

Conversation

@wwood

@wwood wwood commented Jun 26, 2026

Copy link
Copy Markdown
Member

Installs the man command so manual pages are available inside the
mqyolo sandbox.

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01VpBbpsyhdPcYS9oxkWfZHA

Installs the `man` command so manual pages are available inside the
mqyolo sandbox.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VpBbpsyhdPcYS9oxkWfZHA

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d8ff6e3fdb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread singularity/ai_tool.def
# man-db: provides the `man` command for reading manual pages.
apt-get update
apt-get install -y --no-install-recommends time less rsync curl ca-certificates gawk jq
apt-get install -y --no-install-recommends time less rsync curl ca-certificates gawk jq man-db

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Install man pages, not just man-db

When this definition is built from the Debian-based node:24 image, the base image's dpkg Docker config still excludes /usr/share/man/*, and this apt command also keeps --no-install-recommends; as a result this adds /usr/bin/man but continues discarding the manual-page files for man-db, jq, curl, etc. In the rebuilt mqyolo image, common checks like man man or man jq still have no manual entry, so the stated goal of making manuals available is not met unless the dpkg exclude is removed before installing/reinstalling the relevant manpage packages.

Useful? React with 👍 / 👎.

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