[PATCH] Add option to set Hoodconfiguration fix

[mail at adrianschmutzler.de]

Hallo Christian,

habe kein Problem mit der Idee an sich, würde das aber ein bisschen anders
machen:

Langfristig werden wir wohl ein manuelle Hoodselektion einbauen. Deshalb und
wegen der Ordnung würde ich gerne einen eigenen Block "hood" verwenden. Die
Kommandos wären dementsprechend:

uci set fff.hood=fff
uci set fff.hood.fixed=1
uci commit fff

to revert this use:
uci del fff.hood.fixed ODER uci set fff.hood.fixed=0
uci commit fff

Im Code dann dementsprechend:
if [ "$(uci -q get fff.hood.fixed)" = "1" ]; then

Später könnte man dann z.B. fff.hood.name oder fff.hood.setname nutzen, um
die Hood per Namen festzulegen. Dann spammen wir den System Block nicht zu.

Wenn ich es richtig verstanden habe, deaktiviert du das configurehood mit
deiner Lösung komplett? Bin am überlegen, ob man das statt "fix/fixed" dann
auf "disabled" umbenennen sollte?

Und noch die obligatorische Klugscheißer-Anmerkung: Ich würde aus
sprachlichen Gründen überall "fixed" statt "fix" verwenden.

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
> Of Christian Dresel
> Sent: Dienstag, 2. Januar 2018 13:21
> To: franken-dev at freifunk.net
> Subject: [PATCH] Add option to set Hoodconfiguration fix
> 
> To set Hoodconfiguration fix use:
> uci set fff.system.hoodfix=1
> uci commit
> to revert this use:
> uci del fff.system.hoodfix
> uci commit
> 
> Signed-off-by: Christian Dresel <fff at chrisi01.de>
> ---
>  src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> index 2b9c47e..02d9a90 100755
> --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
> @@ -1,5 +1,10 @@
>  #!/bin/sh
> 
> +if [ $(uci -q get fff.system.hoodfix) ]; then
> +        echo "Hood is fix, we do nothing"
> +        exit 1
> +fi
> +
>  . /usr/share/libubox/jshn.sh
>  . /lib/functions/fff/keyxchange
>  . /lib/functions/fff/network
> --
> 2.11.0
> 
> --
> franken-dev mailing list
> franken-dev at freifunk.net
> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net