public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
From: Brian Inglis <Brian.Inglis@SystematicSw.ab.ca>
To: cygwin@cygwin.com
Subject: Re: [ANNOUNCEMENT] Updated: curl 7.71.1-1
Date: Mon, 10 Aug 2020 11:33:57 -0600	[thread overview]
Message-ID: <b66aea8b-2764-357c-8602-0a2e780bc90f@SystematicSw.ab.ca> (raw)
In-Reply-To: <53c7f2a7-3bd8-f90c-b574-969d6b31f300@cornell.edu>

On 2020-08-10 08:52, Ken Brown via Cygwin wrote:
> 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".

I try to avoid looking at autotools plumbing if I can possibly avoid it! ;^>
Someone cleaned up the approach used, as the patch did not apply and was dropped.
If you look at my later post, cygport git build and tests worked for me with no
problems other than at first missing some package build dependencies and in the
end I was still missing a couple of perl module dependencies I cannot decode.

I am waiting to hear back from the git maintainer when he has time.

-- 
Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada

This email may be disturbing to some readers as it contains
too much technical detail. Reader discretion is advised.
[Data in IEC units and prefixes, physical quantities in SI.]

  reply	other threads:[~2020-08-10 17:33 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
2020-08-10 17:33         ` Brian Inglis [this message]
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=b66aea8b-2764-357c-8602-0a2e780bc90f@SystematicSw.ab.ca \
    --to=brian.inglis@systematicsw.ab.ca \
    --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).