| Message ID | 20200331190316.1510308-1-fabian@blaese.de |
|---|---|
| State | Rejected |
| Headers | show |
diff --git a/src/packages/fff/fff-boardname/Makefile b/src/packages/fff/fff-boardname/Makefile index 9c2f76d..01fcc0a 100644 --- a/src/packages/fff/fff-boardname/Makefile +++ b/src/packages/fff/fff-boardname/Makefile @@ -1,7 +1,7 @@ include $(TOPDIR)/rules.mk PKG_NAME:=fff-boardname -PKG_RELEASE:=6 +PKG_RELEASE:=7 PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) diff --git a/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff-boardname b/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff-boardname index cec595c..109dbe9 100644 --- a/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff-boardname +++ b/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff-boardname @@ -73,6 +73,9 @@ case "$BOARD" in archer-c7) BOARD=archer-c7-v2 ;; + tplink,c2600) + BOARD=tplink_c2600 + ;; esac uci set board.model.name=$BOARD diff --git a/src/packages/fff/fff-network/Makefile b/src/packages/fff/fff-network/Makefile index 8f8c7f2..0c7e2bd 100644 --- a/src/packages/fff/fff-network/Makefile +++ b/src/packages/fff/fff-network/Makefile @@ -1,7 +1,7 @@ include $(TOPDIR)/rules.mk PKG_NAME:=fff-network -PKG_RELEASE:=17 +PKG_RELEASE:=18 PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) diff --git a/src/packages/fff/fff-network/ipq806x/network.tplink,c2600 b/src/packages/fff/fff-network/ipq806x/network.tplink_c2600 similarity index 100% rename from src/packages/fff/fff-network/ipq806x/network.tplink,c2600 rename to src/packages/fff/fff-network/ipq806x/network.tplink_c2600
hi Reviewed-by: Christian Dresel <fff@chrisi01.de> Gruß Christian On 31.03.20 21:03, Fabian Bläse wrote: > Because the upgrade filename is built based on > fff-boardname's configuration, and there is no special > treatment for the tplink c2600, the resulting filename > contains 'tplink,c2600', which is not the correct filename. > > This patch sets the boardname explicitly, so the correct filename > is constructed in fff-upgrade, which also requires renaming the > fff-network file for this device. > > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > src/packages/fff/fff-boardname/Makefile | 2 +- > .../fff/fff-boardname/files/etc/uci-defaults/15-fff-boardname | 3 +++ > src/packages/fff/fff-network/Makefile | 2 +- > .../ipq806x/{network.tplink,c2600 => network.tplink_c2600} | 0 > 4 files changed, 5 insertions(+), 2 deletions(-) > rename src/packages/fff/fff-network/ipq806x/{network.tplink,c2600 => network.tplink_c2600} (100%) > > diff --git a/src/packages/fff/fff-boardname/Makefile b/src/packages/fff/fff-boardname/Makefile > index 9c2f76d..01fcc0a 100644 > --- a/src/packages/fff/fff-boardname/Makefile > +++ b/src/packages/fff/fff-boardname/Makefile > @@ -1,7 +1,7 @@ > include $(TOPDIR)/rules.mk > > PKG_NAME:=fff-boardname > -PKG_RELEASE:=6 > +PKG_RELEASE:=7 > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > diff --git a/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff-boardname b/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff-boardname > index cec595c..109dbe9 100644 > --- a/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff-boardname > +++ b/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff-boardname > @@ -73,6 +73,9 @@ case "$BOARD" in > archer-c7) > BOARD=archer-c7-v2 > ;; > + tplink,c2600) > + BOARD=tplink_c2600 > + ;; > esac > > uci set board.model.name=$BOARD > diff --git a/src/packages/fff/fff-network/Makefile b/src/packages/fff/fff-network/Makefile > index 8f8c7f2..0c7e2bd 100644 > --- a/src/packages/fff/fff-network/Makefile > +++ b/src/packages/fff/fff-network/Makefile > @@ -1,7 +1,7 @@ > include $(TOPDIR)/rules.mk > > PKG_NAME:=fff-network > -PKG_RELEASE:=17 > +PKG_RELEASE:=18 > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > diff --git a/src/packages/fff/fff-network/ipq806x/network.tplink,c2600 b/src/packages/fff/fff-network/ipq806x/network.tplink_c2600 > similarity index 100% > rename from src/packages/fff/fff-network/ipq806x/network.tplink,c2600 > rename to src/packages/fff/fff-network/ipq806x/network.tplink_c2600 >
Hallo Fabian, NAK. Der boardname wird in OpenWrt konsistent mit Komma verwendet. Mit Unterstrich wird im Wesentlichen nur der Image-Name und die Config-Variablen gesetzt. Hier für eine Variable, die "board" heißt, den Unterstrich zu verwenden wäre enorm inkonsistent. Wir können uns höchstens entscheiden, für die Zukunft eine "neue" Variable mit Unterstrich zu definieren. Ab der nächsten OpenWrt Version würde man dann einfach für alle Geräte das Komma gegen einen Punkt ersetzen "sed 's/,/_/' ". Ich halte das aber nicht für klug. Beste Grüße Adrian > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf > Of Fabian Bläse > Sent: Dienstag, 31. März 2020 21:03 > To: franken-dev@freifunk.net > Subject: [PATCH] Fix tplink c2600 boardname for fff-upgrade > > Because the upgrade filename is built based on fff-boardname's > configuration, and there is no special treatment for the tplink c2600, the > resulting filename contains 'tplink,c2600', which is not the correct filename. > > This patch sets the boardname explicitly, so the correct filename is > constructed in fff-upgrade, which also requires renaming the fff-network file > for this device. > > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > src/packages/fff/fff-boardname/Makefile | 2 +- > .../fff/fff-boardname/files/etc/uci-defaults/15-fff-boardname | 3 +++ > src/packages/fff/fff-network/Makefile | 2 +- > .../ipq806x/{network.tplink,c2600 => network.tplink_c2600} | 0 > 4 files changed, 5 insertions(+), 2 deletions(-) rename src/packages/fff/fff- > network/ipq806x/{network.tplink,c2600 => network.tplink_c2600} (100%) > > diff --git a/src/packages/fff/fff-boardname/Makefile b/src/packages/fff/fff- > boardname/Makefile > index 9c2f76d..01fcc0a 100644 > --- a/src/packages/fff/fff-boardname/Makefile > +++ b/src/packages/fff/fff-boardname/Makefile > @@ -1,7 +1,7 @@ > include $(TOPDIR)/rules.mk > > PKG_NAME:=fff-boardname > -PKG_RELEASE:=6 > +PKG_RELEASE:=7 > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > diff --git a/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff- > boardname b/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff- > boardname > index cec595c..109dbe9 100644 > --- a/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff- > boardname > +++ b/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff-board > +++ name > @@ -73,6 +73,9 @@ case "$BOARD" in > archer-c7) > BOARD=archer-c7-v2 > ;; > + tplink,c2600) > + BOARD=tplink_c2600 > + ;; > esac > > uci set board.model.name=$BOARD > diff --git a/src/packages/fff/fff-network/Makefile b/src/packages/fff/fff- > network/Makefile > index 8f8c7f2..0c7e2bd 100644 > --- a/src/packages/fff/fff-network/Makefile > +++ b/src/packages/fff/fff-network/Makefile > @@ -1,7 +1,7 @@ > include $(TOPDIR)/rules.mk > > PKG_NAME:=fff-network > -PKG_RELEASE:=17 > +PKG_RELEASE:=18 > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) > > diff --git a/src/packages/fff/fff-network/ipq806x/network.tplink,c2600 > b/src/packages/fff/fff-network/ipq806x/network.tplink_c2600 > similarity index 100% > rename from src/packages/fff/fff-network/ipq806x/network.tplink,c2600 > rename to src/packages/fff/fff-network/ipq806x/network.tplink_c2600 > -- > 2.26.0
Hallo Adrian, das Problem ist, dass wir irgendwie an den Dateinamen für das Upgrade kommen müssen. Aktuell wird dafür immer fff-$VARIANT-$VERSION-$BOARD-sysupgrade.bin verwendet. Gibt es im OpenWRT einen sauberen Weg, um an den Dateinamen für ein Upgrade zu kommen? Falls nein, bin ich sehr dafür, erstmal diesen Fix mit aufzunehmen, ansonsten ist das fff-upgrade.sh für den C2600 kaputt. Das ganze ist mir leider erst jetzt aufgefallen. Ggf. könnte man auch das fff-sysupgrade um den genannten sed ergänzen, das müsste aber direkt in der aktuellen Version gemacht werden. Das fff-sysupgrade können wir später irgendwann einmal sauber überarbeiten. Am besten wäre es, wenn wir an der Stelle keinen Boardnamen getrennt von OpenWRT pflegen müssten. Gruß Fabian On 31.03.20 22:01, mail@adrianschmutzler.de wrote: > Hallo Fabian, > > NAK. > > Der boardname wird in OpenWrt konsistent mit Komma verwendet. > > Mit Unterstrich wird im Wesentlichen nur der Image-Name und die Config-Variablen gesetzt. > > Hier für eine Variable, die "board" heißt, den Unterstrich zu verwenden wäre enorm inkonsistent. > > Wir können uns höchstens entscheiden, für die Zukunft eine "neue" Variable mit Unterstrich zu definieren. Ab der nächsten OpenWrt Version würde man dann einfach für alle Geräte das Komma gegen einen Punkt ersetzen "sed 's/,/_/' ". Ich halte das aber nicht für klug. > > Beste Grüße > > Adrian > >> -----Original Message----- >> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf >> Of Fabian Bläse >> Sent: Dienstag, 31. März 2020 21:03 >> To: franken-dev@freifunk.net >> Subject: [PATCH] Fix tplink c2600 boardname for fff-upgrade >> >> Because the upgrade filename is built based on fff-boardname's >> configuration, and there is no special treatment for the tplink c2600, the >> resulting filename contains 'tplink,c2600', which is not the correct filename. >> >> This patch sets the boardname explicitly, so the correct filename is >> constructed in fff-upgrade, which also requires renaming the fff-network file >> for this device. >> >> Signed-off-by: Fabian Bläse <fabian@blaese.de> >> --- >> src/packages/fff/fff-boardname/Makefile | 2 +- >> .../fff/fff-boardname/files/etc/uci-defaults/15-fff-boardname | 3 +++ >> src/packages/fff/fff-network/Makefile | 2 +- >> .../ipq806x/{network.tplink,c2600 => network.tplink_c2600} | 0 >> 4 files changed, 5 insertions(+), 2 deletions(-) rename src/packages/fff/fff- >> network/ipq806x/{network.tplink,c2600 => network.tplink_c2600} (100%) >> >> diff --git a/src/packages/fff/fff-boardname/Makefile b/src/packages/fff/fff- >> boardname/Makefile >> index 9c2f76d..01fcc0a 100644 >> --- a/src/packages/fff/fff-boardname/Makefile >> +++ b/src/packages/fff/fff-boardname/Makefile >> @@ -1,7 +1,7 @@ >> include $(TOPDIR)/rules.mk >> >> PKG_NAME:=fff-boardname >> -PKG_RELEASE:=6 >> +PKG_RELEASE:=7 >> >> PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) >> >> diff --git a/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff- >> boardname b/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff- >> boardname >> index cec595c..109dbe9 100644 >> --- a/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff- >> boardname >> +++ b/src/packages/fff/fff-boardname/files/etc/uci-defaults/15-fff-board >> +++ name >> @@ -73,6 +73,9 @@ case "$BOARD" in >> archer-c7) >> BOARD=archer-c7-v2 >> ;; >> + tplink,c2600) >> + BOARD=tplink_c2600 >> + ;; >> esac >> >> uci set board.model.name=$BOARD >> diff --git a/src/packages/fff/fff-network/Makefile b/src/packages/fff/fff- >> network/Makefile >> index 8f8c7f2..0c7e2bd 100644 >> --- a/src/packages/fff/fff-network/Makefile >> +++ b/src/packages/fff/fff-network/Makefile >> @@ -1,7 +1,7 @@ >> include $(TOPDIR)/rules.mk >> >> PKG_NAME:=fff-network >> -PKG_RELEASE:=17 >> +PKG_RELEASE:=18 >> >> PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME) >> >> diff --git a/src/packages/fff/fff-network/ipq806x/network.tplink,c2600 >> b/src/packages/fff/fff-network/ipq806x/network.tplink_c2600 >> similarity index 100% >> rename from src/packages/fff/fff-network/ipq806x/network.tplink,c2600 >> rename to src/packages/fff/fff-network/ipq806x/network.tplink_c2600 >> -- >> 2.26.0
Hallo Fabian, > das Problem ist, dass wir irgendwie an den Dateinamen für das Upgrade kommen müssen. > Aktuell wird dafür immer fff-$VARIANT-$VERSION-$BOARD-sysupgrade.bin verwendet. Die saubere Lösung ist, an der Stelle, wo der der Dateiname zusammengebaut wird, ein sed 's/,/_/' zu machen. Z.B. DEVICE_NAME=$(echo $BOARD | sed 's/,/_/') oder die Kurzschreibweise für Variablen, die mir gerade nicht einfällt. > Gibt es im OpenWRT einen sauberen Weg, um an den Dateinamen für ein Upgrade zu kommen? > Falls nein, bin ich sehr dafür, erstmal diesen Fix mit aufzunehmen, ansonsten ist das fff-upgrade.sh für den C2600 kaputt. Das ganze ist mir leider erst jetzt aufgefallen. > Ggf. könnte man auch das fff-sysupgrade um den genannten sed ergänzen, das müsste aber direkt in der aktuellen Version gemacht werden. Wo wir die Variable ändern, ist für das sysupgrade doch egal? Der Zeitpunkt bleibt der gleiche. > Das fff-sysupgrade können wir später irgendwann einmal sauber überarbeiten. > Am besten wäre es, wenn wir an der Stelle keinen Boardnamen getrennt von OpenWRT pflegen müssten. Und genau weil das demnächst so sein wird, sollte wir für den _board name_ nicht von der OpenWrt syntax mit Komma abweichen (im Prinzip ist das die Syntax der DTS compatible). Mit ath79 haben wir dann nämlich nur noch so was wie tplink,archer-c60-v1 und ubnt,nanostation-loco-m. So steht es auch in den board.d Skripten. Und wenn man daraus einen Dateinamen machen will, macht man einfach sed 's/,/_/', aber an der Stelle, wo man es braucht. Gruß Adrian
On 31.03.20 23:10, mail@adrianschmutzler.de wrote: >> Gibt es im OpenWRT einen sauberen Weg, um an den Dateinamen für ein Upgrade zu kommen? >> Falls nein, bin ich sehr dafür, erstmal diesen Fix mit aufzunehmen, ansonsten ist das fff-upgrade.sh für den C2600 kaputt. Das ganze ist mir leider erst jetzt aufgefallen. >> Ggf. könnte man auch das fff-sysupgrade um den genannten sed ergänzen, das müsste aber direkt in der aktuellen Version gemacht werden. > > Wo wir die Variable ändern, ist für das sysupgrade doch egal? Der Zeitpunkt bleibt der gleiche. Wo wir die Variable ändern ist egal, ja. Das war nur darauf bezogen, dass du das erst ab der nächsten OpenWRT Version gemacht hättest, wir das aber schon für die aktuelle unbedingt brauchen. >> Das fff-sysupgrade können wir später irgendwann einmal sauber überarbeiten. >> Am besten wäre es, wenn wir an der Stelle keinen Boardnamen getrennt von OpenWRT pflegen müssten. > > Und genau weil das demnächst so sein wird, sollte wir für den _board name_ nicht von der OpenWrt syntax mit Komma abweichen (im Prinzip ist das die Syntax der DTS compatible). > Mit ath79 haben wir dann nämlich nur noch so was wie tplink,archer-c60-v1 und ubnt,nanostation-loco-m. So steht es auch in den board.d Skripten. > > Und wenn man daraus einen Dateinamen machen will, macht man einfach sed 's/,/_/', aber an der Stelle, wo man es braucht. Jo, dann hatte ich dich nur etwas falsch verstanden. Ich würde das morgen dann mal in einen Patch gießen. Gruß Fabian
Im Prinzip ist das nichts anderes als dieser Patch: https://pw.freifunk-franken.de/patch/1304/ Ich hatte nur damals an die network.* Dateien gedacht, und nicht an das sysupgrade. Ich schicke davon auch mal prophylaktisch eine v2. Grüße Adrian From: Fabian Bläse [mailto:fabian@blaese.de] Sent: Dienstag, 31. März 2020 23:15 To: mail@adrianschmutzler.de; franken-dev@freifunk.net Subject: Re: [PATCH] Fix tplink c2600 boardname for fff-upgrade On 31.03.20 23:10, mail@adrianschmutzler.de <mailto:mail@adrianschmutzler.de> wrote: >> Gibt es im OpenWRT einen sauberen Weg, um an den Dateinamen für ein Upgrade zu kommen? >> Falls nein, bin ich sehr dafür, erstmal diesen Fix mit aufzunehmen, ansonsten ist das fff-upgrade.sh für den C2600 kaputt. Das ganze ist mir leider erst jetzt aufgefallen. >> Ggf. könnte man auch das fff-sysupgrade um den genannten sed ergänzen, das müsste aber direkt in der aktuellen Version gemacht werden. > > Wo wir die Variable ändern, ist für das sysupgrade doch egal? Der Zeitpunkt bleibt der gleiche. Wo wir die Variable ändern ist egal, ja. Das war nur darauf bezogen, dass du das erst ab der nächsten OpenWRT Version gemacht hättest, wir das aber schon für die aktuelle unbedingt brauchen. >> Das fff-sysupgrade können wir später irgendwann einmal sauber überarbeiten. >> Am besten wäre es, wenn wir an der Stelle keinen Boardnamen getrennt von OpenWRT pflegen müssten. > > Und genau weil das demnächst so sein wird, sollte wir für den _board name_ nicht von der OpenWrt syntax mit Komma abweichen (im Prinzip ist das die Syntax der DTS compatible). > Mit ath79 haben wir dann nämlich nur noch so was wie tplink,archer-c60-v1 und ubnt,nanostation-loco-m. So steht es auch in den board.d Skripten. > > Und wenn man daraus einen Dateinamen machen will, macht man einfach sed 's/,/_/', aber an der Stelle, wo man es braucht. Jo, dann hatte ich dich nur etwas falsch verstanden. Ich würde das morgen dann mal in einen Patch gießen. Gruß Fabian
Das ganze geht dann aber widerrum davon aus, dass der OpenWRT Imagename der "vendor_device"-Syntax entspricht. Das ist z.B. beim 4900 aktuell nicht der Fall, löst unser Problem also auch nicht vollständig. Gruß Fabian On 31.03.20 23:22, mail@adrianschmutzler.de wrote: > Im Prinzip ist das nichts anderes als dieser Patch: > > > > https://pw.freifunk-franken.de/patch/1304/ > > > > Ich hatte nur damals an die network.* Dateien gedacht, und nicht an das sysupgrade. > > > > Ich schicke davon auch mal prophylaktisch eine v2. > > > > Grüße > > > > Adrian > > > > > > > > *From:*Fabian Bläse [mailto:fabian@blaese.de] > *Sent:* Dienstag, 31. März 2020 23:15 > *To:* mail@adrianschmutzler.de; franken-dev@freifunk.net > *Subject:* Re: [PATCH] Fix tplink c2600 boardname for fff-upgrade > > > > On 31.03.20 23:10, mail@adrianschmutzler.de <mailto:mail@adrianschmutzler.de> wrote: >>> Gibt es im OpenWRT einen sauberen Weg, um an den Dateinamen für ein Upgrade zu kommen? >>> Falls nein, bin ich sehr dafür, erstmal diesen Fix mit aufzunehmen, ansonsten ist das fff-upgrade.sh für den C2600 kaputt. Das ganze ist mir leider erst jetzt aufgefallen. > >>> Ggf. könnte man auch das fff-sysupgrade um den genannten sed ergänzen, das müsste aber direkt in der aktuellen Version gemacht werden. > >> >> Wo wir die Variable ändern, ist für das sysupgrade doch egal? Der Zeitpunkt bleibt der gleiche. > Wo wir die Variable ändern ist egal, ja. > Das war nur darauf bezogen, dass du das erst ab der nächsten OpenWRT Version gemacht hättest, wir das aber schon für die aktuelle unbedingt brauchen. > >>> Das fff-sysupgrade können wir später irgendwann einmal sauber überarbeiten. >>> Am besten wäre es, wenn wir an der Stelle keinen Boardnamen getrennt von OpenWRT pflegen müssten. >> >> Und genau weil das demnächst so sein wird, sollte wir für den _board name_ nicht von der OpenWrt syntax mit Komma abweichen (im Prinzip ist das die Syntax der DTS compatible). > >> Mit ath79 haben wir dann nämlich nur noch so was wie tplink,archer-c60-v1 und ubnt,nanostation-loco-m. So steht es auch in den board.d Skripten. > >> >> Und wenn man daraus einen Dateinamen machen will, macht man einfach sed 's/,/_/', aber an der Stelle, wo man es braucht. > > Jo, dann hatte ich dich nur etwas falsch verstanden. > Ich würde das morgen dann mal in einen Patch gießen. > > Gruß > Fabian > > >
Das ganze geht dann aber widerrum davon aus, dass der OpenWRT Imagename der "vendor_device"-Syntax entspricht. Das ist z.B. beim 4900 aktuell nicht der Fall, löst unser Problem also auch nicht vollständig. Gruß Fabian
Ah, für den 4900 ist das im master in Commit 118749271b3 schon angepasst. Dann würde ich fast überlegen, diesen Patch gleich mitzunehmen, falls nichts dagegen spricht. On 01.04.20 11:15, Fabian Bläse wrote: > Das ganze geht dann aber widerrum davon aus, dass der OpenWRT Imagename der "vendor_device"-Syntax entspricht. > Das ist z.B. beim 4900 aktuell nicht der Fall, löst unser Problem also auch nicht vollständig. > > Gruß > Fabian >
Hi, wenn kein Komma im Namen steht, passiert beim sed einfach nichts. Kein Problem? Grüße Adrian
Ja, stimmt schon. Aber da ist ja auch nur kein Komma drin, weil wir es im fff-boardname explizit weg machen, womit das ganze dann kaum besser ist, als mein ursprünglicher Patch. Daher würde ich vorschlagen, das möglichst gleich (bzw. dann mit dem Wechsel auf ath79) ordentlich zu machen und für alle Geräte auf die in OpenWRT angestrebte Syntax umzustellen. Das muss jetzt nicht direkt mit ins nächste Release, aber wir sollten uns auf jeden Fall in diese Richtung orientieren. Vielleicht können wir uns dann irgendwann auch mal das fff-boardname (bzw. zumindest das Gerätespezifische überschreiben des Boardnames) wegsparen. Gruß Fabian On 01.04.20 11:26, mail@adrianschmutzler.de wrote: > Hi, > > wenn kein Komma im Namen steht, passiert beim sed einfach nichts. > > Kein Problem? > > Grüße > > Adrian >
Mit der ath79 Umstellung fliegt das boardname file mehr oder weniger komplett weg. Wir können tatsächlich überlegen, ob wir den Pfusch beim WDR4900 auch jetzt schon rauswerfen. Den Patch aus dem Master sollte man ohne weiteres lokal übernehmen können (ich habs gerade mal angekuckt). Ich werde ihn nur nicht bei OpenWrt backporten, weil das eine kosmetische Änderung ist. Einzig etwas eklig wird es bei den ramips Geräten, weil diese von 19.07 zu 20.xx umbenannt werden (z.B. auch tplink,c50-v4 zu tplink,archer-c50-v4). Das sollte uns aber nur am Rande stören. Grüße Adrian From: Fabian Bläse [mailto:fabian@blaese.de] Sent: Mittwoch, 1. April 2020 11:31 To: mail@adrianschmutzler.de; franken-dev@freifunk.net Subject: Re: [PATCH] Fix tplink c2600 boardname for fff-upgrade Ja, stimmt schon. Aber da ist ja auch nur kein Komma drin, weil wir es im fff-boardname explizit weg machen, womit das ganze dann kaum besser ist, als mein ursprünglicher Patch. Daher würde ich vorschlagen, das möglichst gleich (bzw. dann mit dem Wechsel auf ath79) ordentlich zu machen und für alle Geräte auf die in OpenWRT angestrebte Syntax umzustellen. Das muss jetzt nicht direkt mit ins nächste Release, aber wir sollten uns auf jeden Fall in diese Richtung orientieren. Vielleicht können wir uns dann irgendwann auch mal das fff-boardname (bzw. zumindest das Gerätespezifische überschreiben des Boardnames) wegsparen. Gruß Fabian On 01.04.20 11:26, mail@adrianschmutzler.de <mailto:mail@adrianschmutzler.de> wrote: > Hi, > > wenn kein Komma im Namen steht, passiert beim sed einfach nichts. > > Kein Problem? > > Grüße > > Adrian >
On 01.04.20 11:36, mail@adrianschmutzler.de wrote: > Mit der ath79 Umstellung fliegt das boardname file mehr oder weniger komplett weg. > > Wir können tatsächlich überlegen, ob wir den Pfusch beim WDR4900 auch jetzt schon rauswerfen. > > Den Patch aus dem Master sollte man ohne weiteres lokal übernehmen können (ich habs gerade mal angekuckt). > > Ich werde ihn nur nicht bei OpenWrt backporten, weil das eine kosmetische Änderung ist. Das hatte ich auch gar nicht erwartet, Umbenennungen innerhalb des gleichen Major Releases sind imho sehr eklig. Das kommt wenn dann mit in unser build_patches mit rein. Ich hab das schon mal passend gebaut, ich werfe das dann passend ein, wenn wir die ath79 Umstellung machen. > Einzig etwas eklig wird es bei den ramips Geräten, weil diese von 19.07 zu 20.xx umbenannt werden (z.B. auch tplink,c50-v4 zu tplink,archer-c50-v4). Das sollte uns aber nur am Rande stören. Wir werden diese Rewrites am Updateserver sowieso noch einige Zeit mit uns rum ziehen. Aber vielleicht hat das Chaos mit der neuen Syntax endlich mal ein Ende.. :-) Gruß Fabian
Je nachdem, wie du das Namensproblem mit dem C2600 lösen möchtest, würde ich den Namen für den WDR4900 auch noch vor dem Release anpassen: 1. Entweder man entschließt sich, den C2600 als tplink,c2600/tplink_c2600 zu behalten. Dann spricht eigentlich nichts dagegen, diesen Namen auch jetzt schon für den WDR4900 zu verwenden und die Board Name Anpassung zu entfernen. 2. Alternativ könnte man natürlich auch sagen, wir warten damit bis nach dem Release und machen für den c2600 einen vergleichbaren Patch zum WDR4900 im fff-boardname (also tplink,c2600 -> c2600). Dann bräuchte man eine Einzelanpassung fürs sysupgrade. Ich wäre für 1. in Verbindung mit dem WDR4900 Patch von OpenWrt als lokalen Backport. Dann haben wir das schon mal abgeräumt. Ich schicke dafür auch gerne einen Patch, aber vorher will ich wissen, welcher Patch jetzt für das Komma vs. Unterstrich Problem applied werden wird. Grüße Adrian
On 01.04.20 11:45, mail@adrianschmutzler.de wrote: > Je nachdem, wie du das Namensproblem mit dem C2600 lösen möchtest, würde ich den Namen für den WDR4900 auch noch vor dem Release anpassen: > > 1. Entweder man entschließt sich, den C2600 als tplink,c2600/tplink_c2600 zu behalten. Dann spricht eigentlich nichts dagegen, diesen Namen auch jetzt schon für den WDR4900 zu verwenden und die Board Name Anpassung zu entfernen. > > 2. Alternativ könnte man natürlich auch sagen, wir warten damit bis nach dem Release und machen für den c2600 einen vergleichbaren Patch zum WDR4900 im fff-boardname (also tplink,c2600 -> c2600). Dann bräuchte man eine Einzelanpassung fürs sysupgrade. > > > Ich wäre für 1. in Verbindung mit dem WDR4900 Patch von OpenWrt als lokalen Backport. Dann haben wir das schon mal abgeräumt. > > Ich schicke dafür auch gerne einen Patch, aber vorher will ich wissen, welcher Patch jetzt für das Komma vs. Unterstrich Problem applied werden wird. Ich bin hier definitiv für deine Lösung und damit auch (1.), meinen urspünglichen Patch habe ich im Patchwork auf Rejected gesetzt. Die Umbenennung vom 4900 ist aber schon im letzten Release drin gewesen, und wenn wir das jetzt ändern, braucht das wieder neue Rewrites auf dem Updateserver. Daher würde ich das lieber gerne gemeinsam mit dem ath79 machen, wo wir sowieso eine Reihe an Umbenennungen brauchen werden. Gruß Fabian
Because the upgrade filename is built based on fff-boardname's configuration, and there is no special treatment for the tplink c2600, the resulting filename contains 'tplink,c2600', which is not the correct filename. This patch sets the boardname explicitly, so the correct filename is constructed in fff-upgrade, which also requires renaming the fff-network file for this device. Signed-off-by: Fabian Bläse <fabian@blaese.de> --- src/packages/fff/fff-boardname/Makefile | 2 +- .../fff/fff-boardname/files/etc/uci-defaults/15-fff-boardname | 3 +++ src/packages/fff/fff-network/Makefile | 2 +- .../ipq806x/{network.tplink,c2600 => network.tplink_c2600} | 0 4 files changed, 5 insertions(+), 2 deletions(-) rename src/packages/fff/fff-network/ipq806x/{network.tplink,c2600 => network.tplink_c2600} (100%)