Skip to content

chore(docker): remove royalty-obligating codec libs from release image - #2922

Closed
kajalj22 wants to merge 2 commits into
mainfrom
strip-royalty-codecs
Closed

chore(docker): remove royalty-obligating codec libs from release image#2922
kajalj22 wants to merge 2 commits into
mainfrom
strip-royalty-codecs

Conversation

@kajalj22

@kajalj22 kajalj22 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Strips libx264, libx265, libopenh264, and libmp3lame from all venvs (/opt/nemo_rl_venv and /opt/ray_venvs) at image build time
  • H.264 and H.265 carry MPEG LA / HEVC Advance royalty obligations on distribution; libopenh264 bundled directly loses Cisco's CDN royalty-free exemption
  • These libs are not used by NeMo-RL at runtime — removal does not affect functionality

🤖 Generated with Claude Code

H.264 (libx264, libopenh264) and H.265 (libx265) carry MPEG LA / HEVC
Advance royalty obligations on distribution. Strip these and libmp3lame
from all venvs (main + Ray worker) at image build time so the distributed
sqsh does not include them.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Kajal Jain <kajalj@nvidia.com>
@kajalj22
kajalj22 requested a review from a team as a code owner June 24, 2026 23:12
@copy-pr-bot

copy-pr-bot Bot commented Jun 24, 2026

Copy link
Copy Markdown

Auto-sync is disabled for ready for review pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@kajalj22

kajalj22 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test d6ee46c

@kajalj22 kajalj22 added r0.7.0 CI:L1 Run doctests, unit tests, and functional tests labels Jul 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI:L1 Run doctests, unit tests, and functional tests r0.7.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants