Message ID | 20200108172319.8427-1-freifunk@adrianschmutzler.de |
---|---|
State | Accepted |
Headers | show |
diff --git a/src/packages/fff/fff-wireless/Makefile b/src/packages/fff/fff-wireless/Makefile index e805ea22..02328a2b 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:=14 +PKG_RELEASE:=15 PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) diff --git a/src/packages/fff/fff-wireless/files/etc/uci-defaults/24b-config-wireless b/src/packages/fff/fff-wireless/files/etc/uci-defaults/24b-config-wireless index a2c2ff63..d4f2267e 100644 --- a/src/packages/fff/fff-wireless/files/etc/uci-defaults/24b-config-wireless +++ b/src/packages/fff/fff-wireless/files/etc/uci-defaults/24b-config-wireless @@ -41,13 +41,5 @@ case "$BOARD" in ;; esac -# rssileds -case "$BOARD" in - cpe210|\ - cpe210-v2|\ - cpe210-v3|\ - cpe510|\ - tl-wa850re-v1) - /etc/init.d/rssileds disable - ;; -esac +# rssileds: always disable if package is installed +[ -x "/etc/init.d/rssileds" ] && /etc/init.d/rssileds disable
Hallo Adrian, das rssileds wird ja bei den DEVICE_PACKAGES angewählt. Können wir das ganze Paket nicht einfach für die betroffenen Geräte deselektieren? Falls nicht: Reviewed-by: Fabian Bläse <fabian@blaese.de> Gruß Fabian On 08.01.20 18:23, Adrian Schmutzler wrote: > This always disables rssileds if the package is installed, and thus > saves us from specifying particular devices. > > Since rssileds do not work with our concept of resetting WiFi > interfaces, we cannot use it anyway. > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > --- > src/packages/fff/fff-wireless/Makefile | 2 +- > .../files/etc/uci-defaults/24b-config-wireless | 12 ++---------- > 2 files changed, 3 insertions(+), 11 deletions(-) > > diff --git a/src/packages/fff/fff-wireless/Makefile b/src/packages/fff/fff-wireless/Makefile > index e805ea22..02328a2b 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:=14 > +PKG_RELEASE:=15 > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > diff --git a/src/packages/fff/fff-wireless/files/etc/uci-defaults/24b-config-wireless b/src/packages/fff/fff-wireless/files/etc/uci-defaults/24b-config-wireless > index a2c2ff63..d4f2267e 100644 > --- a/src/packages/fff/fff-wireless/files/etc/uci-defaults/24b-config-wireless > +++ b/src/packages/fff/fff-wireless/files/etc/uci-defaults/24b-config-wireless > @@ -41,13 +41,5 @@ case "$BOARD" in > ;; > esac > > -# rssileds > -case "$BOARD" in > - cpe210|\ > - cpe210-v2|\ > - cpe210-v3|\ > - cpe510|\ > - tl-wa850re-v1) > - /etc/init.d/rssileds disable > - ;; > -esac > +# rssileds: always disable if package is installed > +[ -x "/etc/init.d/rssileds" ] && /etc/init.d/rssileds disable >
Hallo Fabian, ja, das ginge genauso. Ich habe das damals auch überlegt, aber mit meiner Lösung kann theoretisch jemand noch die rssileds richtig konfigurieren und benutzen, wenn er möchte. Deshalb ziehe ich diese Lösung vor. Man sollte allerdings überlegen, ob man die Package für den wa850re entfernt, da der tiny ist. Grüße Adrian > -----Original Message----- > From: Fabian Bläse [mailto:fabian@blaese.de] > Sent: Freitag, 10. Januar 2020 11:58 > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken- > dev@freifunk.net > Subject: Re: [PATCH] fff-wireless: always disable rssileds if package is installed > > Hallo Adrian, > > das rssileds wird ja bei den DEVICE_PACKAGES angewählt. Können wir das > ganze Paket nicht einfach für die betroffenen Geräte deselektieren? > > Falls nicht: > Reviewed-by: Fabian Bläse <fabian@blaese.de> > > Gruß > Fabian > > On 08.01.20 18:23, Adrian Schmutzler wrote: > > This always disables rssileds if the package is installed, and thus > > saves us from specifying particular devices. > > > > Since rssileds do not work with our concept of resetting WiFi > > interfaces, we cannot use it anyway. > > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > --- > > src/packages/fff/fff-wireless/Makefile | 2 +- > > .../files/etc/uci-defaults/24b-config-wireless | 12 > ++---------- > > 2 files changed, 3 insertions(+), 11 deletions(-) > > > > diff --git a/src/packages/fff/fff-wireless/Makefile > b/src/packages/fff/fff-wireless/Makefile > > index e805ea22..02328a2b 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:=14 > > +PKG_RELEASE:=15 > > > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > > > diff --git > a/src/packages/fff/fff-wireless/files/etc/uci-defaults/24b-config-wirele > ss > b/src/packages/fff/fff-wireless/files/etc/uci-defaults/24b-config-wirele > ss > > > index a2c2ff63..d4f2267e 100644 > > --- > a/src/packages/fff/fff-wireless/files/etc/uci-defaults/24b-config-wirele > ss > > +++ > b/src/packages/fff/fff-wireless/files/etc/uci-defaults/24b-config-wirele > ss > > @@ -41,13 +41,5 @@ case "$BOARD" in > > ;; > > esac > > > > -# rssileds > > -case "$BOARD" in > > - cpe210|\ > > - cpe210-v2|\ > > - cpe210-v3|\ > > - cpe510|\ > > - tl-wa850re-v1) > > - /etc/init.d/rssileds disable > > - ;; > > -esac > > +# rssileds: always disable if package is installed > > +[ -x "/etc/init.d/rssileds" ] && /etc/init.d/rssileds disable > > > >
Dann würde ich das bevorzugen. Zwar ist das nur ~3KiB groß, mittlerweile sind wir aber dennoch schon so weit, dass wir das Default-Paket wpad wegwerfen wollen. Ich sehe nicht, warum wir das mit rssileds nicht tun sollten. Wer das wirklich haben will, kann sich die Firmware ja immer noch selbst kompilieren. Gruß Fabian On 10.01.20 14:21, Adrian Schmutzler wrote: > Hallo Fabian, > > ja, das ginge genauso. > > Ich habe das damals auch überlegt, aber mit meiner Lösung kann theoretisch jemand noch die rssileds richtig konfigurieren und benutzen, wenn er möchte. Deshalb ziehe ich diese Lösung vor. > > Man sollte allerdings überlegen, ob man die Package für den wa850re entfernt, da der tiny ist. > > Grüße > > Adrian
Man muss halt dann auch immer kucken, welche Geräte gerade die rssileds Package mit anwählen. Das wird dann ein Klassiker, der beim Support immer vergessen wird. Vor ca. zwei Monaten hatten wir dann z.B. auch einen Patch in OpenWrt, der das bei den Ubiquiti-Geräte geändert hat (=repariert), sodass sich dann mit einem OpenWrt Bump ändert, bei welchen Geräten man das entfernen muss (und bei welchen nicht mehr). Da es sich um DEVICE_PACKAGES handelt, muss man das dann auch wirklich immer gerätegenau machen. Aufgrund dieses erhöhten Wartungsaufwands halte ich das für eine unnötige ABM-Maßnahme (auch wenn es wahrscheinlich "ordentlicher" wäre). Grüße Adrian > -----Original Message----- > From: Fabian Bläse [mailto:fabian@blaese.de] > Sent: Freitag, 10. Januar 2020 15:10 > To: Adrian Schmutzler <mail@adrianschmutzler.de>; franken-dev@freifunk.net > Subject: Re: [PATCH] fff-wireless: always disable rssileds if package is installed > > Dann würde ich das bevorzugen. > Zwar ist das nur ~3KiB groß, mittlerweile sind wir aber dennoch schon so weit, > dass wir das Default-Paket wpad wegwerfen wollen. Ich sehe nicht, warum wir > das mit rssileds nicht tun sollten. > > Wer das wirklich haben will, kann sich die Firmware ja immer noch selbst > kompilieren. > > Gruß > Fabian > > On 10.01.20 14:21, Adrian Schmutzler wrote: > > Hallo Fabian, > > > > ja, das ginge genauso. > > > > Ich habe das damals auch überlegt, aber mit meiner Lösung kann theoretisch > jemand noch die rssileds richtig konfigurieren und benutzen, wenn er möchte. > Deshalb ziehe ich diese Lösung vor. > > > > Man sollte allerdings überlegen, ob man die Package für den wa850re entfernt, > da der tiny ist. > > > > Grüße > > > > Adrian
Ich habe den Patch jetzt erstmal so applied. Ich habe nichts dagegen, wenn jemand anderes das so umbaut, dass die Packages für die entsprechenden Geräte entfernt werden, aber ich selbst habe keinen Bock, das zu warten und aktuell zu halten. Ich fände es aber schön, wenn jemand noch den Patch für den WA850RE reviewed, da der (als einziger) tiny ist würde ich bei dem gerne doch die Package ganz entfernen. https://pw.freifunk-franken.de/patch/1267/ Grüße Adrian
Hallo Adrian, okay. Ich würde es entweder bei allen Geräten, oder bei keinem entfernen. Den Code aus diesem Patch kann man ja ggf. drin lassen, falls es doch mal jemand vergessen sollte. Gruß Fabian On 18.01.20 19:54, mail@adrianschmutzler.de wrote: > Ich habe den Patch jetzt erstmal so applied. > > Ich habe nichts dagegen, wenn jemand anderes das so umbaut, dass die Packages für die entsprechenden Geräte entfernt werden, aber ich selbst habe keinen Bock, das zu warten und aktuell zu halten. > > Ich fände es aber schön, wenn jemand noch den Patch für den WA850RE reviewed, da der (als einziger) tiny ist würde ich bei dem gerne doch die Package ganz entfernen. > > https://pw.freifunk-franken.de/patch/1267/ > > Grüße > > Adrian >
This always disables rssileds if the package is installed, and thus saves us from specifying particular devices. Since rssileds do not work with our concept of resetting WiFi interfaces, we cannot use it anyway. Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> --- src/packages/fff/fff-wireless/Makefile | 2 +- .../files/etc/uci-defaults/24b-config-wireless | 12 ++---------- 2 files changed, 3 insertions(+), 11 deletions(-)