| Message ID | 20190414220959.38133-1-freifunk@adrianschmutzler.de |
|---|---|
| State | Superseded |
| Headers | show |
diff --git a/src/packages/fff/fff-hoods/Makefile b/src/packages/fff/fff-hoods/Makefile index f1a62896..9d0e6786 100644 --- a/src/packages/fff/fff-hoods/Makefile +++ b/src/packages/fff/fff-hoods/Makefile @@ -1,8 +1,7 @@ include $(TOPDIR)/rules.mk PKG_NAME:=fff-hoods -PKG_VERSION:=5 -PKG_RELEASE:=1 +PKG_RELEASE:=6 PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood index 2d448fca..1f307fbd 100755 --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood @@ -54,7 +54,7 @@ if [ -s "$hoodfilewww" ] && isGatewayAvailable ; then # Break: wXconfig is up uci -q get "wireless.w${freq}configap" > /dev/null && continue # Break: No mesh interface - (uci -q get "wireless.w${freq}mesh" > /dev/null || uci -q get "wireless.w${freq}ibss" > /dev/null) || continue + uci -q get "wireless.w${freq}mesh" > /dev/null || continue # Create configap iface="configap$freq" diff --git a/src/packages/fff/fff-wireless/Makefile b/src/packages/fff/fff-wireless/Makefile index 6c0b6a9b..9b7b4d3f 100644 --- a/src/packages/fff/fff-wireless/Makefile +++ b/src/packages/fff/fff-wireless/Makefile @@ -1,7 +1,7 @@ include $(TOPDIR)/rules.mk PKG_NAME:=fff-wireless -PKG_RELEASE:=8 +PKG_RELEASE:=9 PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) diff --git a/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless index 2473e09a..71a07f93 100644 --- a/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless +++ b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless @@ -100,9 +100,9 @@ wifiAddAdHocMesh() { local bssid=$3 local channel=$(uci get "wireless.${radio}.channel") - local iface="w2ibss" + local iface="w2mesh" if [ "$channel" -gt "14" ]; then - iface="w5ibss" + iface="w5mesh" fi uci batch <<-__EOF__
Hallo Adrian, ich sehe gerade irgendwie den Vorteil davon nicht, außer dass man so eine Überlappung der Namen hat (was tendenziell erstmal schlecht ist). Ich würde wohl eher darüber nachdenken, IBSS Unterstützung fallen zu lassen; kostet uns nur unnötig Entwicklerressourcen. Gruß Fabian On 15.04.19 00:09, Adrian Schmutzler wrote: > The two different names "wXmesh" and "wXibss" have no functional > purpose and are just for indication. Remove this distinction, so > mesh is just mesh. > > This will make further adjustment easier and clearer. > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > --- > src/packages/fff/fff-hoods/Makefile | 3 +-- > src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 2 +- > src/packages/fff/fff-wireless/Makefile | 2 +- > src/packages/fff/fff-wireless/files/lib/functions/fff/wireless | 4 ++-- > 4 files changed, 5 insertions(+), 6 deletions(-) > > diff --git a/src/packages/fff/fff-hoods/Makefile b/src/packages/fff/fff-hoods/Makefile > index f1a62896..9d0e6786 100644 > --- a/src/packages/fff/fff-hoods/Makefile > +++ b/src/packages/fff/fff-hoods/Makefile > @@ -1,8 +1,7 @@ > include $(TOPDIR)/rules.mk > > PKG_NAME:=fff-hoods > -PKG_VERSION:=5 > -PKG_RELEASE:=1 > +PKG_RELEASE:=6 > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > index 2d448fca..1f307fbd 100755 > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > @@ -54,7 +54,7 @@ if [ -s "$hoodfilewww" ] && isGatewayAvailable ; then > # Break: wXconfig is up > uci -q get "wireless.w${freq}configap" > /dev/null && continue > # Break: No mesh interface > - (uci -q get "wireless.w${freq}mesh" > /dev/null || uci -q get "wireless.w${freq}ibss" > /dev/null) || continue > + uci -q get "wireless.w${freq}mesh" > /dev/null || continue > > # Create configap > iface="configap$freq" > diff --git a/src/packages/fff/fff-wireless/Makefile b/src/packages/fff/fff-wireless/Makefile > index 6c0b6a9b..9b7b4d3f 100644 > --- a/src/packages/fff/fff-wireless/Makefile > +++ b/src/packages/fff/fff-wireless/Makefile > @@ -1,7 +1,7 @@ > include $(TOPDIR)/rules.mk > > PKG_NAME:=fff-wireless > -PKG_RELEASE:=8 > +PKG_RELEASE:=9 > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > diff --git a/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless > index 2473e09a..71a07f93 100644 > --- a/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless > +++ b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless > @@ -100,9 +100,9 @@ wifiAddAdHocMesh() { > local bssid=$3 > > local channel=$(uci get "wireless.${radio}.channel") > - local iface="w2ibss" > + local iface="w2mesh" > if [ "$channel" -gt "14" ]; then > - iface="w5ibss" > + iface="w5mesh" > fi > > uci batch <<-__EOF__ >
Hallo Fabian, dieser Patch dient rein der Präparation auf 2/2. Dort lege ich (pro Frequenz) ein wXap, wXmesh und wXconfigap beim firstboot an und schalte die dann nur noch an und aus, je nachdem was configurehood gerade für einen Status hat. (anstatt jedes mal alle Interfaces zu löschen und neu anzulegen) Hier würden ein parallel wXmesh und wXibss nur unnötigen zusätzlichen Code verursachen, zumal wir immer nur eines von beiden haben, nie mehrere gleichzeitig. Da der Patch prinzipiell für sich allein stehen kann, habe ich ihn ausgelagert. Ob man IBSS braucht? Keine Ahnung, mir tut es im Moment nicht weh … Grüße Adrian From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of Fabian Bläse Sent: Freitag, 31. Mai 2019 19:09 To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-dev@freifunk.net Subject: Re: [PATCH 1/2] fff-wireless: Also use wXmesh label for wXibss Hallo Adrian, ich sehe gerade irgendwie den Vorteil davon nicht, außer dass man so eine Überlappung der Namen hat (was tendenziell erstmal schlecht ist). Ich würde wohl eher darüber nachdenken, IBSS Unterstützung fallen zu lassen; kostet uns nur unnötig Entwicklerressourcen. Gruß Fabian On 15.04.19 00:09, Adrian Schmutzler wrote: > The two different names "wXmesh" and "wXibss" have no functional > purpose and are just for indication. Remove this distinction, so > mesh is just mesh. > > This will make further adjustment easier and clearer. > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de <mailto:freifunk@adrianschmutzler.de> > > --- > src/packages/fff/fff-hoods/Makefile | 3 +-- > src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 2 +- > src/packages/fff/fff-wireless/Makefile | 2 +- > src/packages/fff/fff-wireless/files/lib/functions/fff/wireless | 4 ++-- > 4 files changed, 5 insertions(+), 6 deletions(-) > > diff --git a/src/packages/fff/fff-hoods/Makefile b/src/packages/fff/fff-hoods/Makefile > index f1a62896..9d0e6786 100644 > --- a/src/packages/fff/fff-hoods/Makefile > +++ b/src/packages/fff/fff-hoods/Makefile > @@ -1,8 +1,7 @@ > include $(TOPDIR)/rules.mk > > PKG_NAME:=fff-hoods > -PKG_VERSION:=5 > -PKG_RELEASE:=1 > +PKG_RELEASE:=6 > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > index 2d448fca..1f307fbd 100755 > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > @@ -54,7 +54,7 @@ if [ -s "$hoodfilewww" ] && isGatewayAvailable ; then > # Break: wXconfig is up > uci -q get "wireless.w${freq}configap" > /dev/null && continue > # Break: No mesh interface > - (uci -q get "wireless.w${freq}mesh" > /dev/null || uci -q get "wireless.w${freq}ibss" > /dev/null) || continue > + uci -q get "wireless.w${freq}mesh" > /dev/null || continue > > # Create configap > iface="configap$freq" > diff --git a/src/packages/fff/fff-wireless/Makefile b/src/packages/fff/fff-wireless/Makefile > index 6c0b6a9b..9b7b4d3f 100644 > --- a/src/packages/fff/fff-wireless/Makefile > +++ b/src/packages/fff/fff-wireless/Makefile > @@ -1,7 +1,7 @@ > include $(TOPDIR)/rules.mk > > PKG_NAME:=fff-wireless > -PKG_RELEASE:=8 > +PKG_RELEASE:=9 > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > diff --git a/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless > index 2473e09a..71a07f93 100644 > --- a/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless > +++ b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless > @@ -100,9 +100,9 @@ wifiAddAdHocMesh() { > local bssid=$3 > > local channel=$(uci get "wireless.${radio}.channel") > - local iface="w2ibss" > + local iface="w2mesh" > if [ "$channel" -gt "14" ]; then > - iface="w5ibss" > + iface="w5mesh" > fi > > uci batch <<-__EOF__ >
Hallo Adrian, Ich schlage stattdessen vor, lieber mehrere Wifi Devices zu erstellen und dann wechselseitig zu aktivieren. So müssen Eigenheiten der verschiedenen Protokolle - so denn vorhanden - nicht außerhalb des fff-wireless Pakets verändert (gelöscht, hinzugefügt) werden. Für das Wechseln zwischen den Mesh-Varianten sollte fff-wireless dann eine Funktion (z.B. "SwitchToMeshType <meshtype>") bereitstellen, so dass man keins der anderen Optionen (fff-wireless könnte in Zukunft ja um einen weiteren MeshType erweitert werden) vergisst zu deaktivieren. Was hältst du davon? Gruß Fabian On 15.04.19 00:09, Adrian Schmutzler wrote: > The two different names "wXmesh" and "wXibss" have no functional > purpose and are just for indication. Remove this distinction, so > mesh is just mesh. > > This will make further adjustment easier and clearer. > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > --- > src/packages/fff/fff-hoods/Makefile | 3 +-- > src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 2 +- > src/packages/fff/fff-wireless/Makefile | 2 +- > src/packages/fff/fff-wireless/files/lib/functions/fff/wireless | 4 ++-- > 4 files changed, 5 insertions(+), 6 deletions(-) > > diff --git a/src/packages/fff/fff-hoods/Makefile b/src/packages/fff/fff-hoods/Makefile > index f1a62896..9d0e6786 100644 > --- a/src/packages/fff/fff-hoods/Makefile > +++ b/src/packages/fff/fff-hoods/Makefile > @@ -1,8 +1,7 @@ > include $(TOPDIR)/rules.mk > > PKG_NAME:=fff-hoods > -PKG_VERSION:=5 > -PKG_RELEASE:=1 > +PKG_RELEASE:=6 > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > index 2d448fca..1f307fbd 100755 > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > @@ -54,7 +54,7 @@ if [ -s "$hoodfilewww" ] && isGatewayAvailable ; then > # Break: wXconfig is up > uci -q get "wireless.w${freq}configap" > /dev/null && continue > # Break: No mesh interface > - (uci -q get "wireless.w${freq}mesh" > /dev/null || uci -q get "wireless.w${freq}ibss" > /dev/null) || continue > + uci -q get "wireless.w${freq}mesh" > /dev/null || continue > > # Create configap > iface="configap$freq" > diff --git a/src/packages/fff/fff-wireless/Makefile b/src/packages/fff/fff-wireless/Makefile > index 6c0b6a9b..9b7b4d3f 100644 > --- a/src/packages/fff/fff-wireless/Makefile > +++ b/src/packages/fff/fff-wireless/Makefile > @@ -1,7 +1,7 @@ > include $(TOPDIR)/rules.mk > > PKG_NAME:=fff-wireless > -PKG_RELEASE:=8 > +PKG_RELEASE:=9 > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > diff --git a/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless > index 2473e09a..71a07f93 100644 > --- a/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless > +++ b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless > @@ -100,9 +100,9 @@ wifiAddAdHocMesh() { > local bssid=$3 > > local channel=$(uci get "wireless.${radio}.channel") > - local iface="w2ibss" > + local iface="w2mesh" > if [ "$channel" -gt "14" ]; then > - iface="w5ibss" > + iface="w5mesh" > fi > > uci batch <<-__EOF__ >
Hallo Fabian, ich finde es gerade erstrebenswert, nur ein mesh-Interface zu haben. Für alle Schritte außer dem Setup kann ich das dann als Blackbox betrachten und muss nicht überlegen, welche Typen es gibt etc. Am Schluss ist es natürlich eine Geschmacks-/Gestaltungsfrage. Deinen Ansatz verstehe ich auch, allerdings ist fff-hoods dann ja trotzdem nicht unabhängig: Man muss ja wissen, welche Wifi-Typen es gibt, und diese dann entsprechend ansprechen. Zudem würde der jeweils andere Adapter dann falsch konfiguriert bleiben. Und es macht bumm, wenn jemand manuell in der /etc/config/wireless alles auf disabled=0 setzt. Auch das hoodfile-Konzept sieht ja im Prinzip vor, dass es nur ein Mesh-Interface gibt. Ich persönlich bevorzuge daher meine Lösung und habe auch nicht direkt Lust, eine alternative Variante für die off. FW zu implementieren. Ich habe auch überlegt, ob man einen Teil des Setups in configurehoods dann noch in Funktionen packt. Mein Eindruck war aber, dass es die Sache eher unübersichtlicher macht, da es ja doch überschaubar viele Zeilen sind, die dann in eine Funktion ausgelagert würden. Beste Grüße Adrian From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of Fabian Bläse Sent: Samstag, 15. Juni 2019 21:56 To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken-dev@freifunk.net Subject: Re: [PATCH 1/2] fff-wireless: Also use wXmesh label for wXibss Hallo Adrian, Ich schlage stattdessen vor, lieber mehrere Wifi Devices zu erstellen und dann wechselseitig zu aktivieren. So müssen Eigenheiten der verschiedenen Protokolle - so denn vorhanden - nicht außerhalb des fff-wireless Pakets verändert (gelöscht, hinzugefügt) werden. Für das Wechseln zwischen den Mesh-Varianten sollte fff-wireless dann eine Funktion (z.B. "SwitchToMeshType <meshtype>") bereitstellen, so dass man keins der anderen Optionen (fff-wireless könnte in Zukunft ja um einen weiteren MeshType erweitert werden) vergisst zu deaktivieren. Was hältst du davon? Gruß Fabian On 15.04.19 00:09, Adrian Schmutzler wrote: > The two different names "wXmesh" and "wXibss" have no functional > purpose and are just for indication. Remove this distinction, so > mesh is just mesh. > > This will make further adjustment easier and clearer. > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de <mailto:freifunk@adrianschmutzler.de> > > --- > src/packages/fff/fff-hoods/Makefile | 3 +-- > src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 2 +- > src/packages/fff/fff-wireless/Makefile | 2 +- > src/packages/fff/fff-wireless/files/lib/functions/fff/wireless | 4 ++-- > 4 files changed, 5 insertions(+), 6 deletions(-) > > diff --git a/src/packages/fff/fff-hoods/Makefile b/src/packages/fff/fff-hoods/Makefile > index f1a62896..9d0e6786 100644 > --- a/src/packages/fff/fff-hoods/Makefile > +++ b/src/packages/fff/fff-hoods/Makefile > @@ -1,8 +1,7 @@ > include $(TOPDIR)/rules.mk > > PKG_NAME:=fff-hoods > -PKG_VERSION:=5 > -PKG_RELEASE:=1 > +PKG_RELEASE:=6 > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > index 2d448fca..1f307fbd 100755 > --- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > +++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood > @@ -54,7 +54,7 @@ if [ -s "$hoodfilewww" ] && isGatewayAvailable ; then > # Break: wXconfig is up > uci -q get "wireless.w${freq}configap" > /dev/null && continue > # Break: No mesh interface > - (uci -q get "wireless.w${freq}mesh" > /dev/null || uci -q get "wireless.w${freq}ibss" > /dev/null) || continue > + uci -q get "wireless.w${freq}mesh" > /dev/null || continue > > # Create configap > iface="configap$freq" > diff --git a/src/packages/fff/fff-wireless/Makefile b/src/packages/fff/fff-wireless/Makefile > index 6c0b6a9b..9b7b4d3f 100644 > --- a/src/packages/fff/fff-wireless/Makefile > +++ b/src/packages/fff/fff-wireless/Makefile > @@ -1,7 +1,7 @@ > include $(TOPDIR)/rules.mk > > PKG_NAME:=fff-wireless > -PKG_RELEASE:=8 > +PKG_RELEASE:=9 > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > diff --git a/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless > index 2473e09a..71a07f93 100644 > --- a/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless > +++ b/src/packages/fff/fff-wireless/files/lib/functions/fff/wireless > @@ -100,9 +100,9 @@ wifiAddAdHocMesh() { > local bssid=$3 > > local channel=$(uci get "wireless.${radio}.channel") > - local iface="w2ibss" > + local iface="w2mesh" > if [ "$channel" -gt "14" ]; then > - iface="w5ibss" > + iface="w5mesh" > fi > > uci batch <<-__EOF__ >
Hallo Adrian, On 16.06.19 16:24, mail@adrianschmutzler.de wrote: > Hallo Fabian, > > ich finde es gerade erstrebenswert, nur ein mesh-Interface zu haben. Für alle Schritte außer dem Setup kann ich das dann als Blackbox betrachten und muss nicht überlegen, welche Typen es gibt etc. Aber wo, außer beim Setup, müssen wir die mesh-Interfaces nochmal anfassen? So macht man halt das Setup abhängig von der konkreten Implementierung, obwohl es eigentlich ein Modul zur Abstraktion gibt. Es ist dadurch beispielsweise nicht mehr Trivial möglich, einen weiteren Mesh Modus unabhängig vom fff-hoods Paket (und allen anderen Paketen, die Mesh in Zukunft verwenden werden) hinzuzufügen. Folgender Kompromissvorschlag: Umschalten und Aktivieren der Mesh Interfaces per Funktion, das fff-wireless kann dann in der konkreten Implementierung die entsprechenden Änderungen am wXmesh machen. Dann hat man ein einzelnes Mesh Interface, aber keine Abhängigkeit der Setups zu der konkreten Implementierung. Dann gibts auch weniger doppelten Code, wenn man WLAN Interfaces für die Gatewayfirmware implementiert. > Deinen Ansatz verstehe ich auch, allerdings ist fff-hoods dann ja trotzdem nicht unabhängig: Man muss ja wissen, welche Wifi-Typen es gibt, und diese dann entsprechend ansprechen. Muss man nicht. Man muss nur wissen, ob der Typ, den man verwenden möchte, vom Modul bereitgestellt wird. > Zudem würde der jeweils andere Adapter dann falsch konfiguriert bleiben. Und es macht bumm, wenn jemand manuell in der /etc/config/wireless alles auf disabled=0 setzt. Auch das hoodfile-Konzept sieht ja im Prinzip vor, dass es nur ein Mesh-Interface gibt. Siehe Vorschlag oben. > Ich persönlich bevorzuge daher meine Lösung und habe auch nicht direkt Lust, eine alternative Variante für die off. FW zu implementieren. Das nimmt dir keiner Übel. Ich würde mich bereit erklären meinen Alternativvorschlag von oben zu implementieren, wenn sich dafür eine Mehrheit findet. Gruß Fabian
Hallo Fabian, > Es ist dadurch beispielsweise nicht mehr Trivial möglich, einen weiteren Mesh Modus unabhängig vom fff-hoods Paket (und allen anderen Paketen, die Mesh in Zukunft verwenden werden) hinzuzufügen. Das ist ja auch richtig so, denn es macht ja keinen Sinn, an den Mesh-Interfaces rumzupfuschen, solange configurehood das tut. Ja mehr ich darüber nachdenke, desto mehr bin ich der Meinung, dass es doch keine Geschmacksfrage ist, sondern eigentlich nur ein wXmesh existieren darf, solange man configurehood benutzt. Mit anderen Worten: wXmesh ist das mesh-Interface, das durch configurehood konfiguriert wird. > Folgender Kompromissvorschlag: > Umschalten und Aktivieren der Mesh Interfaces per Funktion, das fff-wireless kann dann in der konkreten Implementierung die entsprechenden Änderungen am wXmesh machen. > Dann hat man ein einzelnes Mesh Interface, aber keine Abhängigkeit der Setups zu der konkreten Implementierung. Dann gibts auch weniger doppelten Code, wenn man WLAN Interfaces für die Gatewayfirmware implementiert. Ich denke mal darüber nach. Grüße Adrian
Hallo Adrian, On 16.06.19 23:21, mail@adrianschmutzler.de wrote: > Hallo Fabian, > >> Es ist dadurch beispielsweise nicht mehr Trivial möglich, einen weiteren Mesh Modus unabhängig vom fff-hoods Paket (und allen anderen Paketen, die Mesh in Zukunft verwenden werden) hinzuzufügen. > > Das ist ja auch richtig so, denn es macht ja keinen Sinn, an den Mesh-Interfaces rumzupfuschen, solange configurehood das tut. Ja mehr ich darüber nachdenke, desto mehr bin ich der Meinung, dass es doch keine Geschmacksfrage ist, sondern eigentlich nur ein wXmesh existieren darf, solange man configurehood benutzt. Mit anderen Worten: wXmesh ist das mesh-Interface, das durch configurehood konfiguriert wird. Im Moment nicht. Mit der Gatewayfirmware vielleicht irgendwann schon. Außerdem lässt sich der Code dann unabhängig voneinander bewerten und muss nicht doppelt reviewed werden, wenn man an anderer Stelle nochmal WLAN Einstellungen verändert (Gatewayfirmware). Genau dafür machen wir diese Modularisierung in Pakete und Funktionen doch, oder? Gruß Fabian
The two different names "wXmesh" and "wXibss" have no functional purpose and are just for indication. Remove this distinction, so mesh is just mesh. This will make further adjustment easier and clearer. Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> --- src/packages/fff/fff-hoods/Makefile | 3 +-- src/packages/fff/fff-hoods/files/usr/sbin/configurehood | 2 +- src/packages/fff/fff-wireless/Makefile | 2 +- src/packages/fff/fff-wireless/files/lib/functions/fff/wireless | 4 ++-- 4 files changed, 5 insertions(+), 6 deletions(-)