Skip to content

Conversation

@Dan-Flores
Copy link
Contributor

In addition, the installation commands now find FFmpeg<9 as suggested in #861.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Oct 28, 2025
CONTRIBUTING.md Outdated

```bash
conda install cmake pkg-config pybind11 "ffmpeg<8" -c conda-forge
conda install cmake pkg-config pybind11 "ffmpeg<9" -c conda-forge
Copy link
Contributor

Choose a reason for hiding this comment

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

Here and below, let's remove the bound, FFmpeg 9 isn't out and probably won't be anytime soon, and using ffmpeg<9 may suggest to some users that we don't support the latest FFmpeg - which we do!

The intent behind ffmpeg<8 was that for a short period of time, FFmpeg 8 was released and we didn't support it, so users would potentially get a conflict. But there's no risk of conflict ATM since FFmpeg 9 doesn't exist.

Suggested change
conda install cmake pkg-config pybind11 "ffmpeg<9" -c conda-forge
conda install cmake pkg-config pybind11 "ffmpeg" -c conda-forge

Copy link
Contributor

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

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

Thanks @Dan-Flores , LGTM with the comment above to remove the bound

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants