| Message ID | 20190203171345.1837-1-freifunk@adrianschmutzler.de |
|---|---|
| State | Accepted |
| Headers | show |
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}"
Hier fehlt die Commit Message: "This is just meant to make image names shorter." > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf > Of Adrian Schmutzler > Sent: Sonntag, 3. Februar 2019 18:14 > To: franken-dev@freifunk.net > Subject: [PATCH] build: Remove "squashfs" from image names > > Signed-off-by: Adrian Schmutzler <freifunk@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
hi müsste glaub ich passen bin aber dabei drüber gestolpert das wir hier ein Problem bekommen: https://github.com/FreifunkFranken/firmware/commit/862f3cdf4109b8c5e1a1b85e337577fa5aae7884 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@chrisi01.de> Gruß Christian On 03.02.19 18:14, mail@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@freifunk.net] On Behalf >> Of Adrian Schmutzler >> Sent: Sonntag, 3. Februar 2019 18:14 >> To: franken-dev@freifunk.net >> Subject: [PATCH] build: Remove "squashfs" from image names >> >> Signed-off-by: Adrian Schmutzler <freifunk@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
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@chrisi01.de] > Sent: Sonntag, 10. Februar 2019 17:25 > To: mail@adrianschmutzler.de; franken-dev@freifunk.net; 'Adrian > Schmutzler' <freifunk@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@chrisi01.de> > > Gruß > > Christian > > On 03.02.19 18:14, mail@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@freifunk.net] On > Behalf > >> Of Adrian Schmutzler > >> Sent: Sonntag, 3. Februar 2019 18:14 > >> To: franken-dev@freifunk.net > >> Subject: [PATCH] build: Remove "squashfs" from image names > >> > >> Signed-off-by: Adrian Schmutzler <freifunk@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
Applied. > -----Original Message----- > From: Christian Dresel [mailto:fff@chrisi01.de] > Sent: Sonntag, 10. Februar 2019 17:25 > To: mail@adrianschmutzler.de; franken-dev@freifunk.net; 'Adrian > Schmutzler' <freifunk@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@chrisi01.de> > > Gruß > > Christian > > On 03.02.19 18:14, mail@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@freifunk.net] On > Behalf > >> Of Adrian Schmutzler > >> Sent: Sonntag, 3. Februar 2019 18:14 > >> To: franken-dev@freifunk.net > >> Subject: [PATCH] build: Remove "squashfs" from image names > >> > >> Signed-off-by: Adrian Schmutzler <freifunk@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
Signed-off-by: Adrian Schmutzler <freifunk@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(-)