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

[Tim Niemeyer]

Hi

Auf die Gefahr hin, mich zu wiederholen..

Am Samstag, den 20.01.2018, 16:16 +0100 schrieb mail at 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 at tn-x.org]
> > Sent: Samstag, 20. Januar 2018 15:39
> > > > To: Adrian Schmutzler <freifunk at adrianschmutzler.de>; franken-
> > dev at 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 at chrisi01.de>
> > > > > > Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> > > > > > Tested-by: Adrian Schmutzler <freifunk at 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
> > > 
> 
> 
-------------- 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/c3e80350/attachment.sig>