[PATCH 4/5] fff-nodewatcher: Consolidate code in nodewatcher.d/10-systemdata.sh

Fabian Bläse fabian at blaese.de
Sa Apr 18 19:33:00 CEST 2020


Hallo Adrian,

On 07.01.20 14:40, Adrian Schmutzler wrote:
> diff --git a/src/packages/fff/fff-nodewatcher/files/usr/lib/nodewatcher.d/10-systemdata.sh b/src/packages/fff/fff-nodewatcher/files/usr/lib/nodewatcher.d/10-systemdata.sh
> index a61e1e57..f4a6980e 100755
> --- a/src/packages/fff/fff-nodewatcher/files/usr/lib/nodewatcher.d/10-systemdata.sh
> +++ b/src/packages/fff/fff-nodewatcher/files/usr/lib/nodewatcher.d/10-systemdata.sh
> [..]
> -batman_adv_version=$(cat /sys/module/batman_adv/version)
> -kernel_version=$(uname -r)
> -if [ -x /usr/bin/fastd ]; then
> -	fastd_version="<fastd_version>$(/usr/bin/fastd -v | awk '{ print $2 }')</fastd_version>"
> -fi
> -nodewatcher_version=$SCRIPT_VERSION
> -if [ -x /usr/sbin/babeld ]; then
> -	babel_version="<babel_version>$(/usr/sbin/babeld -V 2>&1)</babel_version>"
> -fi
> +SYSTEM_DATA="$SYSTEM_DATA<batman_advanced_version>$(cat /sys/module/batman_adv/version)</batman_advanced_version>"
batman_adv kann auch nicht vorhanden sein, das müsste analog zu babeld mit einem passenden if versehen werden.
Würde ich aber einfach als extra Patch senden.

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

Gruß
Fabian

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: OpenPGP digital signature
URL         : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20200418/93d4ca70/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev