[v3,1/3] init.d: Set /tmp/started through init.d instead of rc.local

Submitted by Adrian Schmutzler on Jan. 4, 2018, 11:52 p.m.

Details

Message ID 1515109973-56285-1-git-send-email-freifunk@adrianschmutzler.de
State Changes Requested
Headers show

Commit Message

Adrian Schmutzler Jan. 4, 2018, 11:52 p.m.
This ensures that /tmp/started is actually set after the end
of the boot process.

Signed-off-by: Christian Dresel <fff@chrisi01.de>
Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
Tested-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>

---

Changes in v2/v3:
none
---
 bsp/default/root_file_system/etc/init.d/zdone | 11 +++++++++++
 bsp/default/root_file_system/etc/rc.local     |  2 --
 2 files changed, 11 insertions(+), 2 deletions(-)
 create mode 100755 bsp/default/root_file_system/etc/init.d/zdone

Patch hide | download patch | download mbox

diff --git a/bsp/default/root_file_system/etc/init.d/zdone b/bsp/default/root_file_system/etc/init.d/zdone
new file mode 100755
index 0000000..f4aaf02
--- /dev/null
+++ b/bsp/default/root_file_system/etc/init.d/zdone
@@ -0,0 +1,11 @@ 
+#!/bin/sh /etc/rc.common
+
+# Copyright 2017 Adrian Schmutzler
+# License GPLv3
+
+START=99
+
+start()
+{
+	touch /tmp/started
+}
diff --git a/bsp/default/root_file_system/etc/rc.local b/bsp/default/root_file_system/etc/rc.local
index 59042c3..7410bae 100755
--- a/bsp/default/root_file_system/etc/rc.local
+++ b/bsp/default/root_file_system/etc/rc.local
@@ -8,6 +8,4 @@  sleep 3
 
 /usr/sbin/configurehood
 
-touch /tmp/started
-
 exit 0

Comments

Tim Niemeyer Jan. 20, 2018, 2:38 p.m.
Hi

Am Freitag, den 05.01.2018, 00:52 +0100 schrieb Adrian Schmutzler:
> This ensures that /tmp/started is actually set after the end
> of the boot process.
> 
> Signed-off-by: Christian Dresel <fff@chrisi01.de>
> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> Tested-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> 
> ---
> 
> Changes in v2/v3:
> none
> ---
>  bsp/default/root_file_system/etc/init.d/zdone | 11 +++++++++++
Sry, wir hatten immer gesagt, dass wir diesen Part von Zeit zu Zeit
komplett auflösen wollen.

>  bsp/default/root_file_system/etc/rc.local     |  2 --
>  2 files changed, 11 insertions(+), 2 deletions(-)
>  create mode 100755 bsp/default/root_file_system/etc/init.d/zdone
> 
> diff --git a/bsp/default/root_file_system/etc/init.d/zdone
> b/bsp/default/root_file_system/etc/init.d/zdone
> new file mode 100755
> index 0000000..f4aaf02
> --- /dev/null
> +++ b/bsp/default/root_file_system/etc/init.d/zdone
> @@ -0,0 +1,11 @@
> +#!/bin/sh /etc/rc.common
> +
> +# Copyright 2017 Adrian Schmutzler
> +# License GPLv3
> +
> +START=99
> +
> +start()
> +{
> +	touch /tmp/started
> +}
Gefällt mir auch sehr gut.

> diff --git a/bsp/default/root_file_system/etc/rc.local
> b/bsp/default/root_file_system/etc/rc.local
> index 59042c3..7410bae 100755
> --- a/bsp/default/root_file_system/etc/rc.local
> +++ b/bsp/default/root_file_system/etc/rc.local
> @@ -8,6 +8,4 @@ sleep 3
>  
>  /usr/sbin/configurehood
>  
> -touch /tmp/started
Es ist gut hier wieder etwas loszuwerden.

Trotzdem, nein. Die zdone Datei gehört in ein Package! Und auch nicht
erst in einem späteren Patch, sondern direkt.

In welches Package müsste man noch überlegen. Aktuell wird es ja von
fff-hoods und von fff-nodewatcher verwendet. In Patch 3/3 siehst du vor
das in fff-config zu werfen. Irgendwie wiederstrebt es mir. Eigentlich
ist das /tmp/started ja komplett unabhängig von der zentralen config
Datei.

Ich glaube, wenn wir das /tmp/started nicht am besten eh ganz los
werden können, dann sollte es in ein eigenes Package gepackt werden.

Tim

> -
>  exit 0
> -- 
> 2.7.4
>
Adrian Schmutzler Jan. 20, 2018, 3:16 p.m.
Hallo,

genau weil wir noch ewig streiten, in welche Package das gehört, muss erstmal das Problem behoben werden und dann kann man sich über den Rest streiten.

Im Moment besteht die (recht kleine) Gefahr, dass es kaputt geht.

Grüße

Adrian

> -----Original Message-----
> From: Tim Niemeyer [mailto:tim@tn-x.org]
> Sent: Samstag, 20. Januar 2018 15:39
> To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> dev@freifunk.net
> Subject: Re: [PATCH v3 1/3] init.d: Set /tmp/started through init.d instead of
> rc.local
> 
> Hi
> 
> Am Freitag, den 05.01.2018, 00:52 +0100 schrieb Adrian Schmutzler:
> > This ensures that /tmp/started is actually set after the end of the
> > boot process.
> >
> > Signed-off-by: Christian Dresel <fff@chrisi01.de>
> > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> > Tested-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> >
> > ---
> >
> > Changes in v2/v3:
> > none
> > ---
> >  bsp/default/root_file_system/etc/init.d/zdone | 11 +++++++++++
> Sry, wir hatten immer gesagt, dass wir diesen Part von Zeit zu Zeit komplett
> auflösen wollen.
> 
> >  bsp/default/root_file_system/etc/rc.local     |  2 --
> >  2 files changed, 11 insertions(+), 2 deletions(-)
> >  create mode 100755 bsp/default/root_file_system/etc/init.d/zdone
> >
> > diff --git a/bsp/default/root_file_system/etc/init.d/zdone
> > b/bsp/default/root_file_system/etc/init.d/zdone
> > new file mode 100755
> > index 0000000..f4aaf02
> > --- /dev/null
> > +++ b/bsp/default/root_file_system/etc/init.d/zdone
> > @@ -0,0 +1,11 @@
> > +#!/bin/sh /etc/rc.common
> > +
> > +# Copyright 2017 Adrian Schmutzler
> > +# License GPLv3
> > +
> > +START=99
> > +
> > +start()
> > +{
> > +	touch /tmp/started
> > +}
> Gefällt mir auch sehr gut.
> 
> > diff --git a/bsp/default/root_file_system/etc/rc.local
> > b/bsp/default/root_file_system/etc/rc.local
> > index 59042c3..7410bae 100755
> > --- a/bsp/default/root_file_system/etc/rc.local
> > +++ b/bsp/default/root_file_system/etc/rc.local
> > @@ -8,6 +8,4 @@ sleep 3
> >
> >  /usr/sbin/configurehood
> >
> > -touch /tmp/started
> Es ist gut hier wieder etwas loszuwerden.
> 
> Trotzdem, nein. Die zdone Datei gehört in ein Package! Und auch nicht erst in
> einem späteren Patch, sondern direkt.
> 
> In welches Package müsste man noch überlegen. Aktuell wird es ja von fff-
> hoods und von fff-nodewatcher verwendet. In Patch 3/3 siehst du vor das in
> fff-config zu werfen. Irgendwie wiederstrebt es mir. Eigentlich ist das
> /tmp/started ja komplett unabhängig von der zentralen config Datei.
> 
> Ich glaube, wenn wir das /tmp/started nicht am besten eh ganz los werden
> können, dann sollte es in ein eigenes Package gepackt werden.
> 
> Tim
> 
> > -
> >  exit 0
> > --
> > 2.7.4
> >
Tim Niemeyer Jan. 20, 2018, 3:23 p.m.
Hi

Auf die Gefahr hin, mich zu wiederholen..

Am Samstag, den 20.01.2018, 16:16 +0100 schrieb mail@adrianschmutzler.de:
> Hallo,
> 
> genau weil wir noch ewig streiten, in welche Package das gehört, muss erstmal das Problem behoben werden und dann kann man sich über den Rest streiten.
Trotzdem, nein. Die zdone Datei gehört in ein Package! Und auch nicht
erst in einem späteren Patch, sondern direkt.

> Im Moment besteht die (recht kleine) Gefahr, dass es kaputt geht.
Ich glaube, wenn wir das /tmp/started nicht am besten eh ganz los
werden können, dann sollte es in ein eigenes Package gepackt werden.

In einem eigenen Package geht das nicht kaputt und kann nach belieben
mittels Abhängig verwendet werden.

Tim

> Grüße
> 
> Adrian
> 
> > -----Original Message-----
> > > > From: Tim Niemeyer [mailto:tim@tn-x.org]
> > Sent: Samstag, 20. Januar 2018 15:39
> > > > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> > dev@freifunk.net
> > Subject: Re: [PATCH v3 1/3] init.d: Set /tmp/started through init.d instead of
> > rc.local
> > 
> > Hi
> > 
> > Am Freitag, den 05.01.2018, 00:52 +0100 schrieb Adrian Schmutzler:
> > > This ensures that /tmp/started is actually set after the end of the
> > > boot process.
> > > 
> > > > > > Signed-off-by: Christian Dresel <fff@chrisi01.de>
> > > > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> > > > > > Tested-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> > > 
> > > ---
> > > 
> > > Changes in v2/v3:
> > > none
> > > ---
> > >  bsp/default/root_file_system/etc/init.d/zdone | 11 +++++++++++
> > 
> > Sry, wir hatten immer gesagt, dass wir diesen Part von Zeit zu Zeit komplett
> > auflösen wollen.
> > 
> > >  bsp/default/root_file_system/etc/rc.local     |  2 --
> > >  2 files changed, 11 insertions(+), 2 deletions(-)
> > >  create mode 100755 bsp/default/root_file_system/etc/init.d/zdone
> > > 
> > > diff --git a/bsp/default/root_file_system/etc/init.d/zdone
> > > b/bsp/default/root_file_system/etc/init.d/zdone
> > > new file mode 100755
> > > index 0000000..f4aaf02
> > > --- /dev/null
> > > +++ b/bsp/default/root_file_system/etc/init.d/zdone
> > > @@ -0,0 +1,11 @@
> > > +#!/bin/sh /etc/rc.common
> > > +
> > > +# Copyright 2017 Adrian Schmutzler
> > > +# License GPLv3
> > > +
> > > +START=99
> > > +
> > > +start()
> > > +{
> > > > > > +	touch /tmp/started
> > > +}
> > 
> > Gefällt mir auch sehr gut.
> > 
> > > diff --git a/bsp/default/root_file_system/etc/rc.local
> > > b/bsp/default/root_file_system/etc/rc.local
> > > index 59042c3..7410bae 100755
> > > --- a/bsp/default/root_file_system/etc/rc.local
> > > +++ b/bsp/default/root_file_system/etc/rc.local
> > > @@ -8,6 +8,4 @@ sleep 3
> > > 
> > >  /usr/sbin/configurehood
> > > 
> > > -touch /tmp/started
> > 
> > Es ist gut hier wieder etwas loszuwerden.
> > 
> > Trotzdem, nein. Die zdone Datei gehört in ein Package! Und auch nicht erst in
> > einem späteren Patch, sondern direkt.
> > 
> > In welches Package müsste man noch überlegen. Aktuell wird es ja von fff-
> > hoods und von fff-nodewatcher verwendet. In Patch 3/3 siehst du vor das in
> > fff-config zu werfen. Irgendwie wiederstrebt es mir. Eigentlich ist das
> > /tmp/started ja komplett unabhängig von der zentralen config Datei.
> > 
> > Ich glaube, wenn wir das /tmp/started nicht am besten eh ganz los werden
> > können, dann sollte es in ein eigenes Package gepackt werden.
> > 
> > Tim
> > 
> > > -
> > >  exit 0
> > > --
> > > 2.7.4
> > > 
> 
>
Adrian Schmutzler Jan. 20, 2018, 3:44 p.m.
Hallo Tim,

