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

mail at adrianschmutzler.de mail at adrianschmutzler.de
Sa Feb 10 17:11:07 CET 2018


Hallo Tim,

du glaubst zwar nicht, dass er was kaputt macht, aber wir müssen es prüfen und das kostet Zeit.

Wir können auch einfach /tmp/started lassen und müssen nichts prüfen.

Die vorgeschlagene Änderung hat damit ja erstmal nichts zu tun, sondern ändert nur, wo /tmp/started gesetzt wird. Dabei handelt es sich in meinen Augen um eine Verbesserung ohne funktionale Einschränkung (außer, dass die Blockade durch den fehlenden /tmp/started nun tatsächlich den ganzen Bootprozess anhält).

Das finde ich besser und richtig. Daher der Patch.

Grüße

Adrian


> -----Original Message-----
> From: Tim Niemeyer [mailto:tim at tn-x.org]
> Sent: Samstag, 10. Februar 2018 17:06
> To: mail at adrianschmutzler.de; franken-dev at freifunk.net
> Subject: Re: [PATCH v4 1/2] init.d: Set /tmp/started through init.d instead of
> rc.local
> 
> Hi Adrian
> 
> Am Samstag, den 10.02.2018, 16:59 +0100 schrieb
> mail at adrianschmutzler.de:
> > Findest du es nicht etwas übertrieben, jetzt nur weil du /tmp/started
> > loswerden willst, plötzlich den nodewatcher umzubauen?
> Das habe ich nicht gesagt. Ich habe gesagt, dass ich nicht glaube, dass wir den
> Nodewatcher da groß überarbeiten müssen.
> 
> > Weil man den ja im Boot-Prozess so dringend braucht?
> Nein, da musst du meine Mail bitte genauer lesen:
> --- %< ---
> Wenn nodewatcher Dinge kaputt macht, weil das System noch nicht fertig
> gebootet ist, dann ist nodewatcher kaputt und wir müssen das reparieren.
> 
> Ich glaube sogar, dass es kein Problem sein wird.
> --- >% ---
> 
> Achtung der letzte Satz:
> >> Ich glaube sogar, dass es _kein Problem_ sein wird. <<
> 
> > Insbesondere, da /tmp/started nichts stört und für nichts ein Problem
> > darstellt.
> Es stört nicht, aber du hast ein Patch mit größeren Änderungen auf der Basis
> geschrieben. Das ist _meiner Meinung_ nach aber die falsche Richtung.
> 
> Tim
> 
> >
> > > -----Original Message-----
> > > From: Tim Niemeyer [mailto:tim at tn-x.org]
> > > Sent: Samstag, 10. Februar 2018 16:56
> > > To: mail at adrianschmutzler.de; franken-dev at freifunk.net
> > > Subject: Re: [PATCH v4 1/2] init.d: Set /tmp/started through init.d
> > > instead of rc.local
> > >
> > > Am Samstag, den 10.02.2018, 16:50 +0100 schrieb
> > > mail at adrianschmutzler.de:
> > > > Hallo,
> > > >
> > > > den Kommentar zum nodewatcher verstehe ich nicht, aber da
> brauchen
> > >
> > > wir
> > > > das File im Moment.
> > >
> > > Ich stelle in Frage, dass wir das brauchen.
> > >
> > > Wenn nodewatcher Dinge kaputt macht, weil das System noch nicht
> > > fertig gebootet ist, dann ist nodewatcher kaputt und wir müssen das
> > > reparieren.
> > >
> > > Ich glaube sogar, dass es kein Problem sein wird.
> > >
> > > Tim
> > >
> > > >
> > > > Grüße
> > > >
> > > > Adrian
> > > >
> > > > > -----Original Message-----
> > > > > From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On
> > > > > Behalf Of Tim Niemeyer
> > > > > Sent: Samstag, 10. Februar 2018 15:12
> > > > > To: Adrian Schmutzler <freifunk at adrianschmutzler.de>; franken-
> > > > > dev at freifunk.net
> > > > > Subject: Re: [PATCH v4 1/2] init.d: Set /tmp/started through
> > > > > init.d instead of rc.local
> > > > >
> > > > > Am Sonntag, den 21.01.2018, 22:55 +0100 schrieb Adrian
> > > > > Schmutzler:
> > > > > > This ensures that /tmp/started is actually set after the end
> > > > > > of the boot process.
> > > > >
> > > > > Ich frag mich immer noch, warum wir das noch brauchen.
> > > > >
> > > > > Bei configurehood muss sichergestellt werden, dass das Script
> > > > > nicht mehrfach ausgeführt wird, dafür ist das hier nicht
> > > > > hilfreich.
> > > > >
> > > > > Beim Nodewatcher ist eh alles fragwürdig.
> > > > >
> > > > > Tim
> > > > >
> > > > > >
> > > > > > Signed-off-by: Christian Dresel <fff at chrisi01.de>
> > > > > > Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.d
> > > > > > e>
> > > > > > Tested-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> > > > > >
> > > > > > ---
> > > > > >
> > > > > > Changes in v2/v3:
> > > > > > none
> > > > > >
> > > > > > Changes in v4:
> > > > > > - Moved file to its own package and set dependencies
> > > > > > ---
> > > > > >  bsp/default/root_file_system/etc/rc.local          |  2 --
> > > > > >  src/packages/fff/fff-hoods/Makefile                |  2 +-
> > > > > >  src/packages/fff/fff-nodewatcher/Makefile          |  2 +-
> > > > > >  src/packages/fff/fff-started/Makefile              | 39
> > > > > > ++++++++++++++++++++++
> > > > > >  .../fff/fff-started/files/etc/init.d/zdone         | 11
> > > > > > ++++++
> > > > > >  5 files changed, 52 insertions(+), 4 deletions(-)
> > > > > >  create mode 100644 src/packages/fff/fff-started/Makefile
> > > > > >  create mode 100755 src/packages/fff/fff-
> > > > > > started/files/etc/init.d/zdone
> > > > > >
> > > > > > 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
> > > > > > diff --git a/src/packages/fff/fff-hoods/Makefile
> > > > > > b/src/packages/fff/fff-hoods/Makefile
> > > > > > index 11ab6d1..2081e62 100644
> > > > > > --- a/src/packages/fff/fff-hoods/Makefile
> > > > > > +++ b/src/packages/fff/fff-hoods/Makefile
> > > > > > @@ -13,7 +13,7 @@ define Package/$(PKG_NAME)
> > > > > >      CATEGORY:=Freifunk
> > > > > >      TITLE:= Freifunk-Franken hoods
> > > > > >      URL:=http://www.freifunk-franken.de
> > > > > > -    DEPENDS:=+fff-network +fff-config
> > > > > > +    DEPENDS:=+fff-network +fff-config +fff-started
> > > > > >  endef
> > > > > >
> > > > > >  define Package/$(PKG_NAME)/description diff --git
> > > > > > a/src/packages/fff/fff-nodewatcher/Makefile
> > > > > > b/src/packages/fff/fff-nodewatcher/Makefile
> > > > > > index 633ec71..11a0cb6 100644
> > > > > > --- a/src/packages/fff/fff-nodewatcher/Makefile
> > > > > > +++ b/src/packages/fff/fff-nodewatcher/Makefile
> > > > > > @@ -13,7 +13,7 @@ define Package/$(PKG_NAME)
> > > > > >      CATEGORY:=Freifunk
> > > > > >      TITLE:= Freifunk-Franken Nodewatcher
> > > > > >      URL:=http://www.freifunk-franken.de
> > > > > > -    DEPENDS:=+fff-alfred +micrond +fff-config +fff-random
> > > > > > +ip
> > > > > > +ip-
> > > > > > bridge
> > > > > > +    DEPENDS:=+fff-alfred +micrond +fff-config +fff-random ip
> > > > > > +ip-
> > > > > > bridge +fff-started
> > > > > >  endef
> > > > > >
> > > > > >  define Package/$(PKG_NAME)/description diff --git
> > > > > > a/src/packages/fff/fff-started/Makefile
> > > > > > b/src/packages/fff/fff-started/Makefile
> > > > > > new file mode 100644
> > > > > > index 0000000..39e5e12
> > > > > > --- /dev/null
> > > > > > +++ b/src/packages/fff/fff-started/Makefile
> > > > > > @@ -0,0 +1,39 @@
> > > > > > +include $(TOPDIR)/rules.mk
> > > > > > +
> > > > > > +PKG_NAME:=fff-started
> > > > > > +PKG_VERSION:=1
> > > > > > +PKG_RELEASE:=1
> > > > > > +
> > > > > > +PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME)
> > > > > > +
> > > > > > +include $(INCLUDE_DIR)/package.mk
> > > > > > +
> > > > > > +define Package/$(PKG_NAME)
> > > > > > +    SECTION:=base
> > > > > > +    CATEGORY:=Freifunk
> > > > > > +    TITLE:=Freifunk-Franken started
> > > > > > +    URL:=http://www.freifunk-franken.de
> > > > > > +endef
> > > > > > +
> > > > > > +define Package/$(PKG_NAME)/description
> > > > > > +    This is the Freifunk Franken Firmware started package.
> > > > > > +    This packages provides the /tmp/started file.
> > > > > > +endef
> > > > > > +
> > > > > > +define Build/Prepare
> > > > > > +    echo "all: " > $(PKG_BUILD_DIR)/Makefile endef
> > > > > > +
> > > > > > +define Build/Configure
> > > > > > +    # nothing
> > > > > > +endef
> > > > > > +
> > > > > > +define Build/Compile
> > > > > > +    # nothing
> > > > > > +endef
> > > > > > +
> > > > > > +define Package/$(PKG_NAME)/install
> > > > > > +    $(CP) ./files/* $(1)/
> > > > > > +endef
> > > > > > +
> > > > > > +$(eval $(call BuildPackage,$(PKG_NAME)))
> > > > > > diff --git a/src/packages/fff/fff-
> > > > > > started/files/etc/init.d/zdone
> > > > > > b/src/packages/fff/fff-started/files/etc/init.d/zdone
> > > > > > new file mode 100755
> > > > > > index 0000000..f4aaf02
> > > > > > --- /dev/null
> > > > > > +++ b/src/packages/fff/fff-started/files/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
> > > > > > +}
> > > > > > --
> > > > > > 2.7.4
> > > > > >
> > > >
> > > >
> >
> >



Mehr Informationen über die Mailingliste franken-dev