[PATCH 1/2] nodewatcher: split up

mail at adrianschmutzler.de mail at adrianschmutzler.de
Di Okt 8 01:29:10 CEST 2019


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.

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/20191008/17e019e2/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev