[PATCH 1/2] nodewatcher: split up

[Adrian Schmutzler]

Hallo Tim,

gefällt mir gut. Ein paar Kommentare:
- Die 20-interfaces.sh enthält auch WiFi Zeug. Man könnte die theoretisch nach fff-wireless schieben, aber so richtig richtig wäre das auch nicht. Aber lieber hat fff-wireless eine Abhängigkeit von fff-network als umgekehrt.
- batman-adv-legacy wurde entfernt, der File fällt weg
- Generell müsste nochmal rebased werden, ein paar kleine Änderungen im nodewatcher fehlen
- Die run_with_timeout Funktion muss ich mir noch mal in Ruhe ankucken. Macht der wirklich pro Skript dann einen sleep 2?
- Eine ausführliche Commit Message wäre in der Tat schön
- Beim Durchschauen sind mir diverse Sachen aufgefallen, die man dringend mal ordentlich machen sollte. Finde es aber gut, dass jetzt erstmal nur Copy/Paste gemacht wurde. Danach kann man ja irgendwann mal aufräumen.
- Auf Fabian Kommentare antworte ich separat.
- Das mit der separaten SCRIPT_VERSION stört mich, v.a. wenn die jetzt noch tiefer versteckt wird. Vll. hat hier jemand noch eine bestechende Idee...

Ich denke, viel mehr lässt sich erstmal nicht rausholen.

Beste 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/20191007/4243770b/attachment.sig>