[PATCH 1/2] nodewatcher: split up

[Adrian Schmutzler]

Hallo nochmal,

im Prinzip ist die ganze run_with_timeout Funktion ja ein Zusatzfeature.

Spricht etwas dagegen, erstmal die Aufteilung des nodewatchers umzusetzen und dabei die nodewatcher.d Dateien einfach direkt auszuführen?
Dies würde im Wesentlichen das Gleiche tun wie der nodewatcher jetzt. Dann könnte man den großen Teil des Patches relativ schnell einbauen und würde danach nur noch über die Ausgestaltung der Funktion run_with_timeout diskutieren.

Man würde dann zunächst einfach

tmp=$("$f")
if [ -n "$f" ]; then
   ...

verwenden.

Man müsste dann nur noch:
- klären, ob man 20-interfaces.sh nach network oder wireless tut
- Die Commit-Message schön machen
- Sich bezüglich uci vs. hardcode aus Fabian E-Mail einigen (mMn könnte man das zunächst wie im Patch lassen und danach ggf. wieder in einem separaten Patch uci einbauen/diskutieren)
- Rebasen

@Tim: Hast du Lust, das nochmal zu rebasen/die Commit Message zu schreiben oder soll ich das machen.

Grüße

Adrian

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : openpgp-digital-signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20191010/6c5bebca/attachment.sig>