[PATCH 1/2] nodewatcher: split up
robert
rlanghammer at web.de
Di Okt 8 23:48:00 CEST 2019
Hi Adrian,
Am 08.10.19 um 01:29 schrieb mail at adrianschmutzler.de:
> Hallo Robert,
>
>>> + for f in /usr/lib/nodewatcher.d/*.sh; do
>>> + tmp=$(run_with_timeout "$f")
>>> + if [ $? ]; then
>>> + DATA=$DATA"$tmp"
>> [ $? ] ist IMMER true, da immer exsistent! Ich denke da wurde nur der
>> Test auf 0 vergessen: if [ $? = 0 ]
>> In $? ist in diesem Fall der Exitstatus der Subshell.
> Da müsste man dann überlegen, was den Exitstatus der Subshell determiniert.
>
> Klingt für mich sehr danach, als ob man einfach [ -n "$tmp" ] prüfen sollte.
Nein, das ist schon richtig so. Wenn der Timer zuschlägt und das Skript
noch nicht fertig ist, steht zerhacktes in $tmp. Wir bekommen dann
$?=143 (SIGTERM) und man kann das Ganze mit if [ $? = 0 ] verwerfen.
Wichtig ist auch vernünftiges Fehlerhandling in den Skripten. Ein exit 1
landet hier in $?.
@Tim: sehe ich das richtig so?
Viele Grüße
Robert
>
> Grüße
>
> Adrian
-------------- 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/20191008/5a735422/attachment.sig>
Mehr Informationen über die Mailingliste franken-dev