dann sag mir bitte, in welcher Package du es gerne haben möchtest.

Grüße

Adrian


> -----Original Message-----
> From: Tim Niemeyer [mailto:tim@tn-x.org]
> Sent: Samstag, 20. Januar 2018 16:24
> To: mail@adrianschmutzler.de; franken-dev@freifunk.net
> Subject: Re: [PATCH v3 1/3] init.d: Set /tmp/started through init.d instead of
> rc.local
> 
> Hi
> 
> Auf die Gefahr hin, mich zu wiederholen..
> 
> Am Samstag, den 20.01.2018, 16:16 +0100 schrieb mail@adrianschmutzler.de:
> > Hallo,
> >
> > genau weil wir noch ewig streiten, in welche Package das gehört, muss
> erstmal das Problem behoben werden und dann kann man sich über den
> Rest streiten.
> Trotzdem, nein. Die zdone Datei gehört in ein Package! Und auch nicht erst in
> einem späteren Patch, sondern direkt.
> 
> > Im Moment besteht die (recht kleine) Gefahr, dass es kaputt geht.
> Ich glaube, wenn wir das /tmp/started nicht am besten eh ganz los werden
> können, dann sollte es in ein eigenes Package gepackt werden.
> 
> In einem eigenen Package geht das nicht kaputt und kann nach belieben
> mittels Abhängig verwendet werden.
> 
> Tim
> 
> > Grüße
> >
> > Adrian
> >
> > > -----Original Message-----
> > > > > From: Tim Niemeyer [mailto:tim@tn-x.org]
> > > Sent: Samstag, 20. Januar 2018 15:39
> > > > > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> > > dev@freifunk.net
> > > Subject: Re: [PATCH v3 1/3] init.d: Set /tmp/started through init.d
> > > instead of rc.local
> > >
> > > Hi
> > >
> > > Am Freitag, den 05.01.2018, 00:52 +0100 schrieb Adrian Schmutzler:
> > > > This ensures that /tmp/started is actually set after the end of
> > > > the boot process.
> > > >
> > > > > > > Signed-off-by: Christian Dresel <fff@chrisi01.de>
> > > > > > > Signed-off-by: Adrian Schmutzler
> > > > > > > <freifunk@adrianschmutzler.de>
> > > > > > > Tested-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> > > >
> > > > ---
> > > >
> > > > Changes in v2/v3:
> > > > none
> > > > ---
> > > >  bsp/default/root_file_system/etc/init.d/zdone | 11 +++++++++++
> > >
> > > Sry, wir hatten immer gesagt, dass wir diesen Part von Zeit zu Zeit
> > > komplett auflösen wollen.
> > >
> > > >  bsp/default/root_file_system/etc/rc.local     |  2 --
> > > >  2 files changed, 11 insertions(+), 2 deletions(-)
> > > >  create mode 100755 bsp/default/root_file_system/etc/init.d/zdone
> > > >
> > > > diff --git a/bsp/default/root_file_system/etc/init.d/zdone
> > > > b/bsp/default/root_file_system/etc/init.d/zdone
> > > > new file mode 100755
> > > > index 0000000..f4aaf02
> > > > --- /dev/null
> > > > +++ b/bsp/default/root_file_system/etc/init.d/zdone
> > > > @@ -0,0 +1,11 @@
> > > > +#!/bin/sh /etc/rc.common
> > > > +
> > > > +# Copyright 2017 Adrian Schmutzler # License GPLv3
> > > > +
> > > > +START=99
> > > > +
> > > > +start()
> > > > +{
> > > > > > > +	touch /tmp/started
> > > > +}
> > >
> > > Gefällt mir auch sehr gut.
> > >
> > > > diff --git a/bsp/default/root_file_system/etc/rc.local
> > > > b/bsp/default/root_file_system/etc/rc.local
> > > > index 59042c3..7410bae 100755
> > > > --- a/bsp/default/root_file_system/etc/rc.local
> > > > +++ b/bsp/default/root_file_system/etc/rc.local
> > > > @@ -8,6 +8,4 @@ sleep 3
> > > >
> > > >  /usr/sbin/configurehood
> > > >
> > > > -touch /tmp/started
> > >
> > > Es ist gut hier wieder etwas loszuwerden.
> > >
> > > Trotzdem, nein. Die zdone Datei gehört in ein Package! Und auch
> > > nicht erst in einem späteren Patch, sondern direkt.
> > >
> > > In welches Package müsste man noch überlegen. Aktuell wird es ja von
> > > fff- hoods und von fff-nodewatcher verwendet. In Patch 3/3 siehst du
> > > vor das in fff-config zu werfen. Irgendwie wiederstrebt es mir.
> > > Eigentlich ist das /tmp/started ja komplett unabhängig von der zentralen
> config Datei.
> > >
> > > Ich glaube, wenn wir das /tmp/started nicht am besten eh ganz los
> > > werden können, dann sollte es in ein eigenes Package gepackt werden.
> > >
> > > Tim
> > >
> > > > -
> > > >  exit 0
> > > > --
> > > > 2.7.4
> > > >
> >
> >
Tim Niemeyer Jan. 20, 2018, 3:48 p.m.
Am Samstag, den 20.01.2018, 16:44 +0100 schrieb mail@adrianschmutzler.de:
> Hallo Tim,
> 
> dann sag mir bitte, in welcher Package du es gerne haben möchtest.
[..]
> > In einem eigenen Package geht das nicht kaputt und kann nach belieben
> > mittels Abhängig verwendet werden.

