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

[mail at adrianschmutzler.de]

Hallo,

 

die Frage ist, ob configurenetwork bei S20 schon alles hat, was es braucht.

 

Grüße

 

Adrian

 

From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf Of robert
Sent: Montag, 10. September 2018 10:52
To: franken-dev at freifunk.net
Subject: Re: [PATCH v3 1/3] init.d: Set /tmp/started through init.d instead of rc.local

 

Hallo,

ich wollte das mal wieder aufgreifen. Das ganze ist echt verzwickt. Für mich stellt es sich so dar:

configurehood darf nicht via cron vor/während configurenetwork und auch nicht doppelt laufen.

Der Nodewatcher _sollte_ nach configurehood und configurenetwork laufen.

/tmp/started ist unschön. Blöde Abhängigkeit.

Mein Vorschlag wäre: 

configurenetwork als init.d boot() in der Nähe von S20network unterzubringen. Da würde es doch auch hin passen.

Nodewatcher ist von alfred abhängig. Also kann man prüfen ob alfred schon an ist.

Doppeltes Anlaufen von configurehood verhindern.

=> /tmp/started weg.

Kann das funktionieren? Wenn Ihr meint das geht, würde ich mir mal die Zeit nehmen und das hin basteln.

Robert

 

 

 

 

 

Am 20.01.2018 um 16:48 schrieb Tim Niemeyer:

Am Samstag, den 20.01.2018, 16:44 +0100 schrieb mail at adrianschmutzler.de <mailto:mail at adrianschmutzler.de> :

Hallo Tim,
 
dann sag mir bitte, in welcher Package du es gerne haben möchtest.

[..]

In einem eigenen Package geht das nicht kaputt und kann nach belieben
mittels Abhängig verwendet werden.

 
Es gehört in ein eigenes Package.
 
Jeder, der von /tmp/started abhängt muss dann auch von dem Package
abhängen.
 
Wir haben aber vorhin festgestellt, dass fff-hoods das Zeug gar nicht
benötigt. Und insofern nur noch der Nodewatcher das benötigt. Das wird
deutlich komplexer zu beurteilen, ob der das überhaupt braucht, aber
das sollte man sich mMn vorher angucken.
 
Tim
 

Tim
 

Grüße
 
Adrian
 

-----Original Message-----

From: Tim Niemeyer [mailto:tim at tn-x.org]

 
Sent: Samstag, 20. Januar 2018 15:39

To: Adrian Schmutzler  <mailto:freifunk at adrianschmutzler.de> <freifunk at adrianschmutzler.de>; franken-

 
dev at freifunk.net <mailto:dev at freifunk.net> 
Subject: Re: [PATCH v3 1/3] init.d: Set /tmp/started through init.d
instead of rc.local
 
Hi
 
Am Freitag, den 05.01.2018, 00:52 +0100 schrieb Adrian Schmutzler:

This ensures that /tmp/started is actually set after the end of
the boot process.
 

Signed-off-by: Christian Dresel  <mailto:fff at chrisi01.de> <fff at chrisi01.de>

Signed-off-by: Adrian Schmutzler

 <mailto:freifunk at adrianschmutzler.de> <freifunk at adrianschmutzler.de>

Tested-by: Adrian Schmutzler  <mailto:freifunk at adrianschmutzler.de> <freifunk at adrianschmutzler.de>

 
---
 
Changes in v2/v3:
none
---
 bsp/default/root_file_system/etc/init.d/zdone | 11 +++++++++++

 
Sry, wir hatten immer gesagt, dass wir diesen Part von Zeit zu Zeit
komplett auflösen wollen.
 

 bsp/default/root_file_system/etc/rc.local     |  2 --
 2 files changed, 11 insertions(+), 2 deletions(-)
 create mode 100755 bsp/default/root_file_system/etc/init.d/zdone
 
diff --git a/bsp/default/root_file_system/etc/init.d/zdone
b/bsp/default/root_file_system/etc/init.d/zdone
new file mode 100755
index 0000000..f4aaf02
--- /dev/null
+++ b/bsp/default/root_file_system/etc/init.d/zdone
@@ -0,0 +1,11 @@
+#!/bin/sh /etc/rc.common
+
+# Copyright 2017 Adrian Schmutzler # License GPLv3
+
+START=99
+
+start()
+{

+      touch /tmp/started

 
+}

 
Gefällt mir auch sehr gut.
 

diff --git a/bsp/default/root_file_system/etc/rc.local
b/bsp/default/root_file_system/etc/rc.local
index 59042c3..7410bae 100755
--- a/bsp/default/root_file_system/etc/rc.local
+++ b/bsp/default/root_file_system/etc/rc.local
@@ -8,6 +8,4 @@ sleep 3
 
 /usr/sbin/configurehood
 
-touch /tmp/started

 
Es ist gut hier wieder etwas loszuwerden.
 
Trotzdem, nein. Die zdone Datei gehört in ein Package! Und auch
nicht erst in einem späteren Patch, sondern direkt.
 
In welches Package müsste man noch überlegen. Aktuell wird es ja von
fff- hoods und von fff-nodewatcher verwendet. In Patch 3/3 siehst du
vor das in fff-config zu werfen. Irgendwie wiederstrebt es mir.
Eigentlich ist das /tmp/started ja komplett unabhängig von der zentralen

 
config Datei.

 
Ich glaube, wenn wir das /tmp/started nicht am besten eh ganz los
werden können, dann sollte es in ein eigenes Package gepackt werden.
 
Tim
 

-
 exit 0
--
2.7.4
 

 
 

 





 

-------------- nächster Teil --------------
Ein Dateianhang mit HTML-Daten wurde abgetrennt...
URL: <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20180910/c53e1c06/attachment.html>