[PATCH v3 2/3] init.d/fff-hoods: Move call of configurehood to init.d script

[Tim Niemeyer]

Hi

Am Freitag, den 05.01.2018, 00:52 +0100 schrieb Adrian Schmutzler:
> This ensures that configurehood is executed AFTER the LEDS are
> set up, but BEFORE alfred.
Wir hatten ja vorhin schon festgestellt, dass das nicht nötig ist, weil
wir ja br-mesh eh dauerhaft da haben wollen.

> If a configap is reachable, we are thus able to set up w2mesh
> before alfred tries to access br-mesh.
Ja, das hilft dann manchmal, aber wenn eben kein configap da ist, dann
geht hilft das eh nicht. Also wie besprochen, br-mesh muss immer da
sein.

> As a side effect, we move the call of configurehood to the
> fff-hoods package, where it belongs.
Ja, sehr gut.

> Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> Tested-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> 
> ---
> 
> Changes in v2:
> none
> 
> Changes in v3:
> - Changed START=99 to START=98
> ---
>  bsp/default/root_file_system/etc/rc.local        |  4 ----
>  src/packages/fff/fff-hoods/files/etc/init.d/hood | 12 ++++++++++++
>  2 files changed, 12 insertions(+), 4 deletions(-)
>  create mode 100755 src/packages/fff/fff-hoods/files/etc/init.d/hood
> 
> diff --git a/bsp/default/root_file_system/etc/rc.local
> b/bsp/default/root_file_system/etc/rc.local
> index 7410bae..f174234 100755
> --- a/bsp/default/root_file_system/etc/rc.local
> +++ b/bsp/default/root_file_system/etc/rc.local
> @@ -4,8 +4,4 @@
>  
>  /usr/sbin/configurenetwork
>  
> -sleep 3
> -
> -/usr/sbin/configurehood
> -
>  exit 0
> diff --git a/src/packages/fff/fff-hoods/files/etc/init.d/hood
> b/src/packages/fff/fff-hoods/files/etc/init.d/hood
> new file mode 100755
> index 0000000..63d58d3
> --- /dev/null
> +++ b/src/packages/fff/fff-hoods/files/etc/init.d/hood
> @@ -0,0 +1,12 @@
> +#!/bin/sh /etc/rc.common
> +
> +# Copyright 2017 Adrian Schmutzler
> +# License GPLv3
> +
> +START=98

Kannst du erläutern, warum 98 ?

Über cron darf das Script erst gestartet werden, wenn /tmp/started
angelegt ist, das soll ja aber eigentlich erst nach allem anderen da
sein..

Ich habe den Verdacht, dass /tmp/started für configurehood gar nicht
relevant ist. Oder siehst du da irgendwas, was unbedingt vorher da sein
muss?

Tim

> +
> +start()
> +{
> +	sleep 3
> +	/usr/sbin/configurehood
> +}
> -- 
> 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/20180120/e792bf9c/attachment.sig>