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

mail at adrianschmutzler.de mail at adrianschmutzler.de
So Apr 19 15:54:57 CEST 2020


Hallo,

> > Ich kriege frühestens am Dienstag ein neues Netzteil, und dann werde ich erstmal kucken, was an meinem Server noch geht. Bis dahin werde ich nichts ändern, schicken oder applien.
> > Falls du ein Release machen willst, also bitte selbst einwerfen. 
> Ouh, das klingt ja fies. 
> Wie gesagt: An diesem Dings würde ich jetzt nichts mehr ändern, das kann keiner nachvollziehen. Zumal die meisten meiner Anmerkungen eh neue "Features" sind, und daher in einem eigenen Patch eh besser aufgehoben sind.
> Ich würde diese Wünsche aber gern noch vor der Beta drin haben, daher warte ich gerne auf deinen Server (außer du möchtest ohne Server trotzdem reviewen).

Mein Desktop funktioniert, ich komme halt nur an mein repo nicht ohne Weiteres ran. Ich werde mir die neuen Patches gleich anschauen.

> > Den Timeout mochte ich nicht, da es ja effektiv kein Timeout war, sondern eine fixe Wartezeit für jedes Subskript. Ich halte nichts davon, für jedes Subskript zwei Minuten zu warten, sodass dann der ganze nodewatcher 20-30 Sekunden dauert (im Moment). Und beim Exitstatus wurde es dann noch kompliziert.
> > So oder so waren aber ja der Exitstatus und der "Timeout" im Prinzip Zusatzfeatures, während meine Version aus einem Skript eine Aneinanderreihung von vielen Einzelskripten macht.
> > Ich würde das also separat diskutieren, aber es ist in meinen Augen keine Notwendigkeit für die Aufteilung an sich. 
> Was? Ne. 
> Der Timeout sollte eigentlich tatsächlich als Timeout funktioniert haben, getestet hab ichs aber nie.. 

Der Timeout funktioniert deshalb nicht, weil wir hinterher die Ausgabe abfangen mit $(). Sonst würde er funktionieren.

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/20200419/7b65e72b/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev