[PATCH v2 2/3] gateway.d: Add configuration script for meta information

[Fabian Bläse]

Hallo Adrian,

ja, Duplikate sind nicht so toll. Für mich war das ein einfacher Weg, alles beisammen zu haben, sollte aber wohl für die Upstream-Version wieder raus.

Ich bin dagegen, alles in eine Konfiguration zu packen. Es gibt zwischen Gateway und Node gewaltige Unterschiede was die Konfiguration angeht.
Nicht umsonst trennt man ja auch 'network' und 'dhcp' und wirft nicht alles in eine 'openwrt' Konfiguration.

On 21.03.19 12:40, Adrian Schmutzler wrote:
> - Wir sparen uns doppelte Einträge wie in diesem Patch
Jo, das sollte weg.

> - Wir müssen die Datei nicht extra beim sysupgrade behandeln, fff wird ohnehin migriert
Einfach noch mit in die sysupgrade.conf mit rein.

> - Man kann theoretisch sogar zwischen V2 und GW wechseln, weil die Datei immer die gleiche ist, die gateway-Blöcke würden dann einfach nicht ausgewertet, aber gingen beim sysupgrade nicht verloren
Geht auch, wenn man die Konfiguration getrennt hat.

> Umsetzen könnte man dies relativ leicht, indem man die beiden Eigenschaften der uci-Blöcke ausnutzt: Name und Typ. Du verwendest in deinem Beispiel ja effektiv den Typ als Namen: @gateway[0] .
> In der fff könnte man dann jeweils "gateway" als Typ verwenden und den Namen zur Identifikation verwenden.
Die Auftrennung von Typ und Name ist deshalb da, damit man mehrere "Objekte" eines Typs definieren kann, wie z.B. die VLANs und die Peers.

Alles zusammen in einer Datei finde ich unintuitiv, weil einige Dinge nur von dem einen, einige Dinge nur von dem anderen ausgewertet werden und das nicht klar ersichtlich ist.

Gruß
Fabian

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