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

Tim Niemeyer tim at tn-x.org
Sa Feb 10 17:39:40 CET 2018


Moin Adrian

Am Samstag, den 10.02.2018, 17:11 +0100 schrieb mail at adrianschmutzler.de:
> Hallo Tim,
> 
> du glaubst zwar nicht, dass er was kaputt macht, aber wir müssen es
> prüfen und das kostet Zeit.

Du legst ein Package an, damit /tmp/started aus rc.local raus kommt.
Das wiederum machst du nur, damit configurehood nicht mehr im rc.local
gestartet wird sondern _nach_ den LED's.

Also all diese Diskussion für ein paar LED's?

Du willst eine Mini Verbesserung erreichen, in dem du ein super
hässliches /tmp/started vom files Verzeichnis in ein eigenes Package
verschiebst, _obwohl_ wir /tmp/started los werden wollen.

Anders ausgedrückt: Es wird von der einen hässlichen Implementierung
auf eine andere hässliche gewechselt, weil dann LED's früher angehen.

Finde ich nicht fair, zumal wir hier ja bereits eine gute, saubere
mögliche Lösung diskutiert haben.

Tim

> 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 adrianschmutzl
> > > > > > > er.d
> > > > > > > e>
> > > > > > > Tested-by: Adrian Schmutzler <freifunk at adrianschmutzler.d
> > > > > > > e>
> > > > > > > 
> > > > > > > ---
> > > > > > > 
> > > > > > > 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
> > > > > > > 
> > > > > 
> > > > > 
> > > 
> > > 
> 
> 
-------------- 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/20180210/84a3b0ce/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev