<p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/c/libosmo-netif/+/16760">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/libosmo-netif/+/16760/1/configure.ac">File configure.ac:</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/libosmo-netif/+/16760/1/configure.ac@100">Patch Set #1, Line 100:</a> <code style="font-family:monospace,monospace">AC_CHECK_HEADERS(netinet/sctp.h,,AC_MSG_ERROR(netinet/sctp.h not found))</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Support check for libsctp here is wrong actually. Same for DAHDI probably. We should have explicit build enable/disable flags for those instead, and the header should only be checked in the event --enable-libsctp is passed (the default).</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">See libosmocore (libsctp) and libosmo-abis(dahdi) as an example:<br>  --disable-libsctp       Do not enable socket multiaddr APIs requiring<br>                          libsctp</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><br>AC_ARG_ENABLE([libsctp], [AS_HELP_STRING([--disable-libsctp], [Do not enable socket multiaddr APIs requiring libsctp])],<br>        [ENABLE_LIBSCTP=$enableval], [ENABLE_LIBSCTP="yes"])<br>AM_CONDITIONAL(ENABLE_LIBSCTP, test x"$ENABLE_LIBSCTP" = x"yes")<br>AS_IF([test "x$ENABLE_LIBSCTP" = "xyes"], [<br> old_LIBS=$LIBS<br>        AC_SEARCH_LIBS([sctp_bindx], [sctp], [<br>                AC_DEFINE(HAVE_LIBSCTP, 1, [Define 1 to enable SCTP support])<br>         AC_SUBST(HAVE_LIBSCTP, [1])<br>           if test -n "$ac_lib"; then<br>                  AC_SUBST(LIBSCTP_LIBS, [-l$ac_lib])<br>           fi<br>            ], [<br>          AC_MSG_ERROR([sctp_bindx not found in searched libs])])<br>       LIBS=$old_LIBS<br>])</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><br>AC_ARG_ENABLE([dahdi],<br>        AC_HELP_STRING([--disable-dahdi],<br>                            [disable support for DAHID ISDN (E1/T1) cards [default=yes]]),<br>         [enable_dahdi="$enableval"], [enable_dahdi="yes"])<br>AM_CONDITIONAL(ENABLE_DAHDI, test "x$enable_dahdi" = "xyes")<br>if test "x$enable_dahdi" =  "xyes"; then<br>  AC_CHECK_HEADERS([dahdi/user.h],[],[AC_MSG_ERROR([DAHDI input driver enabled but DAHDI not found])])<br>else<br>    AC_MSG_WARN([DAHDI input driver will not be built])<br>fi</pre></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmo-netif/+/16760">change 16760</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/libosmo-netif/+/16760"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmo-netif </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I4b3e1f1894f13ac1175a71a5139c02a2633be26d </div>
<div style="display:none"> Gerrit-Change-Number: 16760 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </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: Tue, 07 Jan 2020 17:52:08 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>