[PATCH 1/5] nodewatcher: split into nodewatcher.d scripts for individual task

Fabian Bläse fabian at blaese.de
Sa Apr 18 19:22:38 CEST 2020


Hallo Adrian,

Ich hätte $(date) noch mit in die debug Funktion mit rein genommen.
Unbedingt würde ich auch den Exitstatus der Sub-Skripte prüfen und ggf. abbrechen.
Außerdem ist mit deiner Version der timeout weg gefallen, das fand ich eigentlich gar nicht schlecht..

In den Sub-Skripten werden Debugausgaben auf stderr ausgegeben, im Hauptskript in eine Datei geschreiben, deren Länge irgendwie hingepfriemelt wird.
Ich bin hier ja fast dafür, alles auf stderr auszugeben und die Datei fallen zu lassen. Man könnte aber auch stderr der Subskripte mit in die Datei stecken.

Da wir uns an diesem Patch aber noch ewig aufhalten würden, bis jeder absolut zufrieden ist, würde ich diesen Patch so akzeptieren und dann lieber hinterher noch Dinge fixen.

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


Bitte folgendes noch beim applien anpassen:

On 07.01.20 14:40, Adrian Schmutzler wrote:
> Note that since the nodewatcher.d scripts are evaluated by using there
"using their"?

Gruß
Fabian

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: OpenPGP digital signature
URL         : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20200418/a8f0b341/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev