[Buildroot] [PATCH 1/1] netplug (/etc/init.d/S29netplug): Fix the quoting of ${NETWORKING}.

[Thomas Petazzoni]

Hello,

On Thu, 15 Nov 2018 14:48:17 +0100, Thomas Claveirole wrote:
> Since 4adaa581b27dc27298eb7cc421ee8f530f88e18e, S29netplug looks for
> /etc/default/network instead of /etc/sysconfig/network.  When this
> file exists but does not define $NETWORKING, the script fails on line
> 29 with something like:
> 
> /etc/init.d/S29netplug: 29: [: =: unexpected operator
> 
> Fix quoting so this error no longer happens.
> 
> Signed-off-by: Thomas Claveirole <thomas.claveirole at green-communications.fr>
> ---
>  package/netplug/S29netplug | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/package/netplug/S29netplug b/package/netplug/S29netplug
> index 66459061b3..6c344662bb 100755
> --- a/package/netplug/S29netplug
> +++ b/package/netplug/S29netplug
> @@ -26,7 +26,7 @@ if [ -f /etc/default/network ]; then
>  	. /etc/default/network
>  
>  	# Check that networking is up.
> -	[ ${NETWORKING} = "no" ] && exit 0
> +	[ "$NETWORKING" = no ] && exit 0

I don't think it was really necessary to remove the double quotes
around "no", and to remove the curly braces to reference the variable.
So I've added those back, tweaked the commit title and applied to
master. Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com