[PATCH v1 3/4] fff-hoods: call reload_config after uci commit

robert rlanghammer at web.de
Mo Jun 11 18:28:38 CEST 2018


Zur Diskussion, ob das zu oft drin ist: Erstmal so lassen.

Das reload_config bzw. der procd ist intelligent genug, dass ein Aufruf
zu viel nicht schadet, da er nichts tut wenn es keine Aenderungen gab.

Reviewed-by: Robert Langhammer <rlanghammer at web.de>

Am 10.06.2018 um 14:06 schrieb Tim Niemeyer:
> Signed-off-by: Tim Niemeyer <tim at tn-x.org>
> Reviewed-by: Christian Dresel <fff at chrisi01.de>
> ---
>
> Changes in v1: None
>
>  src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> index 42a489d..eaf2578 100755
> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> @@ -56,6 +56,7 @@ if [ -s "$hoodfilecopy" ] && isGatewayAvailable ; then
>  		uci set network.${iface}.proto='static'
>  		uci set network.${iface}.ip6addr='fe80::1/64'
>  		uci commit network
> +		reload_config
>  		if ! wifiAddAP "$radio" "config.franken.freifunk.net" "$iface" "configap" "1"; then
>  			echo "Can't add Config interface on $radio."
>  			exit 1
> @@ -101,6 +102,7 @@ else
>  
>  			uci -q del "system. at system[0].hood"
>  			uci -q commit system
> +			reload_config
>  		
>  			sleep 30 # Wait for the config AP, which may be created at the same time as this script has started
>  
> @@ -127,6 +129,7 @@ else
>  					uci -q set network.configSta=interface
>  					uci -q set network.configSta.proto='static'
>  					uci -q commit network
> +					reload_config
>  				fi
>  			done
>  		
> @@ -232,6 +235,7 @@ if [ -s "$hoodfile" ]; then
>  		echo "Setting hood name: $hood"
>  		uci -q set "system. at system[0].hood=$hood"
>  		uci -q commit system
> +		reload_config
>  
>  		if ! wifiDelIface; then
>  			echo "Can't delete current wifi setup"

-------------- nächster Teil --------------
Ein Dateianhang mit HTML-Daten wurde abgetrennt...
URL: <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20180611/a014c8c2/attachment.html>


Mehr Informationen über die Mailingliste franken-dev