Skip to content

snap: revert to ffmpeg 7.1.1#83

Merged
soumyaDghosh merged 2 commits intosnapcrafters:2404from
soumyaDghosh:2404
Jan 10, 2026
Merged

snap: revert to ffmpeg 7.1.1#83
soumyaDghosh merged 2 commits intosnapcrafters:2404from
soumyaDghosh:2404

Conversation

@soumyaDghosh
Copy link
Copy Markdown
Member

we need to rebuild the sdks, so this reversal is required

Signed-off-by: Soumyadeep Ghosh <soumyadeepghosh2004@zohomail.in>
lucyllewy
lucyllewy previously approved these changes Jan 10, 2026
Copy link
Copy Markdown
Member

@lucyllewy lucyllewy left a comment

Choose a reason for hiding this comment

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

The change itself is correct, but can you detail more about the requirement for downgrading again?

@soumyaDghosh
Copy link
Copy Markdown
Member Author

soumyaDghosh commented Jan 10, 2026

We never shipped ffmpeg 8 as it's ABI incompatible, and will break existing snaps if not rebuilt (didn't test though, just this is how we ship ffmpeg till now). It's just autoupdated by the CI, which I disabled some days ago. The SDKs have some old vulnerable packages, which requires rebuild (got an email from Canonical security team regarding it). Also, looks like I need to update the rust version, before merging this.

EDIT: Looks like I forgot to disable the updater CI

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

@lucyllewy lucyllewy left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the explanations 👍🏻

@soumyaDghosh soumyaDghosh merged commit ef0751e into snapcrafters:2404 Jan 10, 2026
2 checks passed
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