-
Notifications
You must be signed in to change notification settings - Fork 267
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
[mingw64] [FFmpeg+librav1e] multiple definition of ___chkstk_ms #1483
Comments
It's just complaining that there are two functions with the same name |
Thank you |
Might need to report this to rav1e upstream, I think they might be
including libgcc in their static lib.
…On Sat, 23 Nov 2019 at 22:30, Emil ***@***.***> wrote:
Thank you
I disabled --enable-librav1e in ffmpeg_options.txt and the build
completed without error.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#1483?email_source=notifications&email_token=AAA3H5J5IOQZWZNJYXI37G3QVGVJRA5CNFSM4JQ337J2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE77DKA#issuecomment-557838760>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAA3H5KLNYAPFRSLYVCND7LQVGVJRANCNFSM4JQ337JQ>
.
|
I've had the exact same iusse as @subz390 and I also disabled |
For your last problem #1485 |
Looks like it's coming from rust. https://github.com/rust-lang/compiler-builtins/blob/master/src/x86_64.rs |
Now the question is to where do we report it, rav1e, cargo-c, or rust-lang |
Probably rust-lang, chkstk stuff are inserted by compilers if a function needs more than a page worth of stack, rav1e and cargo-c won't have control on that. |
While we wait for Rust to fix the issue, here's a hack to get it working: diff --git a/build/media-suite_compile.sh b/build/media-suite_compile.sh
index f840a23..ad78623 100644
--- a/build/media-suite_compile.sh
+++ b/build/media-suite_compile.sh
@@ -1057,6 +1057,13 @@ if { [[ $rav1e = y ]] || enabled librav1e; } &&
cinstall --release --prefix "$PWD/install-$bits" --jobs "$cpuCount"
# do_install "install-$bits/bin/rav1e.dll" bin-video/
# do_install "install-$bits/lib/librav1e.dll.a" lib/
+
+ compiler_builtins=$(ar t "install-$bits/lib/librav1e.a" | grep -xG compiler_builtins-.*compiler_builtins.*-cgu.0.rcgu.o)
+ ar x "install-$bits/lib/librav1e.a" "$compiler_builtins"
+ objcopy -W ___chkstk_ms "$compiler_builtins"
+ ar r "install-$bits/lib/librav1e.a" "$compiler_builtins"
+ rm "$compiler_builtins"
+
do_install "install-$bits/lib/librav1e.a" lib/
do_install "install-$bits/lib/pkgconfig/rav1e.pc" lib/pkgconfig/
do_install "install-$bits/include/rav1e"/*.h include/rav1e/ |
@linnaea, do you think can submit a PR so I can easily test it? |
logs.zip
My apologies, I'm new to this suite and don't understand the error.
I've run
media-autobuild_suite.bat
four times to make sure everything is up to date and it exits with the same error at the same place each time.In the meantime, is there anything can I disable in
ffmpeg_options.txt
please to avoid getting this error as I'm only building this ffmpeg so I can transcode video using hardware accelleration NVIDIA GPU with --enable-libnpp as that's not included on public realeases of ffmpeg. Anything else can go.The text was updated successfully, but these errors were encountered: