<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On 8 May 2017 at 13:46, Maxin B. John <span dir="ltr"><<a href="mailto:maxin.john@intel.com" target="_blank">maxin.john@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">1.33 -> 1.34<br>
<br>
1. Refreshed 0001-Fix-compile-on-musl-with-<wbr>kernel-4.9-headers.patch<br>
2. Removed upstreamed patch:<br>
0003-stats-Fix-bad-file-<wbr>descriptor-initialisation.<wbr>patch<br>
3. Provided a PACKAGECONFIG for nftables support<br>
<br>
Signed-off-by: Maxin B. John <<a href="mailto:maxin.john@intel.com">maxin.john@intel.com</a>><br>
---<br>
meta/recipes-connectivity/<wbr>connman/connman.inc | 1 +<br>
...x-compile-on-musl-with-<wbr>kernel-4.9-headers.patch | 37 ++++++--<br>
...ll-nftables-fix-build-with-<wbr>libnftnl-1.0.7.patch | 72 +++++++++++++++<br>
...ts-Fix-bad-file-descriptor-<wbr>initialisation.patch | 102 ---------------------<br>
.../connman/{<a href="http://connman_1.33.bb" rel="noreferrer" target="_blank">connman_1.33.bb</a> => <a href="http://connman_1.34.bb" rel="noreferrer" target="_blank">connman_1.34.bb</a>} | 6 +-<br>
5 files changed, 103 insertions(+), 115 deletions(-)<br>
create mode 100644 meta/recipes-connectivity/<wbr>connman/connman/0001-firewall-<wbr>nftables-fix-build-with-<wbr>libnftnl-1.0.7.patch<br>
delete mode 100644 meta/recipes-connectivity/<wbr>connman/connman/0003-stats-<wbr>Fix-bad-file-descriptor-<wbr>initialisation.patch<br>
rename meta/recipes-connectivity/<wbr>connman/{<a href="http://connman_1.33.bb" rel="noreferrer" target="_blank">connman_1.33.bb</a> => <a href="http://connman_1.34.bb" rel="noreferrer" target="_blank">connman_1.34.bb</a>} (70%)<br>
<br>
diff --git a/meta/recipes-connectivity/<wbr>connman/connman.inc b/meta/recipes-connectivity/<wbr>connman/connman.inc<br>
index 64a5418..7c7b5c7 100644<br>
--- a/meta/recipes-connectivity/<wbr>connman/connman.inc<br>
+++ b/meta/recipes-connectivity/<wbr>connman/connman.inc<br>
@@ -50,6 +50,7 @@ PACKAGECONFIG[l2tp] = "--enable-l2tp --with-l2tp=${sbindir}/xl2tpd,<wbr>--disable-l2t<br>
PACKAGECONFIG[pptp] = "--enable-pptp --with-pptp=${sbindir}/pptp,--<wbr>disable-pptp,,pptp-linux"<br>
# WISPr support for logging into hotspots, requires TLS<br>
PACKAGECONFIG[wispr] = "--enable-wispr,--disable-<wbr>wispr,gnutls,"<br>
+PACKAGECONFIG[nftables] = "--with-firewall=nftables ,,nftables,nftables"<br></blockquote><div><br></div><div>The default is "--with-firewall=iptables", I think it would be reasonable to make that explicit in the packageconfig line. Also, the iptables dependency is probably only needed in the iptables case?</div><div> </div><div>Jussi</div></div></div></div>