[PATCH 1/4] buildscript: Add variant variable, store variant in release-file

[mail at adrianschmutzler.de]

Hallo,

> -----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 1/4] buildscript: Add variant variable, store variant in
> release-file

Hier bitte klarmachen, dass /etc/firmware_release gemeint ist und nicht release.nfo.

Ich hatte mich erst gewundert.

Wenn das Einführen der Variable so wichtig ist, sollte es in einen eigenen Patch. Ich würde aber eher im Titel nur auf das Einführen in /etc/firmware_release hinweisen, und die Variable in der Commit Message vergraben.

> 
> The variant is read multiple times from selected_variant file.
> Therefore a variant variable is introduced.
> 
> Signed-off-by: Fabian Bläse <fabian at blaese.de>
> ---
>  buildscript | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/buildscript b/buildscript
> index 00f1874..0920b74 100755
> --- a/buildscript
> +++ b/buildscript
> @@ -200,16 +200,18 @@ prebuild() {
>      done < <(find "${builddir}/files" -name '*.tpl' -print0)
> 
>      #insert actual firware version informations into release file
> +    variant=$(cat selected_variant)
>      version=$(git describe --tags --dirty)
>      if [ 0 -ne $? ]; then
>          version=$(git log -1 --pretty=format:%h)
>      fi
> -    if [ -n "$(cat selected_variant)" ]; then
> -        version="$(cat selected_variant)-$version"
> +    if [ -n "$variant" ]; then
> +        version="$variant-$version"
>      fi
> 
>      {
>          echo "FIRMWARE_VERSION=\"$version\""

Hier steckt die variant nach der Änderung schon mit drin...

> +        echo "VARIANT=\"$variant\""

... also ist das eigentlich doppelt gemoppelt. Man kann die Variante also ohne Probleme zur Laufzeit auf dem Router mit ${FIRMWARE_VERSION%%-*} generieren, wo man sie braucht. So mache ich das in meiner Firmware mit den Varianten "adsc", "jubtl" usw. bisher.

Ich würde auf das zusätzliche VARIANT verzichten.

Grüße

Adrian

>          echo "BUILD_DATE=\"build date: $(date)\""
>          echo "OPENWRT_CORE_REVISION=\"${OPENWRTREV}\""
>          echo "OPENWRT_FEEDS_PACKAGES_REVISION=\"${PACKAGEREV}\""
> --
> 2.24.0
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : openpgp-digital-signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20191120/ca007dc7/attachment.sig>