Opened 4 years ago

Closed 4 years ago

Last modified 3 months ago

#61654 closed defect (fixed)

curl +universal: curl-config differs due to opportunistic use of zstd

Reported by: lyssdod (Alex R) Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: curl

Description

port -d install curl +ssl+universal fails due to curl-config being different. logs are attached

Attachments (1)

install_log (87.9 KB) - added by lyssdod (Alex R) 4 years ago.
sudo port -d install curl +ssl+universal

Download all attachments as: .zip

Change History (10)

Changed 4 years ago by lyssdod (Alex R)

Attachment: install_log added

sudo port -d install curl +ssl+universal

comment:1 Changed 4 years ago by kencu (Ken)

This should work. Please try this first:

sudo port clean curl
sudo port -v install curl +universal

and post back the outcome

Version 0, edited 4 years ago by kencu (Ken) (next)

comment:2 Changed 4 years ago by jmroot (Joshua Root)

Owner: set to ryandesign
Port: curl added
Status: newassigned

comment:3 Changed 4 years ago by ryandesign (Ryan Carsten Schmidt)

Agreed, this log was from an unclean build attempt (log contains lines saying Skipping completed) so cleaning and trying again will either install successfully or will generate a log showing what the real problem is; if so, please attach it.

comment:4 Changed 4 years ago by lyssdod (Alex R)

This can be closed now. Upon cleaner inspection of the log, it turned out that i386 curl variant was linked to zstd, and x86_64 wasn't, thus causing curl-config script mismatch (no idea if they were to merge into universal/fat binary later). Reinstalling zstd and bunch of other stuff as +universal has solved the issue. Thank you Ryan!

comment:5 Changed 4 years ago by kencu (Ken)

Resolution: invalid
Status: assignedclosed

you're welcome.

comment:6 Changed 4 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: invalid
Status: closedreopened
Summary: curl-config differs while trying to install +universalcurl +universal: curl-config differs due to opportunistic use of zstd

Thanks for the additional info. So the bug is that curl links with zstd but does not declare a dependency on it. If it had declared that dependency, MacPorts would have reinstalled zstd universal for you and you would not have experienced this problem.

comment:7 Changed 4 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: reopenedclosed

In b7cb12be475afb73a6f5b9034702b14cc4c2cdd9/macports-ports (master):

curl: Add zstd dependency

Closes: #61654

comment:8 Changed 4 years ago by kencu (Ken)

Last edited 4 years ago by kencu (Ken) (previous) (diff)

comment:9 Changed 3 months ago by ryandesign (Ryan Carsten Schmidt)

In ab601bb9f7d20d00d9f00a309b3c55c5e21763c7/macports-ports (master):

curl: Move brotli/http2/idn/psl/zstd to variants

While some users want commonly-used features enabled in curl by default,
others wish to have the option to disable them to build a minimal curl
for MacPorts to link with on older systems. Therefore move brotli,
http2, idn, psl, and zstd features to variants which are on by default.

See: #55197
See: #61654
See: #65056
See: #70481

Note: See TracTickets for help on using tickets.