[PATCH] ifupdown: fix issue that ifupdown does not configure local interface

[PATCH] ifupdown: fix issue that ifupdown does not configure local interface

[Haiqing Bai]

The patch "inet-6-.defn-fix-inverted-checks-for-loopback.patch" is not
needed for ifupdown-0.8.16. And it causes the issue that ifupdown can
not configure the loopback interface like:
$cat /etc/network/interfaces
auto lo
iface lo inet loopback
$ifconfig shows all up interfaces including 'lo'
$ifup lo
ifup: interface lo already configured
$ifdown lo
$ifconfig still shows the up interface 'lo'

The function of ifup/ifdown depends on '/bin/ip' which is provided by
iproute2, so here add this runtime dependency.

Signed-off-by: Haiqing Bai <Haiqing.Bai@windriver.com>
---
 meta/recipes-core/ifupdown/ifupdown_0.8.16.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/ifupdown/ifupdown_0.8.16.bb b/meta/recipes-core/ifupdown/ifupdown_0.8.16.bb
index e9f3a2a..f91831f 100644
--- a/meta/recipes-core/ifupdown/ifupdown_0.8.16.bb
+++ b/meta/recipes-core/ifupdown/ifupdown_0.8.16.bb
@@ -8,13 +8,13 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f"
 
 SRC_URI = "git://salsa.debian.org/debian/ifupdown.git;protocol=https \
 	   file://defn2-c-man-don-t-rely-on-dpkg-architecture-to-set-a.patch \
-	   file://inet-6-.defn-fix-inverted-checks-for-loopback.patch \
 	   file://99_network \
 	  "
 SRCREV = "11b9f99f7ecc7052497e6786156cfed531f11823"
 
 S = "${WORKDIR}/git"
 
+RDEPENDS_${PN} = "iproute2"
 
 inherit update-alternatives
 
-- 
1.9.1

Re: [PATCH] ifupdown: fix issue that ifupdown does not configure local interface

[Andreas Oberritter]

On Wed, 11 Jul 2018 15:29:46 +0800
Haiqing Bai <Haiqing.Bai@windriver.com> wrote:

> The patch "inet-6-.defn-fix-inverted-checks-for-loopback.patch" is not
> needed for ifupdown-0.8.16. And it causes the issue that ifupdown can
> not configure the loopback interface like:
> $cat /etc/network/interfaces
> auto lo
> iface lo inet loopback
> $ifconfig shows all up interfaces including 'lo'
> $ifup lo
> ifup: interface lo already configured
> $ifdown lo
> $ifconfig still shows the up interface 'lo'
> 
> The function of ifup/ifdown depends on '/bin/ip' which is provided by
> iproute2, so here add this runtime dependency.
> 
> Signed-off-by: Haiqing Bai <Haiqing.Bai@windriver.com>
> ---
>  meta/recipes-core/ifupdown/ifupdown_0.8.16.bb | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-core/ifupdown/ifupdown_0.8.16.bb b/meta/recipes-core/ifupdown/ifupdown_0.8.16.bb
> index e9f3a2a..f91831f 100644
> --- a/meta/recipes-core/ifupdown/ifupdown_0.8.16.bb
> +++ b/meta/recipes-core/ifupdown/ifupdown_0.8.16.bb
> @@ -8,13 +8,13 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f"
>  
>  SRC_URI = "git://salsa.debian.org/debian/ifupdown.git;protocol=https \
>  	   file://defn2-c-man-don-t-rely-on-dpkg-architecture-to-set-a.patch \
> -	   file://inet-6-.defn-fix-inverted-checks-for-loopback.patch \
>  	   file://99_network \
>  	  "
>  SRCREV = "11b9f99f7ecc7052497e6786156cfed531f11823"
>  
>  S = "${WORKDIR}/git"
>  
> +RDEPENDS_${PN} = "iproute2"

Does busybox RPROVIDE iproute2?

>  
>  inherit update-alternatives
>