<p><a href="https://gerrit.osmocom.org/c/libosmocore/+/21457">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/21457/3/include/osmocom/core/mnl.h">File include/osmocom/core/mnl.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/21457/3/include/osmocom/core/mnl.h@16">Patch Set #3, Line 16:</a> <code style="font-family:monospace,monospace">       mnl_cb_t mnl_cb;</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">What about using something osmocom specific here so we don't depent on libmnl in the public API? It  […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The problem is that the user (i.e. the implementation of the callback) will need to parse the mnl specific data type arriving there anyway.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So this current patch here is really very netlink specific, and I don't see anything wrong with depending on libmnl.  It is the smallest netlink library out there, it exists for more than a decade, and as it is a dependency of netfilter/iptables tools, it has been packaged basically everywhere.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Supporting other netlink libraries (like libnl) doesn't seem to make sense  to me.  And supporting other operating systems is impossible as they don't have netlink to begin with.   And this entire osmo_mnl module is really just a very thin wrapper on top of netlink, i.e. nothing that you could ever do on a non-netlink-capable system at all.  It has no specific use case whatsoever.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/21457">change 21457</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/libosmocore/+/21457"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I4f787ee68f0d6d04f0a5655eb57d55b3b326a42f </div>
<div style="display:none"> Gerrit-Change-Number: 21457 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 02 Dec 2020 20:45:50 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>