Skip to content

sdk: enable stripping for parts#85

Merged
brlin-tw merged 2 commits into2404from
work/strip-sdk
Jan 22, 2026
Merged

sdk: enable stripping for parts#85
brlin-tw merged 2 commits into2404from
work/strip-sdk

Conversation

@soumyaDghosh
Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Soumyadeep Ghosh <soumyadeepghosh2004@zohomail.in>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enables binary stripping across multiple parts in the ffmpeg-2404-sdk snapcraft configuration to reduce the final snap size. The changes also include simplifying the libdovi build process and adding --disable-static to zimg.

Changes:

  • Add stripping flags to multiple parts (zimg, vulkan-headers, vulkan-loaders, placebo, svt-av1) using appropriate build-system-specific methods
  • Remove the rustup part and simplify libdovi build by using the rust plugin directly
  • Add --disable-static to zimg and --library-type=cdylib to libdovi to build only dynamic libraries

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@soumyaDghosh soumyaDghosh requested a review from brlin-tw January 18, 2026 12:24
Signed-off-by: Soumyadeep Ghosh <soumyadeepghosh2004@zohomail.in>
@soumyaDghosh soumyaDghosh requested a review from brlin-tw January 22, 2026 16:06
@brlin-tw brlin-tw self-assigned this Jan 22, 2026
Copy link
Copy Markdown
Contributor

@brlin-tw brlin-tw left a comment

Choose a reason for hiding this comment

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

LGTM.

@brlin-tw brlin-tw merged commit 47f35dd into 2404 Jan 22, 2026
1 check passed
@soumyaDghosh
Copy link
Copy Markdown
Member Author

@brlin-tw I hope the build works now

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.

3 participants