[PATCH v2 1/3] fff-gateway: add package

[Adrian Schmutzler]

Hallo Fabian,

lass das execute_subshell einfach.

Bei dem exit 0 ging es mir primär um den return status. Gibt eine Funktion da dann automatisch „0“ zurück?

>>> +revert_changes() { 
>>> +    execute_subshell revert 
>> 
>> 1. Kein reload_config/restart_services hier? Ich weiß, dass das bei test_changes mit dort steht, aber bei apply_changes hast du es mit in die Funktion gezogen. Vll. kann man das einheitlich machen (d.h. ich würde es auch bei revert_changes mit in die Funktion ziehen.
>Das revert war nur eine Möglichkeit im uci gesetzte Einstellungen, die noch nicht applied sind, zurückzusetzen. Daher ist hier auch kein reload_config nötig.
>Applied werden können Dinge nur durch test und apply. test() reverted dann selbst und apply() lässt keinen revert mehr zu.
>Dieses revert() verändert also (by design) nur den Konfigurationszustand im RAM. 

Okay, kann man so auch sehen. Lass es, wie es ist.

Bleiben von meiner Seite das grep -q und v.a. die restart/reload-Konsistenz, danach würde ich das Ding reviewen.
Kannste wegen mir auch schon mal vorab schicken (ohne 2/3 und 3/3), dann ist das schon mal weg.

Beste Grüße

Adrian
-------------- 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         : <https://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190321/9de6554b/attachment.sig>