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

[mail at adrianschmutzler.de]

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 at tn-x.org]
> Sent: Samstag, 20. Januar 2018 16:24
> To: mail 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
> 
> 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
> > > >
> >
> >