[PATCH] configurehood: Fix "wifi" blocking correct network status detection

[Tim Niemeyer]

Hi

Applied.

Bitte das nächste mal, wenn wir kurz vor einem Release stehen für
Release-kritische Bugs ein Mantis Ticket anlegen.

Das Thema hier hat mir jetzt n netten Berg Extra-Aufwand beschert, weil
es einfach niemand als relevant angesehen hat.

Tim

Am Dienstag, den 13.02.2018, 13:14 +0100 schrieb Adrian Schmutzler:
> In commit 5c1d3b1, conditions were changed so that "wifi" is called
> almost every run directly before the network status (internet vs.
> gateway contact vs. wXsta).
> 
> This causes strange effects configurehood and nodewatcher, which is
> called at the same time.
> 
> The fix only does "wifi" if required and waits 10 sec. after that.
> 
> Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> ---
>  src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> index cf79b42..3facc79 100755
> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> @@ -39,6 +39,7 @@ hasInternet() {
>  # Hidden AP check
>  
>  if [ -s "$hoodfilecopy" ] && isGatewayAvailable ; then
> +	needwifi="0"
>  	for radio in $(uci show wireless | sed -n 's,.*\.\([a-z0-
> 9]*\)=wifi-device,\1,p'); do
>  		freq="2"
>  		if [ "$(uci get "wireless.${radio}.channel")" -gt
> "14" ]; then
> @@ -59,9 +60,13 @@ if [ -s "$hoodfilecopy" ] && isGatewayAvailable ;
> then
>  			echo "Can't add Config interface on $radio."
>  			exit 1
>  		fi
> +		needwifi="1"
>  	done
>  
> -	wifi
> +	if [ "$needwifi" = "1" ] ; then
> +		wifi
> +		sleep 10
> +	fi
>  fi
>  
>  lat=$(uci -q get fff.system.latitude)
> -- 
> 2.7.4
> 
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 488 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20180303/5e384ae1/attachment.sig>