[PATCH] fff-sysupgrade: Fix comparison for version in sysupgrade.sh

Adrian Schmutzler mail at adrianschmutzler.de
Do Okt 25 16:39:32 CEST 2018


Applied.

 

From: Fabian Bläse [mailto:fabian at blaese.de] 
Sent: Mittwoch, 24. Oktober 2018 22:05
To: Adrian Schmutzler <freifunk at adrianschmutzler.de>; franken-dev at freifunk.net
Subject: Re: [PATCH] fff-sysupgrade: Fix comparison for version in sysupgrade.sh

 

Hallo Adrian, 

da unsere Versionsnummern definitiv nicht ausschließlich nummerisch sind, sollte das auf jeden Fall rein. 

Reviewed-by: Fabian Bläse <fabian at blaese.de <mailto:fabian at blaese.de> > 

Fabian 

On 14.09.18 23:49, Adrian Schmutzler wrote: 
> The previous version seemed to work only for numeric data. 
> 
> Fixes #113 
> 
> Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de <mailto:freifunk at adrianschmutzler.de> > 
> --- 
>  src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh | 2 +- 
>  1 file changed, 1 insertion(+), 1 deletion(-) 
> 
> diff --git a/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh b/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh

> index 6b6c10af..3e4da41e 100755 
> --- a/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh 
> +++ b/src/packages/fff/fff-sysupgrade/files/etc/sysupgrade.sh 
> @@ -42,7 +42,7 @@ VERSION=$(awk -F: '/VERSION:/ { print $2 }' release.nfo) 
>  rm -f release.nfo 
>  echo "Firmware found on server: $VERSION" 
>  
> -if [ "$VERSION" -eq "$FIRMWARE_VERSION" ]; then 
> +if [ "$VERSION" = "$FIRMWARE_VERSION" ]; then 
>    echo "The installed firmware version is already the current version." 
>    echo "" 
>  
> 

 
-------------- nächster Teil --------------
Ein Dateianhang mit HTML-Daten wurde abgetrennt...
URL: <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20181025/3bee7707/attachment.html>
-------------- 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/20181025/3bee7707/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev