-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bundling ffmpeg #3
Comments
I'll look into it when i hvave the time |
Cool. Supposedly there's a version of ffmpeg bundled with Electron itself, though the documentation on that is sparse. I hope to dig into this soon and produce some documentation around using it. If you get to it first, please let me know! |
The included ffmpeg version might not be recent enough and have all the features needed for muxing/demuxing/cutting etc. Also it's only the lib, not the ffmpeg CLI. So node native bindings are required, which complicates things. https://github.com/node-ffi/node-ffi http://qiita.com/kjunichi/items/8c1955c6bc2013b636a3 libavformatもnodeも64ビットな環境で動かしてみたら、 |
ffmpeg is now bundled in as of 1.2.0. |
Nice! Any thoughts on the process you went through? Did you follow the same technique the Kap team used? I notice a bunch of different source hosts: ffmpeg_linux_ia32=https://johnvansickle.com/ffmpeg/releases/ffmpeg-release-32bit-static.tar.xz
ffmpeg_linux_x64=https://johnvansickle.com/ffmpeg/releases/ffmpeg-release-64bit-static.tar.xz
ffmpeg_darwin_x64=http://evermeet.cx/ffmpeg/ffmpeg-3.2.7z
ffmpeg_win32_ia32=https://ffmpeg.zeranoe.com/builds/win32/static/ffmpeg-3.1.5-win32-static.zip
ffmpeg_win32_x64=https://ffmpeg.zeranoe.com/builds/win64/static/ffmpeg-3.1.5-win64-static.zip
ffprobe_darwin_x64=http://evermeet.cx/ffmpeg/ffprobe-3.2.7z How did you compile this list? |
I used pretty much the same technique, yes. |
I have no idea what I'm doing™, but I see that the team that built Kap was able to vendor ffmpeg in their app:
https://github.com/wulkano/kap/search?utf8=%E2%9C%93&q=ffmpeg
It would be pretty nice for users not to have to set this up.
The text was updated successfully, but these errors were encountered: