Skip to content

use modern FFmpeg API - #1029

Closed
rathann wants to merge 1 commit into
OpenShot:developfrom
rathann:fix-build-with-ffmpeg8
Closed

use modern FFmpeg API#1029
rathann wants to merge 1 commit into
OpenShot:developfrom
rathann:fix-build-with-ffmpeg8

Conversation

@rathann

@rathann rathann commented Dec 19, 2025

Copy link
Copy Markdown

Fixes build with FFmpeg 8, where FF_PROFILE_* enums were replaced with AV_PROFILE_* and side_data API was replaced with codecpar.

References:
FFmpeg/FFmpeg@8238bc0 FFmpeg/FFmpeg@5432d2a

Supersedes #1018 .

Fixes #1028 .

Fixes build with FFmpeg 8, where `FF_PROFILE_*` enums were replaced with
`AV_PROFILE_*` and side_data API was replaced with codecpar.

References:
FFmpeg/FFmpeg@8238bc0
FFmpeg/FFmpeg@5432d2a

Supersedes OpenShot#1018 .

Fixes OpenShot#1028 .
@rathann

rathann commented Dec 19, 2025

Copy link
Copy Markdown
Author

Unfortunately, the following tests still fail for me:

Clip:time remapping
Clip:resample_audio_8000_to_48000_reverse
FrameMapper:resample_audio_mapper
FrameMapper:resample_audio_48000_to_41000
SphericalMetadata:SphericalMetadata_Test
SphericalMetadata:SphericalMetadata_FullOrientation
Timeline:Multi-threaded Timeline Add/Remove Effect

See #964 .

@github-actions

Copy link
Copy Markdown

Merge conflicts have been detected on this PR, please resolve.

@rathann

rathann commented Aug 20, 2026

Copy link
Copy Markdown
Author

This was apparently fixed differently in #1045 .

@rathann rathann closed this Aug 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conflicts A PR with unresolved merge conflicts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

regression in 0.5.0 unknow member nb_side_data and side_data in AVStream

1 participant