[PATCH 1/2] fff-wireless: Also use wXmesh label for wXibss

So Jun 16 16:24:14 CEST 2019

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 at freifunk.net] On Behalf Of Fabian Bläse
Sent: Samstag, 15. Juni 2019 21:56
To: Adrian Schmutzler <freifunk at adrianschmutzler.de>; franken-dev at 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 at adrianschmutzler.de <mailto:freifunk at 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__ 
> 

-------------- nächster Teil --------------
Ein Dateianhang mit HTML-Daten wurde abgetrennt...
URL: <https://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190616/f7e5fbaf/attachment.html>
-------------- 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://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190616/f7e5fbaf/attachment.sig>