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

[mail at adrianschmutzler.de]

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 at freifunk.net] On Behalf Of Fabian Bläse
Sent: Freitag, 31. Mai 2019 19:09
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 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 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/20190531/e12b65c4/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/20190531/e12b65c4/attachment.sig>