<div dir="ltr"><div dir="ltr">On Thu, 19 Dec 2019 at 18:18, Khem Raj <<a href="mailto:raj.khem@gmail.com">raj.khem@gmail.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, 2019-12-19 at 18:12 +0100, Alexander Kanavin wrote:<br>
> Sure, added them.<br>
> <br>
> I booted a system without the patch, and the network did go up<br>
> without errors.<br>
> <br>
> If the patch is still relevant, I would suggest that an effort is<br>
> made to get it upstream. It will continue to be difficult to rebase.<br>
> <br>
<br>
I dont disagree. Until then I also worry that we might regress on what<br>
we have provided to end user. So it will help in getting more info<br>
either way.<br></blockquote><div><br></div><div>Sure, although I also tend to think that if something is not covered by the AB testing, then there are no promises about that something. If AB doesn't fail, then it's fine to remove.</div><div><br></div><div>Alex<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> Alex<br>
> <br>
> On Thu, 19 Dec 2019 at 18:03, Khem Raj <<a href="mailto:raj.khem@gmail.com" target="_blank">raj.khem@gmail.com</a>> wrote:<br>
> > On Thu, 2019-12-19 at 14:22 +0100, Alexander Kanavin wrote:<br>
> > > Remove inet-6-.defn-fix-inverted-checks-for-loopback.patch as<br>
> > > it is difficult to rebase and not clear if still necessary.<br>
> > > <br>
> > <br>
> > perhaps Cc the patch authors and get an opinion ?<br>
> > <br>
> > > Signed-off-by: Alexander Kanavin <<a href="mailto:alex.kanavin@gmail.com" target="_blank">alex.kanavin@gmail.com</a>><br>
> > > ---<br>
> > > ...efn-fix-inverted-checks-for-loopback.patch | 403 ------------<br>
> > ----<br>
> > > --<br>
> > > ...{<a href="http://ifupdown_0.8.22.bb" rel="noreferrer" target="_blank">ifupdown_0.8.22.bb</a> => <a href="http://ifupdown_0.8.35.bb" rel="noreferrer" target="_blank">ifupdown_0.8.35.bb</a>} | 11 +-<br>
> > > 2 files changed, 5 insertions(+), 409 deletions(-)<br>
> > > delete mode 100644 meta/recipes-core/ifupdown/files/inet-6-<br>
> > .defn-<br>
> > > fix-inverted-checks-for-loopback.patch<br>
> > > rename meta/recipes-core/ifupdown/{<a href="http://ifupdown_0.8.22.bb" rel="noreferrer" target="_blank">ifupdown_0.8.22.bb</a> =><br>
> > > <a href="http://ifupdown_0.8.35.bb" rel="noreferrer" target="_blank">ifupdown_0.8.35.bb</a>} (83%)<br>
> > > <br>
> > > diff --git a/meta/recipes-core/ifupdown/files/inet-6-.defn-fix-<br>
> > > inverted-checks-for-loopback.patch b/meta/recipes-<br>
> > > core/ifupdown/files/inet-6-.defn-fix-inverted-checks-for-<br>
> > > loopback.patch<br>
> > > deleted file mode 100644<br>
> > > index 5b0d51d736f..00000000000<br>
> > > --- a/meta/recipes-core/ifupdown/files/inet-6-.defn-fix-inverted-<br>
> > > checks-for-loopback.patch<br>
> > > +++ /dev/null<br>
> > > @@ -1,403 +0,0 @@<br>
> > > -From 6fce99c9e42cbacde1855473b745ca1fded3fbf7 Mon Sep 17<br>
> > 00:00:00<br>
> > > 2001<br>
> > > -From: "Maxin B. John" <<a href="mailto:maxin.john@intel.com" target="_blank">maxin.john@intel.com</a>><br>
> > > -Date: Wed, 21 Dec 2016 15:32:07 +0200<br>
> > > -Subject: [PATCH 2/2] inet[6].defn: fix inverted checks for<br>
> > loopback<br>
> > > -<br>
> > > -Compared to the hurd link.defn for loopback, we see these<br>
> > > -are inverted, meaning that you would only be able to configure<br>
> > > -a loopback device that was _not_ named "lo" (unlikely to exist).<br>
> > > -<br>
> > > -The result was that we'd update /run/network/ifstate for "lo"<br>
> > > -but never actually do anything for up/down, as shown below:<br>
> > > -<br>
> > > -root@localhost:~# ifconfig -s<br>
> > > -Iface MTU Met RX-OK RX-ERR RX-DRP RX-OVR TX-OK TX-ERR TX-<br>
> > DRP<br>
> > > TX-OVR Flg<br>
> > > -eth0 1500 0 7736329 0 2016 0<br>
> > > 5289422 0 0 0 BMRU<br>
> > > -lo 65536 0 18 0 0<br>
> > > 0 18 0 0 0 LRU<br>
> > > -root@localhost:~# ifdown lo<br>
> > > -root@localhost:~# echo $?<br>
> > > -0<br>
> > > -root@localhost:~# ifconfig -s<br>
> > > -Iface MTU Met RX-OK RX-ERR RX-DRP RX-OVR TX-OK TX-ERR<br>
> > TX-DRP <br>
> > > TX-OVR Flg<br>
> > > -eth0 1500 0 7736406 0 2016<br>
> > > 0 5289455 0 0 0 BMRU<br>
> > > -lo 65536 0 18 0 0<br>
> > > 0 18 0 0 0 LRU<br>
> > > -root@localhost:~# ifconfig lo down<br>
> > > -root@localhost:~# ifconfig -s<br>
> > > -Iface MTU Met RX-OK RX-ERR RX-DRP RX-OVR TX-OK TX-ERR TX-<br>
> > DRP<br>
> > > TX-OVR Flg<br>
> > > -eth0 1500 0 7736474 0 2016<br>
> > > 0 5289481 0 0 0 BMRU<br>
> > > -root@localhost:~#<br>
> > > -<br>
> > > -Also reverted the commit:<br>
> > > -commit 80b878497663dae08f70b4d3cffe127b57a3cfc<br>
> > > -which uses absolute paths to binaries called by ifup/ifdown.<br>
> > > -<br>
> > > -Upstream-Status: Inappropriate [OE specific]<br>
> > > -<br>
> > > -Signed-off-by: Paul Gortmaker <<a href="mailto:paul.gortmaker@windriver.com" target="_blank">paul.gortmaker@windriver.com</a>><br>
> > > -Signed-off-by: Maxin B. John <<a href="mailto:maxin.john@intel.com" target="_blank">maxin.john@intel.com</a>><br>
> > > -Signed-off-by: Yi Zhao <<a href="mailto:yi.zhao@windriver.com" target="_blank">yi.zhao@windriver.com</a>><br>
> > > ----<br>
> > > - inet.defn | 134 +++++++++++++++++++++++++++------------------<br>
> > ----<br>
> > > -----<br>
> > > - 1 file changed, 67 insertions(+), 67 deletions(-)<br>
> > > -<br>
> > > -diff --git a/inet.defn b/inet.defn<br>
> > > -index 182b56b..31067bc 100644<br>
> > > ---- a/inet.defn<br>
> > > -+++ b/inet.defn<br>
> > > -@@ -6,10 +6,10 @@ method loopback<br>
> > > - This method may be used to define the IPv4 loopback<br>
> > interface.<br>
> > > - <br>
> > > - up<br>
> > > -- /bin/ip link set dev %iface% up if (!iface_is_lo())<br>
> > > -+ ip link set dev %iface% up if (!iface_is_lo())<br>
> > > - <br>
> > > - down<br>
> > > -- /bin/ip link set dev %iface% down if (!iface_is_lo())<br>
> > > -+ ip link set dev %iface% down if (!iface_is_lo())<br>
> > > - <br>
> > > - method static<br>
> > > - description<br>
> > > -@@ -36,17 +36,17 @@ method static<br>
> > > - broadcast compute_v4_broadcast<br>
> > > - <br>
> > > - up<br>
> > > -- /bin/ip addr add %address%[[/%netmask%]] [[broadcast<br>
> > > %broadcast%]] \<br>
> > > -+ ip addr add %address%[[/%netmask%]] [[broadcast<br>
> > %broadcast%]] \<br>
> > > - [[peer %pointopoint%]] [[scope %scope%]] dev %iface% label<br>
> > > %iface%<br>
> > > -- /bin/ip link set dev %iface% [[mtu %mtu%]] [[address<br>
> > > %hwaddress%]] up<br>
> > > -+ ip link set dev %iface% [[mtu %mtu%]] [[address<br>
> > %hwaddress%]]<br>
> > > up<br>
> > > - <br>
> > > -- [[ /bin/ip route add default via %gateway% [[metric<br>
> > %metric%]]<br>
> > > dev %iface% onlink ]]<br>
> > > -+ [[ ip route add default via %gateway% [[metric %metric%]]<br>
> > dev<br>
> > > %iface% onlink ]]<br>
> > > - <br>
> > > - down<br>
> > > -- [[ /bin/ip route del default via %gateway% [[metric<br>
> > %metric%]]<br>
> > > dev %iface% 2>&1 1>/dev/null || true ]]<br>
> > > -- /bin/ip addr del %address%[[/%netmask%]] [[broadcast<br>
> > > %broadcast%]] \<br>
> > > -+ [[ ip route del default via %gateway% [[metric %metric%]]<br>
> > dev<br>
> > > %iface% 2>&1 1>/dev/null || true ]]<br>
> > > -+ ip addr del %address%[[/%netmask%]] [[broadcast<br>
> > %broadcast%]] \<br>
> > > - [[peer %pointopoint%]] [[scope %scope%]] dev %iface% label<br>
> > > %iface%<br>
> > > -- /bin/ip link set dev %iface% down \<br>
> > > -+ ip link set dev %iface% down \<br>
> > > - if (iface_is_link())<br>
> > > - <br>
> > > - method manual<br>
> > > -@@ -63,12 +63,12 @@ method manual<br>
> > > - hwaddress cleanup_hwaddress<br>
> > > - <br>
> > > - up<br>
> > > -- [[/bin/ip link set dev %iface% mtu %mtu%]]<br>
> > > -- [[/bin/ip link set dev %iface% address %hwaddress%]]<br>
> > > -- /bin/ip link set dev %iface% up 2>/dev/null || true<br>
> > > -+ [[ip link set dev %iface% mtu %mtu%]]<br>
> > > -+ [[ip link set dev %iface% address %hwaddress%]]<br>
> > > -+ ip link set dev %iface% up 2>/dev/null || true<br>
> > > - <br>
> > > - down<br>
> > > -- /bin/ip link set dev %iface% down 2>/dev/null || true \<br>
> > > -+ ip link set dev %iface% down 2>/dev/null || true \<br>
> > > - if (iface_is_link() && !do_all)<br>
> > > - <br>
> > > - method dhcp<br>
> > > -@@ -93,33 +93,33 @@ method dhcp<br>
> > > - hwaddress cleanup_hwaddress<br>
> > > - <br>
> > > - up<br>
> > > -- [[/bin/ip link set dev %iface% address %hwaddress%]]<br>
> > > -- /sbin/dhclient -4 -v -pf /run/dhclient.%iface%.pid -lf<br>
> > > /var/lib/dhcp/dhclient.%iface%.leases -I -df<br>
> > > /var/lib/dhcp/dhclient6.%iface%.leases %iface% \<br>
> > > -+ [[ip link set dev %iface% address %hwaddress%]]<br>
> > > -+ dhclient -4 -v -pf /run/dhclient.%iface%.pid -lf<br>
> > > /var/lib/dhcp/dhclient.%iface%.leases -I -df<br>
> > > /var/lib/dhcp/dhclient6.%iface%.leases %iface% \<br>
> > > - [[-e IF_METRIC=%metric%]] \<br>
> > > - if (execable("/sbin/dhclient"))<br>
> > > -- /sbin/pump -i %iface% [[-h %hostname%]] [[-l %leasehours%]]<br>
> > \<br>
> > > -+ pump -i %iface% [[-h %hostname%]] [[-l %leasehours%]] \<br>
> > > - elsif (execable("/sbin/pump"))<br>
> > > -- /sbin/udhcpc -n -p /run/udhcpc.%iface%.pid -i %iface% [[-x<br>
> > > hostname:%hostname%]] \<br>
> > > -+ udhcpc -n -p /run/udhcpc.%iface%.pid -i %iface% [[-x<br>
> > > hostname:%hostname%]] \<br>
> > > - elsif (execable("/sbin/udhcpc"))<br>
> > > -- /sbin/dhcpcd [[-h %hostname%]] [[-i %vendor%]] [[-I<br>
> > %client%]]<br>
> > > \<br>
> > > -+ dhcpcd [[-h %hostname%]] [[-i %vendor%]] [[-I %client%]] \<br>
> > > - [[-l %leasetime%]] [[-m %metric%]] %iface% \<br>
> > > - elsif (execable("/sbin/dhcpcd"))<br>
> > > - echo 'No DHCP client software found!' >/dev/stderr; false \<br>
> > > - elsif (1)<br>
> > > - <br>
> > > - down<br>
> > > -- /sbin/dhclient -4 -v -r -pf /run/dhclient.%iface%.pid -lf<br>
> > > /var/lib/dhcp/dhclient.%iface%.leases -I -df<br>
> > > /var/lib/dhcp/dhclient6.%iface%.leases %iface% \<br>
> > > -+ dhclient -4 -v -r -pf /run/dhclient.%iface%.pid -lf<br>
> > > /var/lib/dhcp/dhclient.%iface%.leases -I -df<br>
> > > /var/lib/dhcp/dhclient6.%iface%.leases %iface% \<br>
> > > - if (execable("/sbin/dhclient"))<br>
> > > -- /sbin/pump -i %iface% -r \<br>
> > > -+ pump -i %iface% -r \<br>
> > > - elsif (execable("/sbin/pump"))<br>
> > > - if test -f /run/udhcpc.%iface%.pid; then kill -USR2<br>
> > $(/bin/cat<br>
> > > /run/udhcpc.%iface%.pid); kill -TERM $(/bin/cat<br>
> > > /run/udhcpc.%iface%.pid); fi \<br>
> > > - elsif (execable("/sbin/udhcpc"))<br>
> > > -- /sbin/dhcpcd -k %iface% \<br>
> > > -+ dhcpcd -k %iface% \<br>
> > > - elsif (execable("/sbin/dhcpcd"))<br>
> > > - echo 'No DHCP client software found!' >/dev/stderr; false \<br>
> > > - elsif (1)<br>
> > > - <br>
> > > -- /bin/ip link set dev %iface% down \<br>
> > > -+ ip link set dev %iface% down \<br>
> > > - if (iface_is_link())<br>
> > > - <br>
> > > - method bootp<br>
> > > -@@ -134,11 +134,11 @@ method bootp<br>
> > > - whatever it really is.<br>
> > > - <br>
> > > - up<br>
> > > -- /sbin/bootpc [[--bootfile %bootfile%]] --dev %iface% [[<br>
> > --server<br>
> > > %server%]] \<br>
> > > -+ bootpc [[--bootfile %bootfile%]] --dev %iface% [[--server<br>
> > > %server%]] \<br>
> > > - [[--hwaddr %hwaddr%]] --returniffail --serverbcast<br>
> > > - <br>
> > > - down<br>
> > > -- /bin/ip link set dev %iface% down \<br>
> > > -+ ip link set dev %iface% down \<br>
> > > - if (iface_is_link())<br>
> > > - <br>
> > > - method tunnel<br>
> > > -@@ -158,13 +158,13 @@ method tunnel<br>
> > > - ttl time -- TTL setting<br>
> > > - mtu size -- MTU size<br>
> > > - up<br>
> > > -- /bin/ip tunnel add %iface% mode %mode% remote %endpoint%<br>
> > > [[local %local%]] \<br>
> > > -+ ip tunnel add %iface% mode %mode% remote %endpoint% [[local<br>
> > > %local%]] \<br>
> > > - [[ttl %ttl%]]<br>
> > > -- /bin/ip link set %iface% up [[mtu %mtu%]]<br>
> > > -- /bin/ip addr add %address%/%netmask% dev %iface% [[peer<br>
> > > %dstaddr%]]<br>
> > > -- [[ /bin/ip route add default via %gateway% [[metric<br>
> > %metric%]]<br>
> > > dev %iface% onlink ]]<br>
> > > -+ ip link set %iface% up [[mtu %mtu%]]<br>
> > > -+ ip addr add %address%/%netmask% dev %iface% [[peer<br>
> > %dstaddr%]]<br>
> > > -+ [[ ip route add default via %gateway% [[metric %metric%]]<br>
> > dev<br>
> > > %iface% onlink ]]<br>
> > > - down<br>
> > > -- /bin/ip tunnel del %iface%<br>
> > > -+ ip tunnel del %iface%<br>
> > > - <br>
> > > - method ppp<br>
> > > - description<br>
> > > -@@ -175,9 +175,9 @@ method ppp<br>
> > > - unit number -- Use /number/ as the ppp unit number.<br>
> > > - options string -- Pass /string/ as additional options to<br>
> > pon.<br>
> > > - up<br>
> > > -- /usr/bin/pon [[%provider%]] [[unit %unit%]] [[%options%]]<br>
> > > -+ pon [[%provider%]] [[unit %unit%]] [[%options%]]<br>
> > > - down<br>
> > > -- /usr/bin/poff [[%provider%]]<br>
> > > -+ poff [[%provider%]]<br>
> > > - <br>
> > > - method wvdial<br>
> > > - description<br>
> > > -@@ -186,10 +186,10 @@ method wvdial<br>
> > > - options<br>
> > > - provider name -- Use /name/ as the provider (from<br>
> > > /etc/wvdial.conf).<br>
> > > - up<br>
> > > -- /sbin/start-stop-daemon --start -x /usr/bin/wvdial \<br>
> > > -+ start-stop-daemon --start -x /usr/bin/wvdial \<br>
> > > - -p /run/wvdial.%iface%.pid -b -m -- [[<br>
> > > %provider% ]]<br>
> > > - down<br>
> > > -- /sbin/start-stop-daemon --stop -x /usr/bin/wvdial \<br>
> > > -+ start-stop-daemon --stop -x /usr/bin/wvdial \<br>
> > > - -p /run/wvdial.%iface%.pid -s 2<br>
> > > - <br>
> > > - <br>
> > > -@@ -200,9 +200,9 @@ method ipv4ll<br>
> > > - known as APIPA or IPAC, and often colloquially referred to<br>
> > > - as "Zeroconf address".<br>
> > > - up<br>
> > > -- /usr/sbin/avahi-autoipd -D %iface%<br>
> > > -+ avahi-autoipd -D %iface%<br>
> > > - down<br>
> > > -- /usr/sbin/avahi-autoipd --kill %iface%<br>
> > > -+ avahi-autoipd --kill %iface%<br>
> > > - <br>
> > > - architecture kfreebsd<br>
> > > - <br>
> > > -@@ -211,11 +211,11 @@ method loopback<br>
> > > - This method may be used to define the IPv4 loopback<br>
> > interface.<br>
> > > - <br>
> > > - up<br>
> > > -- /sbin/ifconfig %iface% 127.0.0.1 up \<br>
> > > -+ ifconfig %iface% 127.0.0.1 up \<br>
> > > - if (!iface_is_lo())<br>
> > > - <br>
> > > - down<br>
> > > -- /sbin/ifconfig %iface% down \<br>
> > > -+ ifconfig %iface% down \<br>
> > > - if (!iface_is_lo())<br>
> > > - <br>
> > > - method static<br>
> > > -@@ -238,15 +238,15 @@ method static<br>
> > > - hwaddress cleanup_hwaddress<br>
> > > - <br>
> > > - up<br>
> > > -- [[ /sbin/ifconfig %iface% link %hwaddress%]]<br>
> > > -- /sbin/ifconfig %iface% %address% [[netmask %netmask%]]<br>
> > > [[broadcast %broadcast%]] \<br>
> > > -+ [[ ifconfig %iface% link %hwaddress%]]<br>
> > > -+ ifconfig %iface% %address% [[netmask %netmask%]]<br>
> > [[broadcast<br>
> > > %broadcast%]] \<br>
> > > - [[pointopoint %pointopoint%]] [[media %media%]] [[mtu<br>
> > %mtu%]] \<br>
> > > - up<br>
> > > -- [[ /sbin/route add default %gateway% ]]<br>
> > > -+ [[ route add default %gateway% ]]<br>
> > > - <br>
> > > - down<br>
> > > -- [[ /sbin/route del default %gateway% 2>&1 1>/dev/null ||<br>
> > true<br>
> > > ]]<br>
> > > -- /sbin/ifconfig %iface% down<br>
> > > -+ [[ route del default %gateway% 2>&1 1>/dev/null || true ]]<br>
> > > -+ ifconfig %iface% down<br>
> > > - <br>
> > > - method manual<br>
> > > - description<br>
> > > -@@ -279,30 +279,30 @@ method dhcp<br>
> > > - hwaddress cleanup_hwaddress<br>
> > > - <br>
> > > - up<br>
> > > -- [[/sbin/ifconfig %iface% link %hwaddress%]]<br>
> > > -- /sbin/dhclient -4 -v -pf /run/dhclient.%iface%.pid -lf<br>
> > > /var/lib/dhcp/dhclient.%iface%.leases -I -df<br>
> > > /var/lib/dhcp/dhclient6.%iface%.leases %iface% \<br>
> > > -+ [[ifconfig %iface% link %hwaddress%]]<br>
> > > -+ dhclient -4 -v -pf /run/dhclient.%iface%.pid -lf<br>
> > > /var/lib/dhcp/dhclient.%iface%.leases -I -df<br>
> > > /var/lib/dhcp/dhclient6.%iface%.leases %iface% \<br>
> > > - [[-e IF_METRIC=%metric%]] \<br>
> > > - if (execable("/sbin/dhclient"))<br>
> > > -- /sbin/udhcpc -n -p /run/udhcpc.%iface%.pid -i %iface% [[-H<br>
> > > %hostname%]] \<br>
> > > -+ udhcpc -n -p /run/udhcpc.%iface%.pid -i %iface% [[-H<br>
> > > %hostname%]] \<br>
> > > - [[-c %client%]] \<br>
> > > - elsif (execable("/sbin/udhcpc"))<br>
> > > -- /sbin/dhcpcd [[-h %hostname%]] [[-i %vendor%]] [[-I<br>
> > %client%]]<br>
> > > \<br>
> > > -+ dhcpcd [[-h %hostname%]] [[-i %vendor%]] [[-I %client%]] \<br>
> > > - [[-l %leasetime%]] %iface% \<br>
> > > - elsif (execable("/sbin/dhcpcd"))<br>
> > > - echo 'No DHCP client software found!' >/dev/stderr; false \<br>
> > > - elsif (1)<br>
> > > - <br>
> > > - down<br>
> > > -- /sbin/dhclient -4 -v -r -pf /run/dhclient.%iface%.pid -lf<br>
> > > /var/lib/dhcp/dhclient.%iface%.leases -I -df<br>
> > > /var/lib/dhcp/dhclient6.%iface%.leases %iface% \<br>
> > > -+ dhclient -4 -v -r -pf /run/dhclient.%iface%.pid -lf<br>
> > > /var/lib/dhcp/dhclient.%iface%.leases -I -df<br>
> > > /var/lib/dhcp/dhclient6.%iface%.leases %iface% \<br>
> > > - if (execable("/sbin/dhclient"))<br>
> > > - if test -f /run/udhcpc.%iface%.pid; then kill -USR2<br>
> > $(/bin/cat<br>
> > > /run/udhcpc.%iface%.pid); kill -TERM $(/bin/cat<br>
> > > /run/udhcpc.%iface%.pid); fi \<br>
> > > - elsif (execable("/sbin/udhcpc"))<br>
> > > -- /sbin/dhcpcd -k %iface% \<br>
> > > -+ dhcpcd -k %iface% \<br>
> > > - elsif (execable("/sbin/dhcpcd"))<br>
> > > - echo 'No DHCP client software found!' >/dev/stderr; false \<br>
> > > - elsif (1)<br>
> > > - <br>
> > > -- /sbin/ifconfig %iface% down<br>
> > > -+ ifconfig %iface% down<br>
> > > - <br>
> > > - method bootp<br>
> > > - description<br>
> > > -@@ -316,11 +316,11 @@ method bootp<br>
> > > - whatever it really is.<br>
> > > - <br>
> > > - up<br>
> > > -- /sbin/bootpc [[--bootfile %bootfile%]] --dev %iface% [[<br>
> > --server<br>
> > > %server%]] \<br>
> > > -+ bootpc [[--bootfile %bootfile%]] --dev %iface% [[--server<br>
> > > %server%]] \<br>
> > > - [[--hwaddr %hwaddr%]] --returniffail --serverbcast<br>
> > > - <br>
> > > - down<br>
> > > -- /sbin/ifconfig %iface% down<br>
> > > -+ ifconfig %iface% down<br>
> > > - <br>
> > > - method ppp<br>
> > > - description<br>
> > > -@@ -331,9 +331,9 @@ method ppp<br>
> > > - unit number -- Use /number/ as the ppp unit number.<br>
> > > - options string -- Pass /string/ as additional options to<br>
> > pon.<br>
> > > - up<br>
> > > -- /usr/bin/pon [[%provider%]] [[unit %unit%]] [[%options%]]<br>
> > > -+ pon [[%provider%]] [[unit %unit%]] [[%options%]]<br>
> > > - down<br>
> > > -- /usr/bin/poff [[%provider%]]<br>
> > > -+ poff [[%provider%]]<br>
> > > - <br>
> > > - method wvdial<br>
> > > - description<br>
> > > -@@ -342,10 +342,10 @@ method wvdial<br>
> > > - options<br>
> > > - provider name -- Use /name/ as the provider (from<br>
> > > /etc/wvdial.conf).<br>
> > > - up<br>
> > > -- /sbin/start-stop-daemon --start -x /usr/bin/wvdial \<br>
> > > -+ start-stop-daemon --start -x /usr/bin/wvdial \<br>
> > > - -p /run/wvdial.%iface%.pid -b -m -- [[<br>
> > > %provider% ]]<br>
> > > - down<br>
> > > -- /sbin/start-stop-daemon --stop -x /usr/bin/wvdial \<br>
> > > -+ start-stop-daemon --stop -x /usr/bin/wvdial \<br>
> > > - -p /run/wvdial.%iface%.pid -s 2<br>
> > > - <br>
> > > - <br>
> > > -@@ -356,9 +356,9 @@ method ipv4ll<br>
> > > - known as APIPA or IPAC, and often colloquially referred to<br>
> > > - as "Zeroconf address".<br>
> > > - up<br>
> > > -- /usr/sbin/avahi-autoipd -D %iface%<br>
> > > -+ avahi-autoipd -D %iface%<br>
> > > - down<br>
> > > -- /usr/sbin/avahi-autoipd --kill %iface%<br>
> > > -+ avahi-autoipd --kill %iface%<br>
> > > - architecture hurd<br>
> > > - <br>
> > > - method loopback<br>
> > > -@@ -432,23 +432,23 @@ method dhcp<br>
> > > - <br>
> > > - up<br>
> > > - [[Warning: Option hwaddress: %hwaddress% not yet<br>
> > supported]]<br>
> > > -- /sbin/dhclient -4 -v -pf /run/dhclient.%iface///.%.pid -lf<br>
> > > /var/lib/dhcp/dhclient.%iface///.%.leases -I -df<br>
> > > /var/lib/dhcp/dhclient6.%iface///.%.leases %iface% \<br>
> > > -+ dhclient -4 -v -pf /run/dhclient.%iface///.%.pid -lf<br>
> > > /var/lib/dhcp/dhclient.%iface///.%.leases -I -df<br>
> > > /var/lib/dhcp/dhclient6.%iface///.%.leases %iface% \<br>
> > > - if (execable("/sbin/dhclient"))<br>
> > > -- /sbin/udhcpc -n -p /run/udhcpc.%iface///.%.pid -i %iface%<br>
> > [[-H<br>
> > > %hostname%]] \<br>
> > > -+ udhcpc -n -p /run/udhcpc.%iface///.%.pid -i %iface% [[-H<br>
> > > %hostname%]] \<br>
> > > - [[-c %client%]] \<br>
> > > - elsif (execable("/sbin/udhcpc"))<br>
> > > -- /sbin/dhcpcd [[-h %hostname%]] [[-i %vendor%]] [[-I<br>
> > %client%]]<br>
> > > \<br>
> > > -+ dhcpcd [[-h %hostname%]] [[-i %vendor%]] [[-I %client%]] \<br>
> > > - [[-l %leasetime%]] %iface% \<br>
> > > - elsif (execable("/sbin/dhcpcd"))<br>
> > > - echo 'No DHCP client software found!' >/dev/stderr; false \<br>
> > > - elsif (1)<br>
> > > - <br>
> > > - down<br>
> > > -- /sbin/dhclient -4 -v -r -pf /run/dhclient.%iface///.%.pid<br>
> > -lf<br>
> > > /var/lib/dhcp/dhclient.%iface///.%.leases -I -df<br>
> > > /var/lib/dhcp/dhclient6.%iface///.%.leases %iface% \<br>
> > > -+ dhclient -4 -v -r -pf /run/dhclient.%iface///.%.pid -lf<br>
> > > /var/lib/dhcp/dhclient.%iface///.%.leases -I -df<br>
> > > /var/lib/dhcp/dhclient6.%iface///.%.leases %iface% \<br>
> > > - if (execable("/sbin/dhclient"))<br>
> > > - if test -f /run/udhcpc.%iface///.%.pid; then kill -USR2<br>
> > > $(/bin/cat /run/udhcpc.%iface///.%.pid); kill -TERM $(/bin/cat<br>
> > > /run/udhcpc.%iface///.%.pid); fi \<br>
> > > - elsif (execable("/sbin/udhcpc"))<br>
> > > -- /sbin/dhcpcd -k %iface% \<br>
> > > -+ dhcpcd -k %iface% \<br>
> > > - elsif (execable("/sbin/dhcpcd"))<br>
> > > - echo 'No DHCP client software found!' >/dev/stderr; false \<br>
> > > - elsif (1)<br>
> > > -@@ -482,9 +482,9 @@ method ppp<br>
> > > - unit number -- Use /number/ as the ppp unit number.<br>
> > > - options string -- Pass /string/ as additional options to<br>
> > pon.<br>
> > > - up<br>
> > > -- /usr/bin/pon [[%provider%]] [[unit %unit%]] [[%options%]]<br>
> > > -+ pon [[%provider%]] [[unit %unit%]] [[%options%]]<br>
> > > - down<br>
> > > -- /usr/bin/poff [[%provider%]]<br>
> > > -+ poff [[%provider%]]<br>
> > > - <br>
> > > - method wvdial<br>
> > > - description<br>
> > > -@@ -493,10 +493,10 @@ method wvdial<br>
> > > - options<br>
> > > - provider name -- Use /name/ as the provider (from<br>
> > > /etc/wvdial.conf).<br>
> > > - up<br>
> > > -- /sbin/start-stop-daemon --start -x /usr/bin/wvdial \<br>
> > > -+ start-stop-daemon --start -x /usr/bin/wvdial \<br>
> > > - -p /run/wvdial.%iface///.%.pid -b -m --<br>
> > [[<br>
> > > %provider% ]]<br>
> > > - down<br>
> > > -- /sbin/start-stop-daemon --stop -x /usr/bin/wvdial \<br>
> > > -+ start-stop-daemon --stop -x /usr/bin/wvdial \<br>
> > > - -p /run/wvdial.%iface///.%.pid -s 2<br>
> > > - <br>
> > > - <br>
> > > -@@ -507,6 +507,6 @@ method ipv4ll<br>
> > > - known as APIPA or IPAC, and often colloquially referred to<br>
> > > - as "Zeroconf address".<br>
> > > - up<br>
> > > -- /usr/sbin/avahi-autoipd -D %iface%<br>
> > > -+ avahi-autoipd -D %iface%<br>
> > > - down<br>
> > > -- /usr/sbin/avahi-autoipd --kill %iface%<br>
> > > -+ avahi-autoipd --kill %iface%<br>
> > > --- <br>
> > > -2.17.1<br>
> > > -<br>
> > > diff --git a/meta/recipes-core/ifupdown/<a href="http://ifupdown_0.8.22.bb" rel="noreferrer" target="_blank">ifupdown_0.8.22.bb</a><br>
> > > b/meta/recipes-core/ifupdown/<a href="http://ifupdown_0.8.35.bb" rel="noreferrer" target="_blank">ifupdown_0.8.35.bb</a><br>
> > > similarity index 83%<br>
> > > rename from meta/recipes-core/ifupdown/<a href="http://ifupdown_0.8.22.bb" rel="noreferrer" target="_blank">ifupdown_0.8.22.bb</a><br>
> > > rename to meta/recipes-core/ifupdown/<a href="http://ifupdown_0.8.35.bb" rel="noreferrer" target="_blank">ifupdown_0.8.35.bb</a><br>
> > > index bd0989e8855..de8e1cf66ba 100644<br>
> > > --- a/meta/recipes-core/ifupdown/<a href="http://ifupdown_0.8.22.bb" rel="noreferrer" target="_blank">ifupdown_0.8.22.bb</a><br>
> > > +++ b/meta/recipes-core/ifupdown/<a href="http://ifupdown_0.8.35.bb" rel="noreferrer" target="_blank">ifupdown_0.8.35.bb</a><br>
> > > @@ -7,12 +7,11 @@ LICENSE = "GPLv2"<br>
> > > LIC_FILES_CHKSUM =<br>
> > > "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f"<br>
> > > <br>
> > > SRC_URI =<br>
> > "git://<a href="http://salsa.debian.org/debian/ifupdown.git;protocol=https" rel="noreferrer" target="_blank">salsa.debian.org/debian/ifupdown.git;protocol=https</a><br>
> > > \<br>
> > > - file://defn2-c-man-don-t-rely-on-dpkg-architecture-to-<br>
> > set-<br>
> > > a.patch \<br>
> > > - file://inet-6-.defn-fix-inverted-checks-for-<br>
> > loopback.patch \<br>
> > > - file://99_network \<br>
> > > - file://0001-Define-FNM_EXTMATCH-for-musl.patch \<br>
> > > - "<br>
> > > -SRCREV = "ab5a0f464e53e172316a5ca8b5dcdc49e8848999"<br>
> > > + file://defn2-c-man-don-t-rely-on-dpkg-architecture-<br>
> > to-<br>
> > > set-a.patch \<br>
> > > + file://99_network \<br>
> > > + file://0001-Define-FNM_EXTMATCH-for-musl.patch \<br>
> > > + "<br>
> > > +SRCREV = "4af76318cfc57f8e4a44d357104188666213bd4b"<br>
> > > <br>
> > > S = "${WORKDIR}/git"<br>
> > > <br>
> > > -- <br>
> > > 2.17.1<br>
> > > <br>
> > <br>
<br>
</blockquote></div></div>