[PATCH] Fix tplink c2600 boardname for fff-upgrade

[Fabian Bläse]

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 at 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 at freifunk.net] On Behalf
>> Of Fabian Bläse
>> Sent: Dienstag, 31. März 2020 21:03
>> To: franken-dev at 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 at 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

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: OpenPGP digital signature
URL         : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20200331/c3541c91/attachment.sig>