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

[Christian Dresel]

hi

war mir nicht bewusst das dies so kritisch ist :( Aber jetzt weiß ich wo
mein Problem liegt das mich schon einige Zeit nervt:

Reviewed-by: Christian Dresel <fff at chrisi01.de>

Danke

mfg

Christian

On 13.02.2018 13:14, Adrian Schmutzler wrote:
> 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)
> 

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: OpenPGP digital signature
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20180303/1f3982ba/attachment.sig>