[PATCH 3/4] sysupgrade: Adjust sysupgrade for changed firmware binary structure

Fabian Bläse fabian at blaese.de
Mi Nov 20 17:17:56 CET 2019


On 20.11.19 00:29, mail at adrianschmutzler.de wrote:
> Hi,
> 
>> -----Original Message-----
>> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
>> Of Fabian Bläse
>> Sent: Dienstag, 19. November 2019 21:20
>> To: franken-dev at freifunk.net
>> Subject: [PATCH 3/4] sysupgrade: Adjust sysupgrade for changed firmware
>> binary structure
> 
> "folder structure" statt "binary structure"?
Jo, klingt besser.

>> As we now have multiple variants, the binaries on the update server should
>> be seperated into different directories.
>>
>> This ajusts sysupgrade.sh for this updated file structure.
> 
> "ajusts" -> "adjusts"
Jo.
Beides Dinge fürs applien, falls sonst nichts mehr ist.

>> Signed-off-by: Fabian Bläse <fabian at blaese.de>
>> ---
>>  src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh
>> b/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh
>> index 2b749f3..6133966 100755
>> --- a/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh
>> +++ b/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh
>> @@ -12,6 +12,9 @@ if [ -z "$UPGRADE_PATH" ]; then
>>    exit 1
>>  fi
>>
>> +#add firmware path to UPGRADE_PATH
> 
> Hier könnte man jetzt entsprechend meinem Vorschlag von 1/4 einfach
> 
> VARIANT="${FIRMWARE_VERSION%%-*}"
> 
> hinschreiben.
> 
> Siehe auch
> https://github.com/adrianschmutzler/fff-firmware/blob/mainline/src/packages/fff/fff-adsc/files/etc/adsc.cfg
Könnte man, dann hat man aber wieder das rumgepfusche mit Strings, die an einem (eigentlich gar nicht so wirklich definiertem Zeichen) getrennt werden.
Kann man schon so machen, aber wenn sichs vermeiden lässt..

Eher könnte ich mir noch vorstellen, falls dir die Redundanz nicht gefällt, die Variante nicht in FIRMWARE_VERSION zu haben.
(Und dann den nodewatcher anzupassen, entweder indem der variant-version String wieder zusammen gebaut wird, oder als eigenes Feld.

Gruß
Fabian

-------------- 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/20191120/a4ff2bd7/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev