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

Submitted by Adrian Schmutzler on Jan. 21, 2018, 9:55 p.m.

Details

Message ID 1516571720-5210-1-git-send-email-freifunk@adrianschmutzler.de
State Superseded
Headers show

Commit Message

Adrian Schmutzler Jan. 21, 2018, 9:55 p.m.
This ensures that /tmp/started is actually set after the end
of the boot process.

Signed-off-by: Christian Dresel <fff@chrisi01.de>
Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
Tested-by: Adrian Schmutzler <freifunk@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

Patch hide | download patch | download mbox

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
+}

Comments

Fabian Blaese Feb. 2, 2018, 10:18 a.m.
Hallo,

das gefällt mir soweit, dann hat man das mal zusammen, was davon abhängt. Auch wenn mir dieses pseudo-file eigentlich nicht so ganz zusagt, ich weiß aber spontan auch nichts besseres.

> On 21. Jan 2018, at 22:55, Adrian Schmutzler <freifunk@adrianschmutzler.de> wrote:
> 
> +define Package/$(PKG_NAME)/description
> +    This is the Freifunk Franken Firmware started package.
> +    This packages provides the /tmp/started file.

Vielleicht wäre es noch ganz cool hier hinzuschreiben, was dieses started file eigentlich aussagt, damit man das auch ohne ins Paket reinzuschauen rauskriegt..

Fabian
Fabian Blaese Feb. 2, 2018, 10:33 a.m.
Oh, was vergessen...

Reviewed-by: Fabian Bläse <fabian@blaese.de>

Fabian

> On 2. Feb 2018, at 11:18, Fabian Bläse <fabian@blaese.de> wrote:
> 
> Hallo,
> 
> das gefällt mir soweit, dann hat man das mal zusammen, was davon abhängt. Auch wenn mir dieses pseudo-file eigentlich nicht so ganz zusagt, ich weiß aber spontan auch nichts besseres.
> 
>> On 21. Jan 2018, at 22:55, Adrian Schmutzler <freifunk@adrianschmutzler.de> wrote:
>> 
>> +define Package/$(PKG_NAME)/description
>> +    This is the Freifunk Franken Firmware started package.
>> +    This packages provides the /tmp/started file.
> 
> Vielleicht wäre es noch ganz cool hier hinzuschreiben, was dieses started file eigentlich aussagt, damit man das auch ohne ins Paket reinzuschauen rauskriegt..
> 
> Fabian
> --
> franken-dev mailing list
> franken-dev@freifunk.net
> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Adrian Schmutzler Feb. 2, 2018, 10:46 a.m.
Hallo,

wäre schön, wenn das "jemand" beim Applien machen könnte.

"This package provides a file to signal the end of the booting process."

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
> Of Fabian Bläse
> Sent: Freitag, 2. Februar 2018 11:19
> To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> dev@freifunk.net
> Subject: Re: [PATCH v4 1/2] init.d: Set /tmp/started through init.d instead of
> rc.local
> 
> Hallo,
> 
> das gefällt mir soweit, dann hat man das mal zusammen, was davon abhängt.
> Auch wenn mir dieses pseudo-file eigentlich nicht so ganz zusagt, ich weiß
> aber spontan auch nichts besseres.
> 
> > On 21. Jan 2018, at 22:55, Adrian Schmutzler
> <freifunk@adrianschmutzler.de> wrote:
> >
> > +define Package/$(PKG_NAME)/description
> > +    This is the Freifunk Franken Firmware started package.
> > +    This packages provides the /tmp/started file.
> 
> Vielleicht wäre es noch ganz cool hier hinzuschreiben, was dieses started file
> eigentlich aussagt, damit man das auch ohne ins Paket reinzuschauen
> rauskriegt..
> 
> Fabian
Tim Niemeyer Feb. 10, 2018, 2:12 p.m.
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@chrisi01.de>
> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> Tested-by: Adrian Schmutzler <freifunk@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
>
Adrian Schmutzler Feb. 10, 2018, 3:50 p.m.
Hallo,

den Kommentar zum nodewatcher verstehe ich nicht, aber da brauchen wir das File im Moment.

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
> Of Tim Niemeyer
> Sent: Samstag, 10. Februar 2018 15:12
> To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> dev@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@chrisi01.de>
> > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> > Tested-by: Adrian Schmutzler <freifunk@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
> >
Tim Niemeyer Feb. 10, 2018, 3:55 p.m.
Am Samstag, den 10.02.2018, 16:50 +0100 schrieb
mail@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@freifunk.net] On
> > Behalf
> > Of Tim Niemeyer
> > Sent: Samstag, 10. Februar 2018 15:12
> > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> > dev@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@chrisi01.de>
> > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> > > Tested-by: Adrian Schmutzler <freifunk@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
> > > 
> 
>
Adrian Schmutzler Feb. 10, 2018, 3:59 p.m.
Findest du es nicht etwas übertrieben, jetzt nur weil du /tmp/started loswerden willst, plötzlich den nodewatcher umzubauen?

Weil man den ja im Boot-Prozess so dringend braucht?

Insbesondere, da /tmp/started nichts stört und für nichts ein Problem darstellt.


> -----Original Message-----
> From: Tim Niemeyer [mailto:tim@tn-x.org]
> Sent: Samstag, 10. Februar 2018 16:56
> To: mail@adrianschmutzler.de; franken-dev@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@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@freifunk.net] On
> > > Behalf Of Tim Niemeyer
> > > Sent: Samstag, 10. Februar 2018 15:12
> > > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> > > dev@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@chrisi01.de>
> > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> > > > Tested-by: Adrian Schmutzler <freifunk@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
> > > >
> >
> >
Tim Niemeyer Feb. 10, 2018, 4:05 p.m.
Hi Adrian

Am Samstag, den 10.02.2018, 16:59 +0100 schrieb
mail@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@tn-x.org]
> > Sent: Samstag, 10. Februar 2018 16:56
> > To: mail@adrianschmutzler.de; franken-dev@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@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@freifunk.net] On
> > > > Behalf Of Tim Niemeyer
> > > > Sent: Samstag, 10. Februar 2018 15:12
> > > > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> > > > dev@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@chrisi01.de>
> > > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.d
> > > > > e>
> > > > > Tested-by: Adrian Schmutzler <freifunk@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
> > > > > 
> > > 
> > > 
> 
>
Adrian Schmutzler Feb. 10, 2018, 4:11 p.m.
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@tn-x.org]
> Sent: Samstag, 10. Februar 2018 17:06
> To: mail@adrianschmutzler.de; franken-dev@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@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@tn-x.org]
> > > Sent: Samstag, 10. Februar 2018 16:56
> > > To: mail@adrianschmutzler.de; franken-dev@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@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@freifunk.net] On
> > > > > Behalf Of Tim Niemeyer
> > > > > Sent: Samstag, 10. Februar 2018 15:12
> > > > > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-
> > > > > dev@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@chrisi01.de>
> > > > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.d
> > > > > > e>
> > > > > > Tested-by: Adrian Schmutzler <freifunk@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
> > > > > >
> > > >
> > > >
> >
> >
Tim Niemeyer Feb. 10, 2018, 4:39 p.m.
Moin Adrian

Am Samstag, den 10.02.2018, 17:11 +0100 schrieb mail@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@tn-x.org]
> > Sent: Samstag, 10. Februar 2018 17:06
> > To: mail@adrianschmutzler.de; franken-dev@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@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@tn-x.org]
> > > > Sent: Samstag, 10. Februar 2018 16:56
> > > > To: mail@adrianschmutzler.de; franken-dev@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@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@freifunk.net]
> > > > > > On
> > > > > > Behalf Of Tim Niemeyer
> > > > > > Sent: Samstag, 10. Februar 2018 15:12
> > > > > > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>;
> > > > > > franken-
> > > > > > dev@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@chrisi01.de>
> > > > > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzl
> > > > > > > er.d
> > > > > > > e>
> > > > > > > Tested-by: Adrian Schmutzler <freifunk@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
> > > > > > > 
> > > > > 
> > > > > 
> > > 
> > > 
> 
>
Adrian Schmutzler Feb. 10, 2018, 4:46 p.m.
Hallo Tim,

> -----Original Message-----
> From: Tim Niemeyer [mailto:tim@tn-x.org]
> Sent: Samstag, 10. Februar 2018 17:40
> To: mail@adrianschmutzler.de; franken-dev@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@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@tn-x.org]
> > > Sent: Samstag, 10. Februar 2018 17:06
> > > To: mail@adrianschmutzler.de; franken-dev@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@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@tn-x.org]
> > > > > Sent: Samstag, 10. Februar 2018 16:56
> > > > > To: mail@adrianschmutzler.de; franken-dev@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@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@freifunk.net]
> > > > > > > On
> > > > > > > Behalf Of Tim Niemeyer
> > > > > > > Sent: Samstag, 10. Februar 2018 15:12
> > > > > > > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>;
> > > > > > > franken-
> > > > > > > dev@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@chrisi01.de>
> > > > > > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzl
> > > > > > > > er.d
> > > > > > > > e>
> > > > > > > > Tested-by: Adrian Schmutzler <freifunk@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
> > > > > > > >
> > > > > >
> > > > > >
> > > >
> > > >
> >
> >