autotools: accept linker flags via CURL_LDFLAGS_{LIB,BIN}#12312
Closed
vszakats wants to merge 2 commits into
Closed
autotools: accept linker flags via CURL_LDFLAGS_{LIB,BIN}#12312vszakats wants to merge 2 commits into
CURL_LDFLAGS_{LIB,BIN}#12312vszakats wants to merge 2 commits into
Conversation
autotools passes `LDFLAGS` automatically to the linker command. curl's `lib/Makefile.am` customizes libcurl linker flags. In that customization, it added `LDFLAGS` to the custom options. This resulted in passing `LDFLAGS` _twice_ to the `libtool` command. Most of the time this is benign, but some `LDFLAGS` options can break the build when passed twice. One such example is passing `.o` files, e.g. `crt*.o` files necessary when customizing the C runtime, e.g. for MUSL builds. Passing them twice resulted in duplicate symbol errors: ``` libtool: link: clang-15 --target=aarch64-unknown-linux-musl [...] /usr/lib/aarch64-linux-musl/crt1.o [...] /usr/lib/aarch64-linux-musl/crt1.o [...] ld.lld-15: error: duplicate symbol: _start >>> defined at crt1.c >>> /usr/lib/aarch64-linux-musl/crt1.o:(.text+0x0) >>> defined at crt1.c >>> /usr/lib/aarch64-linux-musl/crt1.o:(.text+0x0) [...] clang: error: linker command failed with exit code 1 (use -v to see invocation) ``` This patch deletes `LDFLAGS` from our customized options, leaving a single copy of them as passed by autotools automatically. This behaviour came with commit 1a59319 (2013-07-23) as a fix for bug https://curl.haxx.se/bug/view.cgi?id=1217. The patch was a works-for-me hack that ended up merged in curl: https://sourceforge.net/p/curl/bugs/1217/#06ef With the root cause remaining unclear. Perhaps the SUNPro 12 linker was sensitive to `-L` `-l` order, requiring `-L` first? This would be unusual and might suggest a bug in either the linker or in `libtool`. The curl build does indeed pass the list of detected libs via its own `LIBCURL_LIBS` variable, which ends up before `LDFLAGS` on the `libtool` command line, but it would seem to be the job of `libtool` to ensure that even a buggy/peculiar linker gets the options in the order it's expecting it. Especially because autotools passes `LDFLAGS` last, making it hardly possibly to pass anything after it. Perhaps in the 10 years since this issue, this already got a fix upstream. IMO passing `LDFLAGS` twice to fix the above is a nuclear option with unintended side-effects. Closes #xxxxx
…BIN` To allow passing `LDFLAGS` specific to libcurl (`CURL_LDFLAGS_LIB`) and curl tool (`CURL_LDFLAGS_BIN`). This makes it possible to build libcurl and curl with a single invocation with lib- and tool-specific custom linker flags. Such flag can be enabling `.map` files, a `.def` file for libcurl DLL, controlling static/shared, incl. requesting a static curl tool (with `-static-libtool-libs`) while building both shared and static libcurl. curl-for-win uses the above and some more. Closes #xxxxx
vszakats
changed the title
CURL_LDFLAGS_LIB, CURL_LDFLAGS_BINCURL_LDFLAGS_LIB, CURL_LDFLAGS_BIN
vszakats
changed the title
CURL_LDFLAGS_LIB, CURL_LDFLAGS_BINCURL_LDFLAGS_{LIB,BIN}
vszakats
added a commit
to curl/curl-for-win
that referenced
this pull request
Nov 13, 2023
Disabled for now. Pending upstream curl/curl#12312
vszakats
added a commit
to curl/curl-for-win
that referenced
this pull request
Nov 14, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
To allow passing
LDFLAGSspecific to libcurl (CURL_LDFLAGS_LIB) andcurl tool (
CURL_LDFLAGS_BIN).This makes it possible to build libcurl and curl with a single
invocation with lib- and tool-specific custom linker flags.
Such flag can be enabling
.mapfiles, a.deffile for libcurl DLL,controlling static/shared, incl. requesting a static curl tool (with
-static-libtool-libs) while building both shared and static libcurl.curl-for-win uses the above and some more.
These options are already supported in
Makefile.mk. CMake has built-invariables for this.
Closes #12312