[PATCH] build: Remove "squashfs" from image names

mail at adrianschmutzler.de mail at adrianschmutzler.de
So Feb 10 17:43:43 CET 2019


Hallo Christian,

gut aufgepasst, aber:

Das Node-Prefix steht mit in der "VERSION" Variable im /etc/firmware_release.

Dieses wird beim sysupgrade gesourced:

https://github.com/FreifunkFranken/firmware/blob/master/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh#L5

Wir kriegen also über die "VERSION" Variable schon das komplette node-20190130 mit in den Dateinamen rein.

Und im release.nfo sollte es auch richtig stehen. Wir müssen also hier sysupgrade.sh nicht anpassen, weil alles über die VERSION-Variable läuft und bereits im Vorfeld ordentlich angelegt wurde.

Unabhängig davon wird die heruntergeladene Version ohnehin nur vom release.nfo bestimmt, die lokale Version wird ausschließlich beim Vergleich benutzt.

Ungeklärt ist lediglich, ob man dann ein zentrales sysupgrade nur für die node-Firmware machen kann. Das gibt dann Probleme bei der Gateway-Firmware. Das hat aber mit diesem Patch erstmal gar nichts zu tun, im Gegenteil, durch den Patch merkt der Nutzer dann zumindest, dass er ggf. die falsche Firmware herunterlädt.

Dieses Thema sollte man separat nochmal diskutieren:
- Braucht die GW-Firmware ein Update-Skript?
- Wenn nein: Wie stellt man sicher, dass niemand mit dem Update-Skript versehentlich node über layer3 updated?
- Wenn ja: Wie unterscheidet man zwischen node und layer3? Gibt es dann zwei verschiedene Server, Pfade, o.ä.? In der Praxis löst sich das wahrscheinlich dadurch, dass der Update-Server ja per Hoodfile gesetzt wird, d.h. zentral wird nur node-Firmware angeboten und GW-Firmware muss sich dann jeder selber kümmern? Was passiert, wenn kein Hoodfile in der GW-Firmware hinterlegt ist? Ich glaube, dann geht das Skript gar nicht, da kein Server da .............

Grüße

Adrian



> -----Original Message-----
> From: Christian Dresel [mailto:fff at chrisi01.de]
> Sent: Sonntag, 10. Februar 2019 17:25
> To: mail at adrianschmutzler.de; franken-dev at freifunk.net; 'Adrian
> Schmutzler' <freifunk at adrianschmutzler.de>
> Subject: Re: [PATCH] build: Remove "squashfs" from image names
> 
> hi
> 
> müsste glaub ich passen bin aber dabei drüber gestolpert das wir hier ein
> Problem bekommen:
> 
> https://github.com/FreifunkFranken/firmware/commit/862f3cdf4109b8c5e1
> a1b85e337577fa5aae7884
> 
> Da passt der Filename dann beim sysupgrade nicht mehr, hier in diesem
> Patch wurde es mit korrigiert.
> 
> Eine Node Firmware mit diesen Patch auf dem aktuellen Master aufgebaut
> will nun folgende File beim sysupgrade.sh holen:
> 
> fff-20181202-ar71xx-t-tl-wr1043nd-v2-sysupgrade.bin
> 
> RIchtig müsste aber sein:
> 
> fff-node-20181202-ar71xx-t-tl-wr1043nd-v2-sysupgrade.bin
> 
> Da muss das node noch mit rein.
> 
> Seh ich das soweit richtig? Da es dieses Patch aber eigentlich gar nicht betrifft
> hab ich das jetzt nur nebenbei erwähnt.
> 
> ----
> 
> Grundsätzlich begrüße ich es das squashfs rauszunehmen. Ich seh keinen
> Sinn was das jemand helfen soll das es im Filename mit enthalten ist.
> 
> Reviewed-by: Christian Dresel <fff at chrisi01.de>
> 
> Gruß
> 
> Christian
> 
> On 03.02.19 18:14, mail at adrianschmutzler.de wrote:
> > Hier fehlt die Commit Message:
> >
> > "This is just meant to make image names shorter."
> >
> >> -----Original Message-----
> >> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On
> Behalf
> >> Of Adrian Schmutzler
> >> Sent: Sonntag, 3. Februar 2019 18:14
> >> To: franken-dev at freifunk.net
> >> Subject: [PATCH] build: Remove "squashfs" from image names
> >>
> >> Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> >> ---
> >>  buildscript                                             | 1 +
> >>  src/packages/fff/fff-sysupgrade/Makefile                | 2 +-
> >>  src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh | 2 +-
> >>  3 files changed, 3 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/buildscript b/buildscript index 02cb6c4e..3ba36a6a
> >> 100755
> >> --- a/buildscript
> >> +++ b/buildscript
> >> @@ -285,6 +285,7 @@ cp_firmware() {
> >>          filename_build=${image//openwrt/fff-${version}}
> >>          filename_build=${filename_build//generic/g}
> >>          filename_build=${filename_build//tiny/t}
> >> +        filename_build=${filename_build//squashfs-/}
> >>          cp "$target/bin/targets/${chipset}/${subtarget}/$image"
> >> "./bin/$filename_build"
> >>
> >>          for region in "" "-eu" "-us"; do diff --git
> >> a/src/packages/fff/fff- sysupgrade/Makefile
> >> b/src/packages/fff/fff-sysupgrade/Makefile
> >> index 1f7b65eb..90ef66af 100644
> >> --- a/src/packages/fff/fff-sysupgrade/Makefile
> >> +++ b/src/packages/fff/fff-sysupgrade/Makefile
> >> @@ -1,7 +1,7 @@
> >>  include $(TOPDIR)/rules.mk
> >>
> >>  PKG_NAME:=fff-sysupgrade
> >> -PKG_VERSION:=8
> >> +PKG_VERSION:=9
> >>  PKG_RELEASE:=1
> >>
> >>  PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME)
> >> diff --git a/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh
> >> b/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh
> >> index 87ac48ab..c8d037ce 100755
> >> --- a/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh
> >> +++ b/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh
> >> @@ -69,7 +69,7 @@ if [ "$VERSION" = "$FIRMWARE_VERSION" ]; then
> >>    done
> >>  fi
> >>
> >> -FILE="fff-${VERSION}-${SOC}-${BOARD}-squashfs-sysupgrade.bin"
> >> +FILE="fff-${VERSION}-${SOC}-${BOARD}-sysupgrade.bin"
> >>  echo "Downloading $FILE"
> >>  echo ""
> >>  wget "${UPGRADE_PATH}/${FILE}"
> >> --
> >> 2.11.0
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : nicht verfügbar
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190210/a0bce116/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev