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

robert rlanghammer at web.de
Mo Sep 10 11:47:49 CEST 2018


Hi Adrian,

soweit ich das sehe, ja hat es. Es könnte dann sogar die letzte Zeile
"fff-uradvd restart" raus.

(oh, da ist ja noch das gute alte ifconfig drin :-)

Robert


Am 10.09.2018 um 10:58 schrieb 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 <freifunk at adrianschmutzler.de>
>                             <mailto: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 <fff at chrisi01.de>
>                                                                     <mailto:fff at chrisi01.de>
>
>                                     Signed-off-by: Adrian Schmutzler
>
>                                                                     <freifunk at adrianschmutzler.de>
>                                                                     <mailto:freifunk at adrianschmutzler.de>
>
>                                     Tested-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
>                                     <mailto: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/b017ebc9/attachment.html>


Mehr Informationen über die Mailingliste franken-dev