Attention is currently required from: osmith.
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-upf/+/29188 )

Change subject: packaging: add missing libnftnl, libnftables deps
......................................................................


Patch Set 1:

(1 comment)

Patchset:

PS1:
I am wondering why we are depending on libnftnl at all.  Are we using it 
directly from osmo-upf codebase?   To me it seesm that all we use is 
libnftables, and that in turn has an upstream dependency to libnftnl, which 
again has a dependency to libmnl.  But those downstream dependencies should not 
be of our concern, it is the job of distribution packaging and pkg-config to 
pull those in.



--
To view, visit https://gerrit.osmocom.org/c/osmo-upf/+/29188
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-upf
Gerrit-Branch: master
Gerrit-Change-Id: I974dd3163e50e6b1a099eac5d5f6a1cac07f44ad
Gerrit-Change-Number: 29188
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-CC: laforge <[email protected]>
Gerrit-Attention: osmith <[email protected]>
Gerrit-Comment-Date: Mon, 22 Aug 2022 13:19:49 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to