build: Fix libbitcoinconsensus cross-compiling for Windows with DEBUG=1 #21344
build: Fix libbitcoinconsensus cross-compiling for Windows with DEBUG=1 #21344hebasto wants to merge 1 commit intobitcoin:masterfrom
Conversation
fanquake
left a comment
There was a problem hiding this comment.
Concept ACK. Please rebase this, and remove/trim the link error output from the PR description. There's no need to have all of that pasted in there.
|
Rebased 99a1523 -> f2b7ea4 (pr21344.01 -> pr21344.02) due to the conflict with #21339.
Done. |
|
Looks like this might also fix the issue of cross compiling libbitcoinconsensus for windows when building on macOS: # Using brew installed Mingw-w64 v8.0.0 (based on GCC 10.2).
CXXLD libbitcoinconsensus.la
/usr/local/Cellar/mingw-w64/8.0.0_3/toolchain-x86_64/bin/x86_64-w64-mingw32-ld: /usr/local/Cellar/mingw-w64/8.0.0_3/toolchain-x86_64/x86_64-w64-mingw32/lib/libssp.a(ssp.o): in function `__stack_chk_fail':
/private/tmp/mingw-w64-20210210-85400-sa28mi/mingw-w64-v8.0.0/gcc/build-x86_64/x86_64-w64-mingw32/libssp/../../../libssp/ssp.c:183: multiple definition of `__stack_chk_fail'; /usr/local/Cellar/mingw-w64/8.0.0_3/toolchain-x86_64/x86_64-w64-mingw32/lib/../lib/libssp.dll.a(d000007.o):(.text+0x0): first defined here
collect2: error: ld returned 1 exit status |
You are right. Tested on macOS Mojave 10.14.6 (18G8022) with Homebrew's |
luke-jr
left a comment
There was a problem hiding this comment.
Won't this produce the wrong output? ie, a static library instead of shared?
It's called LIBTOOL_APP_LDFLAGS for a reason ;)
You're right. The Closing. |
Fix #19772.
Steps to reproduce the bug on master (4f223e9):