Es gehört in ein eigenes Package.

Jeder, der von /tmp/started abhängt muss dann auch von dem Package
abhängen.

Wir haben aber vorhin festgestellt, dass fff-hoods das Zeug gar nicht
benötigt. Und insofern nur noch der Nodewatcher das benötigt. Das wird
deutlich komplexer zu beurteilen, ob der das überhaupt braucht, aber
das sollte man sich mMn vorher angucken.

Tim

> > Tim
> > 
> > > Grüße
> > > 
> > > Adrian
> > > 
> > > > -----Original Message-----
> > > > > > From: Tim Niemeyer [mailto:tim@tn-x.org]
> > > > 
> > > > Sent: Samstag, 20. Januar 2018 15:39
> > > > > > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> > > > 
> > > > dev@freifunk.net
> > > > Subject: Re: [PATCH v3 1/3] init.d: Set /tmp/started through init.d
> > > > instead of rc.local
> > > > 
> > > > Hi
> > > > 
> > > > Am Freitag, den 05.01.2018, 00:52 +0100 schrieb Adrian Schmutzler:
> > > > > This ensures that /tmp/started is actually set after the end of
> > > > > the boot process.
> > > > > 
> > > > > > > > > > > > > > > > Signed-off-by: Christian Dresel <fff@chrisi01.de>
> > > > > > > > Signed-off-by: Adrian Schmutzler
> > > > > > > > > > > > > > > > <freifunk@adrianschmutzler.de>
> > > > > > > > Tested-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> > > > > 
> > > > > ---
> > > > > 
> > > > > Changes in v2/v3:
> > > > > none
> > > > > ---
> > > > >  bsp/default/root_file_system/etc/init.d/zdone | 11 +++++++++++
> > > > 
> > > > Sry, wir hatten immer gesagt, dass wir diesen Part von Zeit zu Zeit
> > > > komplett auflösen wollen.
> > > > 
> > > > >  bsp/default/root_file_system/etc/rc.local     |  2 --
> > > > >  2 files changed, 11 insertions(+), 2 deletions(-)
> > > > >  create mode 100755 bsp/default/root_file_system/etc/init.d/zdone
> > > > > 
> > > > > diff --git a/bsp/default/root_file_system/etc/init.d/zdone
> > > > > b/bsp/default/root_file_system/etc/init.d/zdone
> > > > > new file mode 100755
> > > > > index 0000000..f4aaf02
> > > > > --- /dev/null
> > > > > +++ b/bsp/default/root_file_system/etc/init.d/zdone
> > > > > @@ -0,0 +1,11 @@
> > > > > +#!/bin/sh /etc/rc.common
> > > > > +
> > > > > +# Copyright 2017 Adrian Schmutzler # License GPLv3
> > > > > +
> > > > > +START=99
> > > > > +
> > > > > +start()
> > > > > +{
> > > > > > > > +	touch /tmp/started
> > > > > 
> > > > > +}
> > > > 
> > > > Gefällt mir auch sehr gut.
> > > > 
> > > > > diff --git a/bsp/default/root_file_system/etc/rc.local
> > > > > b/bsp/default/root_file_system/etc/rc.local
> > > > > index 59042c3..7410bae 100755
> > > > > --- a/bsp/default/root_file_system/etc/rc.local
> > > > > +++ b/bsp/default/root_file_system/etc/rc.local
> > > > > @@ -8,6 +8,4 @@ sleep 3
> > > > > 
> > > > >  /usr/sbin/configurehood
> > > > > 
> > > > > -touch /tmp/started
> > > > 
> > > > Es ist gut hier wieder etwas loszuwerden.
> > > > 
> > > > Trotzdem, nein. Die zdone Datei gehört in ein Package! Und auch
> > > > nicht erst in einem späteren Patch, sondern direkt.
> > > > 
> > > > In welches Package müsste man noch überlegen. Aktuell wird es ja von
> > > > fff- hoods und von fff-nodewatcher verwendet. In Patch 3/3 siehst du
> > > > vor das in fff-config zu werfen. Irgendwie wiederstrebt es mir.
> > > > Eigentlich ist das /tmp/started ja komplett unabhängig von der zentralen
> > 
> > config Datei.
> > > > 
> > > > Ich glaube, wenn wir das /tmp/started nicht am besten eh ganz los
> > > > werden können, dann sollte es in ein eigenes Package gepackt werden.
> > > > 
> > > > Tim
> > > > 
> > > > > -
> > > > >  exit 0
> > > > > --
> > > > > 2.7.4
> > > > > 
> > > 
> > > 
> 
>
Robert Langhammer Sept. 10, 2018, 8:52 a.m.
Hallo,

