Skip to content

Add CLAUDE.md: local builds and RTSP checks in pinned Docker images - #40

Merged
compot-dev merged 2 commits into
mainfrom
docs/claude-md
Oct 17, 2026
Merged

compot-dev merged 2 commits into
mainfrom
docs/claude-md

Conversation

@compot-dev

Copy link
Copy Markdown
Collaborator

Adds CLAUDE.md with the commands agent sessions should use locally. Every image runs pinned, with --pull never:

Documentation only.

Tested on main dcab94e: native make test passes 63 checks with 0 failures, and QEMU=1 ci/build-musl.sh arm32 passes 62 with 0 failures. The 6.1-alpine image includes ffprobe and the rtsp demuxer.

Native tests, static musl builds and the Android APKs run in pinned
images with --pull never. RTSP checks use jrottenberg/ffmpeg:6.1-alpine,
because the untagged image pulls latest (about 300 MB) for nothing.
@qodo-free-for-open-source-projects

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can pick a display mode instead of tweaking every Display option by hand

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

PR Summary by Qodo

Document pinned Docker commands for local builds and RTSP checks

📝 Documentation 🕐 Less than 10 minutes

Grey Divider

AI Description

• Add local Docker commands for native tests, musl builds, and Android APKs.
• Document pinned FFmpeg commands for RTSP checks, avoiding an untagged image pull.
• Clarify that release builds belong in CI and benchmark containers may be shared.
Diagram

graph TD
  Docs["CLAUDE.md"] --> Native["Native tests"] --> Cross["Cross image"]
  Docs --> Musl["Musl builds"] --> Cross
  Docs --> Android["Android wrapper"] --> AndroidImage["Android image"]
  Docs --> FFmpeg["FFmpeg image"] --> RTSP["RTSP stream"]
Loading
High-Level Assessment

A repository guide is appropriate for agent-facing local commands without changing build behavior. Before relying on the examples, reconcile the musl cache mount with build-musl.sh’s default cache path and clarify that android/build.sh does not pass --pull never.

Files changed (1) +47 / -0

Documentation (1) +47 / -0
CLAUDE.mdDocument local Docker builds and RTSP checks +47/-0

Document local Docker builds and RTSP checks

• Adds commands for native tests, QEMU-backed musl builds, Android APK builds, and RTSP probing with tagged Docker images. It also distinguishes local development from CI releases and warns that benchmark containers may be shared.

CLAUDE.md

@compot-dev

compot-dev commented Oct 16, 2026 •

Copy link
Copy Markdown
Collaborator Author

Addressed the two points from the summary in bf93222. The toolchain cache reaches ci/build-musl.sh through TOOLCHAIN_CACHE=/opt/tc, which the image sets. Verified: QEMU=1 ci/build-musl.sh arm32 with that mount passes 62 checks with 0 failures. A read-only mount cannot take a new toolchain, so the guide says to mount it read-write for a first download. android/build.sh calls docker run without --pull never, and the guide now says that too.

/review

@compot-dev
compot-dev merged commit e8dc7cf into main Oct 17, 2026
9 checks passed
@compot-dev
compot-dev deleted the docs/claude-md branch October 17, 2026 00:02
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