Skip to content

Conversation

Polochon-street
Copy link

Changed two things:
I saw rust-ffmpeg4-sys version was bumped, but it's still not working using 4.2.2 so just switched the branch to the stable version (it's failing anyways 😄 )
More important change: I've removed the alloc() in the context resampling, since swr_convert_frame allocates stuff itself here https://ffmpeg.org/doxygen/trunk/swresample__frame_8c_source.html more precisely, calculating itself the number of frame needed. Also fixed the flush()

Hope it makes sense 😄

@bacek
Copy link
Owner

bacek commented Mar 16, 2020

It should work with master branch of ffmpeg4-sys. I just can't release -sys crate because it's version is tied to ffmpeg, which is unfortunate.

@Polochon-street
Copy link
Author

Err, just cloning it gives

polochon@arch-coucou rust-ffmpeg4 $ cargo build --verbose --features "build"                       [0]
    Updating crates.io index
    Updating git repository `https://github.com/bacek/rust-ffmpeg4-sys`
error: no matching package named `ffmpeg4-sys` found
location searched: https://github.com/bacek/rust-ffmpeg4-sys
prerelease package needs to be specified explicitly
ffmpeg4-sys = { version = "4.3.0-pre.0" }
required by package `ffmpeg4 v0.4.0 (/home/polochon/rust-ffmpeg4)`

I guess master's travis build is passing because it has been built prior to the rust-ffmpeg4-sys bump?

@bacek
Copy link
Owner

bacek commented Mar 17, 2020 via email

@Polochon-street Polochon-street force-pushed the fix-flush-resampling-context branch from 11fe490 to cb21a6c Compare March 17, 2020 22:47
@Polochon-street Polochon-street force-pushed the fix-flush-resampling-context branch from cb21a6c to 67a0982 Compare July 29, 2020 20:24
@Polochon-street
Copy link
Author

@bacek I've updated stuff with your latest changes on ffmpeg4-sys, how does it look?
It passes on my machine, but hard to tell without the travis build 😄

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