ich wollte das mal wieder aufgreifen. Das ganze ist echt verzwickt. Für
mich stellt es sich so dar:

configurehood darf nicht via cron vor/während configurenetwork und auch
nicht doppelt laufen.

Der Nodewatcher _sollte_ nach configurehood und configurenetwork laufen.

/tmp/started ist unschön. Blöde Abhängigkeit.

Mein Vorschlag wäre:

configurenetwork als init.d boot() in der Nähe von S20network
unterzubringen. Da würde es doch auch hin passen.

Nodewatcher ist von alfred abhängig. Also kann man prüfen ob alfred
schon an ist.

Doppeltes Anlaufen von configurehood verhindern.

=> /tmp/started weg.

Kann das funktionieren? Wenn Ihr meint das geht, würde ich mir mal die
Zeit nehmen und das hin basteln.

Robert






Am 20.01.2018 um 16:48 schrieb Tim Niemeyer:
> Am Samstag, den 20.01.2018, 16:44 +0100 schrieb mail@adrianschmutzler.de:
>> Hallo Tim,
>>
>> dann sag mir bitte, in welcher Package du es gerne haben möchtest.
> [..]
>>> In einem eigenen Package geht das nicht kaputt und kann nach belieben
>>> mittels Abhängig verwendet werden.
> Es gehört in ein eigenes Package.
>
> Jeder, der von /tmp/started abhängt muss dann auch von dem Package
> abhängen.
>
> Wir haben aber vorhin festgestellt, dass fff-hoods das Zeug gar nicht
> benötigt. Und insofern nur noch der Nodewatcher das benötigt. Das wird
> deutlich komplexer zu beurteilen, ob der das überhaupt braucht, aber
> das sollte man sich mMn vorher angucken.
>
> Tim
>
>>> Tim
>>>
>>>> Grüße
>>>>
>>>> Adrian
>>>>
>>>>> -----Original Message-----
>>>>>>> From: Tim Niemeyer [mailto:tim@tn-x.org]
>>>>> Sent: Samstag, 20. Januar 2018 15:39
>>>>>>> To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
>>>>> dev@freifunk.net
>>>>> Subject: Re: [PATCH v3 1/3] init.d: Set /tmp/started through init.d
>>>>> instead of rc.local
>>>>>
>>>>> Hi
>>>>>
>>>>> Am Freitag, den 05.01.2018, 00:52 +0100 schrieb Adrian Schmutzler:
>>>>>> This ensures that /tmp/started is actually set after the end of
>>>>>> the boot process.
>>>>>>
>>>>>>>>>>>>>>>>> Signed-off-by: Christian Dresel <fff@chrisi01.de>
>>>>>>>>> Signed-off-by: Adrian Schmutzler
>>>>>>>>>>>>>>>>> <freifunk@adrianschmutzler.de>
>>>>>>>>> Tested-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
>>>>>> ---
>>>>>>
>>>>>> Changes in v2/v3:
>>>>>> none
>>>>>> ---
>>>>>>  bsp/default/root_file_system/etc/init.d/zdone | 11 +++++++++++
>>>>> Sry, wir hatten immer gesagt, dass wir diesen Part von Zeit zu Zeit
>>>>> komplett auflösen wollen.
>>>>>
>>>>>>  bsp/default/root_file_system/etc/rc.local     |  2 --
>>>>>>  2 files changed, 11 insertions(+), 2 deletions(-)
>>>>>>  create mode 100755 bsp/default/root_file_system/etc/init.d/zdone
>>>>>>
>>>>>> diff --git a/bsp/default/root_file_system/etc/init.d/zdone
>>>>>> b/bsp/default/root_file_system/etc/init.d/zdone
>>>>>> new file mode 100755
>>>>>> index 0000000..f4aaf02
>>>>>> --- /dev/null
>>>>>> +++ b/bsp/default/root_file_system/etc/init.d/zdone
>>>>>> @@ -0,0 +1,11 @@
>>>>>> +#!/bin/sh /etc/rc.common
>>>>>> +
>>>>>> +# Copyright 2017 Adrian Schmutzler # License GPLv3
>>>>>> +
>>>>>> +START=99
>>>>>> +
>>>>>> +start()
>>>>>> +{
>>>>>>>>> +	touch /tmp/started
>>>>>> +}
>>>>> Gefällt mir auch sehr gut.
>>>>>
>>>>>> diff --git a/bsp/default/root_file_system/etc/rc.local
>>>>>> b/bsp/default/root_file_system/etc/rc.local
>>>>>> index 59042c3..7410bae 100755
>>>>>> --- a/bsp/default/root_file_system/etc/rc.local
>>>>>> +++ b/bsp/default/root_file_system/etc/rc.local
>>>>>> @@ -8,6 +8,4 @@ sleep 3
>>>>>>
>>>>>>  /usr/sbin/configurehood
>>>>>>
>>>>>> -touch /tmp/started
>>>>> Es ist gut hier wieder etwas loszuwerden.
>>>>>
>>>>> Trotzdem, nein. Die zdone Datei gehört in ein Package! Und auch
>>>>> nicht erst in einem späteren Patch, sondern direkt.
>>>>>
>>>>> In welches Package müsste man noch überlegen. Aktuell wird es ja von
>>>>> fff- hoods und von fff-nodewatcher verwendet. In Patch 3/3 siehst du
>>>>> vor das in fff-config zu werfen. Irgendwie wiederstrebt es mir.
>>>>> Eigentlich ist das /tmp/started ja komplett unabhängig von der zentralen
>>> config Datei.
>>>>> Ich glaube, wenn wir das /tmp/started nicht am besten eh ganz los
>>>>> werden können, dann sollte es in ein eigenes Package gepackt werden.
>>>>>
>>>>> Tim
>>>>>
>>>>>> -
>>>>>>  exit 0
>>>>>> --
>>>>>> 2.7.4
>>>>>>
>>>>
>>
>>