public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
From: Ken Brown <kbrown@cornell.edu>
To: cygwin@cygwin.com
Subject: Re: [ANNOUNCEMENT] Updated: curl 7.71.1-1
Date: Mon, 10 Aug 2020 10:52:22 -0400	[thread overview]
Message-ID: <53c7f2a7-3bd8-f90c-b574-969d6b31f300@cornell.edu> (raw)
In-Reply-To: <4f1d6241-c86b-f2f9-7b49-a9f5b998b0eb@SystematicSw.ab.ca>

On 8/9/2020 5:48 PM, Brian Inglis wrote:
> On 2020-08-09 12:28, Adam Dinwoodie wrote:
>> On Fri, 24 Jul 2020 at 13:56, Adam Dinwoodie wrote:
>>> In the failing case, the `/configure` stage includes the following
>>> text: `configure: Setting CURL_LDFLAGS to '-lcurl -lnghttp2 -lidn2
>>> -lssh -lpsl -lssl -lcrypto -lldap -llber -lbrotlidec -lbrotlidec
>>> -lz'`; in the working case using -7.66.0-1, the corresponding line is
>>> just `configure: Setting CURL_LDFLAGS to '-lcurl'`.
[...]
> Meanwhile, that info likely comes from:
> 
> $ tail -vn15 /lib/pkgconfig/libcurl.pc
> ==> /lib/pkgconfig/libcurl.pc <==
> #
> prefix=/usr
> exec_prefix=/usr
> libdir=${exec_prefix}/lib
> includedir=${prefix}/include
> supported_protocols="DICT FILE FTP FTPS GOPHER HTTP HTTPS IMAP IMAPS LDAP LDAPS
> POP3 POP3S RTSP SCP SFTP SMB SMBS SMTP SMTPS TELNET TFTP"
> supported_features="SSL IPv6 UnixSockets libz brotli AsynchDNS IDN PSL NTLM
> NTLM_WB TLS-SRP HTTP2 HTTPS-proxy"
> 
> Name: libcurl
> URL: https://curl.haxx.se/
> Description: Library to transfer files with ftp, http, etc.
> Version: 7.71.1
> Libs: -L${libdir} -lcurl
> Libs.private: -lnghttp2 -lidn2 -lssh -lpsl -lssl -lcrypto -lldap -llber
> -lbrotlidec -lbrotlidec -lz
> Cflags: -I${includedir}
> 
> it looks like your git build should be using Libs for dynamic builds and adding
> Libs.private for static builds

If you look at configure.ac in the git sources, you'll see that it uses 
curl-config, not pkg-config.  The former reports:

$ curl-config --libs
-lcurl -lnghttp2 -lidn2 -lssh -lpsl -lssl -lcrypto -lldap -llber -lbrotlidec 
-lbrotlidec -lz

Yaakov's build of earlier versions of curl used a patch 7.33.0-libdeps.patch to 
prevent the unnecessary libs from being listed.  He even had a comment at the 
beginning of the patch, explaining the reason for it: "These are needed only for 
static linking, but we build with --disable-static".

Ken

  parent reply	other threads:[~2020-08-10 14:52 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-20  2:34 Brian Inglis
2020-07-20 17:58 ` ASSI
2020-08-10 18:16   ` Brian Inglis
2020-08-10 20:11     ` Marco Atzeri
     [not found] ` <CA+kUOa=-AJxr_2TaZoNerOquMGR8FCzi7hg3DyuvntuQqpAVOg@mail.gmail.com>
2020-08-09 18:28   ` Adam Dinwoodie
2020-08-09 21:48     ` Brian Inglis
2020-08-10  3:32       ` Brian Inglis
2020-08-10 14:52       ` Ken Brown [this message]
2020-08-10 17:33         ` Brian Inglis
2020-08-10 19:14           ` Ken Brown
2020-08-10 20:52             ` Adam Dinwoodie
2020-08-11  1:27             ` Brian Inglis
2020-08-11 11:13               ` Ken Brown
2020-08-11 11:27                 ` Adam Dinwoodie
2020-08-11 22:00                   ` Brian Inglis
2020-08-14 18:19                     ` Brian Inglis
2020-08-15 22:44                     ` [ANNOUNCEMENT] Updated: curl 7.71.1-2 Brian Inglis
2020-08-16 19:28                       ` Adam Dinwoodie
2020-08-11  5:15       ` [ANNOUNCEMENT] Updated: curl 7.71.1-1 ASSI
2020-08-11  5:55         ` Brian Inglis
2020-08-11 15:03           ` ASSI
2020-08-11 22:00             ` Brian Inglis

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53c7f2a7-3bd8-f90c-b574-969d6b31f300@cornell.edu \
    --to=kbrown@cornell.edu \
    --cc=cygwin@cygwin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).