diff mbox

[3/3,v3] package/ifupdown-scripts: allow use with a custom skeleton

Message ID 7af4e1b5c5c358fd2a41e609ea250138a5565c18.1499006091.git.yann.morin.1998@free.fr
State Accepted
Headers show

Commit Message

Yann E. MORIN July 2, 2017, 2:35 p.m. UTC
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Arnout Vandecappelle <arnout@mind.be>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
---
 package/ifupdown-scripts/Config.in | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

Comments

Arnout Vandecappelle July 2, 2017, 4:08 p.m. UTC | #1
On 02-07-17 16:35, Yann E. MORIN wrote:
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Arnout Vandecappelle <arnout@mind.be>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>

Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

 You'll also send a patch adding a select to package/ifupdown/Config.in?

 Regards,
 Arnout

> ---
>  package/ifupdown-scripts/Config.in | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/package/ifupdown-scripts/Config.in b/package/ifupdown-scripts/Config.in
> index 7086253a03..7064d58fec 100644
> --- a/package/ifupdown-scripts/Config.in
> +++ b/package/ifupdown-scripts/Config.in
> @@ -1,8 +1,7 @@
>  config BR2_PACKAGE_IFUPDOWN_SCRIPTS
> -	bool
> -	depends on BR2_ROOTFS_SKELETON_DEFAULT
> +	bool "ifupdown scripts"
>  	depends on !BR2_PACKAGE_SYSTEMD_NETWORKD
> -	default y
> +	default y if BR2_ROOTFS_SKELETON_DEFAULT
>  	help
>  	  Set of scripts used by ifupdown (either the standalone one, or the
>  	  busybox one) to bring network up, or tear it down.
>
Yann E. MORIN July 2, 2017, 4:37 p.m. UTC | #2
Arnout, All,

On 2017-07-02 18:08 +0200, Arnout Vandecappelle spake thusly:
> On 02-07-17 16:35, Yann E. MORIN wrote:
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > Cc: Arnout Vandecappelle <arnout@mind.be>
> > Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> > Cc: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
> 
> Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> 
>  You'll also send a patch adding a select to package/ifupdown/Config.in?

As discussed IRL, it is also a legit use-case to have ifupdown without
out scripts.

Thanks for the review! :-)

Regards,
Yann E. MORIN.

>  Regards,
>  Arnout
> 
> > ---
> >  package/ifupdown-scripts/Config.in | 5 ++---
> >  1 file changed, 2 insertions(+), 3 deletions(-)
> > 
> > diff --git a/package/ifupdown-scripts/Config.in b/package/ifupdown-scripts/Config.in
> > index 7086253a03..7064d58fec 100644
> > --- a/package/ifupdown-scripts/Config.in
> > +++ b/package/ifupdown-scripts/Config.in
> > @@ -1,8 +1,7 @@
> >  config BR2_PACKAGE_IFUPDOWN_SCRIPTS
> > -	bool
> > -	depends on BR2_ROOTFS_SKELETON_DEFAULT
> > +	bool "ifupdown scripts"
> >  	depends on !BR2_PACKAGE_SYSTEMD_NETWORKD
> > -	default y
> > +	default y if BR2_ROOTFS_SKELETON_DEFAULT
> >  	help
> >  	  Set of scripts used by ifupdown (either the standalone one, or the
> >  	  busybox one) to bring network up, or tear it down.
> > 
> 
> -- 
> Arnout Vandecappelle                          arnout at mind be
> Senior Embedded Software Architect            +32-16-286500
> Essensium/Mind                                http://www.mind.be
> G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
> LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
> GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
diff mbox

Patch

diff --git a/package/ifupdown-scripts/Config.in b/package/ifupdown-scripts/Config.in
index 7086253a03..7064d58fec 100644
--- a/package/ifupdown-scripts/Config.in
+++ b/package/ifupdown-scripts/Config.in
@@ -1,8 +1,7 @@ 
 config BR2_PACKAGE_IFUPDOWN_SCRIPTS
-	bool
-	depends on BR2_ROOTFS_SKELETON_DEFAULT
+	bool "ifupdown scripts"
 	depends on !BR2_PACKAGE_SYSTEMD_NETWORKD
-	default y
+	default y if BR2_ROOTFS_SKELETON_DEFAULT
 	help
 	  Set of scripts used by ifupdown (either the standalone one, or the
 	  busybox one) to bring network up, or tear it down.