Page MenuHomeFreeBSD
Differential D58275

devel/sem: Declare libzstd shlib dependency
AcceptedPublic
Actions

Authored by olgeni on Thu, Jul 16, 8:09 AM.
Tags
None
Referenced Files
F163029690: D58275.diff
Sun, Jul 19, 9:27 AM
F162997364: D58275.diff
Sun, Jul 19, 1:04 AM
Unknown Object (File)
Fri, Jul 17, 6:53 AM
Unknown Object (File)
Thu, Jul 16, 4:35 PM
Unknown Object (File)
Thu, Jul 16, 9:27 AM
Subscribers
None

Details

Reviewers
adamw
Summary

The binary links libzstd.so.1 (via the zstd-sys crate, which uses
pkg-config to pick up the system libzstd by default), but the port
does not declare the runtime dependency.

Detected by poudriere stage-qa:

Warning: sem-0.21.0 will be rebuilt as it misses libzstd.so.1
which no dependency provides. It is likely (silently) failing
testport/stage-qa.

Add LIB_DEPENDS=libzstd.so:archivers/zstd and bump PORTREVISION.

Test Plan

poudriere testport on 15.1-amd64 - clean build, stage-qa no longer
reports the missing libzstd.so.1 shlib, package installs and
deinstalls cleanly.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 74845
Build 71728: arc lint + arc unit

Event Timeline

olgeni created this revision.Thu, Jul 16, 8:09 AM
olgeni held this revision as a draft.
Harbormaster completed remote builds in B74845: Diff 182068.Thu, Jul 16, 8:09 AM
olgeni published this revision for review.Thu, Jul 16, 8:17 AM
adamw accepted this revision.Sat, Jul 18, 4:23 AM

Great catch! Thank you, Jimmy!

This revision is now accepted and ready to land.Sat, Jul 18, 4:23 AM

Revision Contents
Changeset List

PathSize
devel/
sem/
4 lines

Diff 182068

View Options

devel/sem/Makefile

Loading...