[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:46:59 CET 2018
Hallo Tim,
> -----Original Message-----
> From: Tim Niemeyer [mailto:tim at tn-x.org]
> Sent: Samstag, 10. Februar 2018 17:40
> 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
>
> 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.
Das ist Patch 2/2. Patch 1/2 ist dazu da, dass /tmp/started NACH dem Bootvergang angelegt wird und nicht mitten in S95.
>
> Also all diese Diskussion für ein paar LED's?
Siehe oben.
>
> 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.
DU willst /tmp/started los werden.
>
> Anders ausgedrückt: Es wird von der einen hässlichen Implementierung auf
> eine andere hässliche gewechselt, weil dann LED's früher angehen.
So hässlich finde ich /tmp/started nicht, eigentlich isses mir ziemlich egal, solange es richtig funktioniert. Die LEDs haben nichts mit /tmp/started zu tun.
>
> Finde ich nicht fair, zumal wir hier ja bereits eine gute, saubere mögliche
> Lösung diskutiert haben.
Nein, haben wir nicht, du hast eine Lösung diskutiert, von der du glaubst, dass sie sauber ist, weil du vermutest, dass der nodewatcher nichts komisches tut.
Grüße
Adrian
>
> 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
> > > > > > > >
> > > > > >
> > > > > >
> > > >
> > > >
> >
> >
Mehr Informationen über die Mailingliste franken-dev