Prevent multiple start of configurehood and nodewatcher

[Adrian Schmutzler]

Hallo,

 

wie Fabian angeregt hat, sollten wir uns nochmals mit der Frage beschäftigen, ein mehrfaches Ausführen von configurehood zu vermeiden (und meiner Meinung nach auch des nodewatcher).

 

Ich möchte dies gerne von der Debatte um /tmp/started trennen.

 

Vorgeschlagen sind:

 

A)

if [ $( pidof $( basename $0 )  | wc -w ) -gt 2 ]; then

                echo "Only one instance of $0 allowed."

                exit 1

fi

 

B)

lockfile="/var/lock/${0##*/}.lock"

if ! lock -n "$lockfile"; then

                echo "Only one instance of $0 allowed."

                exit 1

fi

trap "lock -u \"$lockfile\"" INT TERM EXIT

 

 

Ich war so frei, und habe bei A das echo und den exit code ergänzt.

 

Ich würde mich über Rückmeldung/Einschätzungen/Abstimmungen freuen und würde dann je nach Präferenz JEWEILS einen Patch für configurehood und nodewatcher mit der favorisierten Variante schicken.

Bei A sollte man auch nochmal durchdenken, ob das mit string vs. numeric so passt.

 

Je nachdem wie lange der Rest dauert wäre ich auch durchaus noch daran interessiert, das mit in das beta-Release zu bauen.

 

Grüße

 

Adrian
-------------- nächster Teil --------------
Ein Dateianhang mit HTML-Daten wurde abgetrennt...
URL: <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20181030/3b2ed5b3/attachment.html>
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : nicht verfügbar
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20181030/3b2ed5b3/attachment.sig>