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

[mail at adrianschmutzler.de]

Hallo Tim,

> -----Original Message-----
> From: Tim Niemeyer [mailto:tim at tn-x.org]
> Sent: Samstag, 20. Januar 2018 15:46
> To: Adrian Schmutzler <freifunk at adrianschmutzler.de>; franken-
> dev at freifunk.net
> Subject: Re: [PATCH v3 2/3] init.d/fff-hoods: Move call of configurehood to
> init.d script
> 
> 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.

Im Moment hat _niemand_ den Hauch einer Idee, wie wir br-mesh zwingen, da zu sein.
Ich selbst habe mich sehr lange damit auseinandergesetzt, dass Problem einzuengen.
Mit und ohne diesen Patch wird configurehood VOR alfred ausgeführt, also ist es insofern egal.
Der Unterschied ist, dass mit diesem Patch die LEDS (S96...) schon richtig an sind, wenn das configurehood wegen des random ewig braucht.

Ein Grund, das configurehood nach Alfred zu starten ist mir nicht bekannt.

> 
> > 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.

Das hilft genau dann, wenn kein Kabel verbunden ist und ein configap da.

> 
> > 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 ?

Nach den LEDs (96), vor Alfred (99); Begründung siehe oben.
97 ginge auch.

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

Genau, wenn du also 1/3 nicht magst, den hier aber schon, dann ist /tmp/started in der rc.local (S95) und configurehood später und dann geht’s kaputt.

> 
> 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?

/tmp/started steht explizit im cron und hat NUR den simplen Zweck, ein gleichzeitiges Ausführen von configurehood durch init.d und cron zu verhindern. Das ließ sich so am einfachsten lösen, da man keine eigene Datei anlegen muss.

Abschließend: Ich bin solange NICHT der Meinung, dass das br-mesh immer an sein muss, bis ich eine Lösung dafür sehe, die sinnvoll ist. Im Moment haben wir noch nicht mal eine grobe Idee davon. Deshalb weiß ICH auch nicht, ob ich so eine Lösung dann will.

Beste Grüße

Adrian

> 
> Tim
> 
> > +
> > +start()
> > +{
> > +	sleep 3
> > +	/usr/sbin/configurehood
> > +}
> > --
> > 2.7.4
> >