From patchwork Tue Feb 13 12:14:31 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: configurehood: Fix "wifi" blocking correct network status detection From: Adrian Schmutzler X-Patchwork-Id: 786 Message-Id: <1518524071-2386-1-git-send-email-freifunk@adrianschmutzler.de> To: franken-dev@freifunk.net Date: Tue, 13 Feb 2018 13:14:31 +0100 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 Reviewed-by: Christian Dresel --- 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)