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

[mayosemmel]

Hallo Steffen

Am Montag, den 14.03.2016, 19:50 +0100 schrieb Steffen Pankratz:
> 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.
> 
Ich habe dies mit Absicht so gelöst, auch wenn es nicht so schön
aussieht wie dein Vorschlag.
Ich bin selbst im first Level Support tätig und weiß wie es in der
Praxis läuft. Da kommen dann so Aussage wie "Ich weiß nicht was ich
gemacht habe und jetzt ist es kaputt".
Wenn du nicht explizit mit y|Y(Yes) oder j|J(Ja) antwortest, macht das
Script einfach nichts.
Das vermeidet versehentliches was auch immer.

Grüße Jan
> 
> Gruss
> -Steffen
> 

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 473 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20160314/61841615/attachment-0002.sig>