git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v2 1/1] Makefile: link libcurl before openssl and crypto

From
Junio C Hamano <gitster@pobox.com>
Date
Sep 25, 2015, 00:15 UTC
Message-ID
<xmqqmvwbuzep.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<1443116590-6222-1-git-send-email-repk@triplefau.lt>
Remi Pommarel <repk@triplefau.lt> writes:
> For static linking especially library order while linking is important. For
> example libssl contains symbol from libcrypto so the former should be linked
> before the latter.
That example is not a very good one to use for two reasons:
 (1) NEEDS_SSL_WITH_CRYPTO is a conter-example of crypto wanting to
     be before the latter.
 (2) More importantly, you are not changing the order between them.

The breakage you are fixing with this patch is that libcurl want to come before libssl when building http-push, http-fetch, and remote-curl, so that is a very good example to use in the above sentence.

Previous: Remi Pommarel
Message 2 of 2 in “Makefile: link libcurl before openssl and crypto”
  1. 1/1 Makefile: link libcurl before openssl and cryptoRemi Pommarel, Sep 24, 2015
  2. Junio C HamanoSep 25, 2015

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.