[PATCH v7 2/2] Added Version check in sysupgrade

Steffen Pankratz kratz00 at gmx.de
Mo Mär 14 19:50:59 CET 2016


On Sun, 13 Mar 2016 22:04:55 +0100
Jan Kraus <mayosemmel at googlemail.com> wrote:

Hallo Jan

> Signed-off-by: Jan Kraus <mayosemmel at gmail.com>
> ---
>  bsp/default/root_file_system/etc/sysupgrade.sh | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
> 
> diff --git a/bsp/default/root_file_system/etc/sysupgrade.sh b/bsp/default/root_file_system/etc/sysupgrade.sh
> index a90164d..e2e51c5 100755
> --- a/bsp/default/root_file_system/etc/sysupgrade.sh
> +++ b/bsp/default/root_file_system/etc/sysupgrade.sh
> @@ -15,6 +15,22 @@ VERSION=$(grep -Po '(?<=^VERSION:).*' release.nfo)
>  rm -f release.nfo
>  echo -ne "Firmware found on server: $VERSION\n"
>  
> +if [ $VERSION -eq $FIRMWARE_VERSION ]; then
> +  echo -ne "The installed firmware version is already the current version.\n\n"
> +  echo -ne "Do you want to reinstall the current version? [y/n]\n"
> +  read DO_UPDATE
> +
> +  case $DO_UPDATE in
> +    [yY] | [jJ])
> +      #nothing to do here
> +      : 
> +      ;;
> +    *)
> +      exit 1
> +      ;;
> +  esac
> +fi
> +
>  FILE="fff-${FIRMWARE_COMMUNITY}_${VERSION}_${BOARD}-sysupgrade.bin"
>  echo -ne "Downloading $FILE\n\n"
>  wget $(uci get firmware.upgrade.path)/${FILE}

Warum pruefst du noch auf "jJ"?
Ich wuerde im default-Case eher was ausgeben, wie "Invalid input"
und nur explizit im "nN"-Case -> "exit 1" aufrufen.


Gruss
-Steffen

-- 
Hermes powered by Manjaro Linux (Linux 4.4.5)

Best regards, Steffen Pankratz.
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : nicht verfügbar
Dateityp    : application/pgp-signature
Dateigröße  : 181 bytes
Beschreibung: OpenPGP digital signature
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20160314/7286dc88/attachment-0002.sig>


Mehr Informationen über die Mailingliste franken-dev