[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