[PATCH v2 2/2] show_info: Treat case of missing simple-tc package
Fabian Bläse
fabian at blaese.de
So Mär 15 16:38:49 CET 2020
Passt, wobei ich den Installationsstatus eher mit dem executable geprüft hätte, aber so ist das denke ich auch ok. Richtig sauber ist eh keine der beiden Möglichkeiten.
Reviewed-by: Fabian Bläse <fabian at blaese.de>
On 07.01.20 12:58, Adrian Schmutzler wrote:
> Suppress error in show_info when simple-tc is not built into the
> firmware.
>
> Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
>
> ---
>
> Changes in v2:
> Resend with added commit message.
> ---
> src/packages/fff/fff-support/Makefile | 2 +-
> .../fff/fff-support/files/usr/sbin/show_info | 14 +++++++++-----
> 2 files changed, 10 insertions(+), 6 deletions(-)
>
> diff --git a/src/packages/fff/fff-support/Makefile b/src/packages/fff/fff-support/Makefile
> index 7b23a959..76ca26c8 100644
> --- a/src/packages/fff/fff-support/Makefile
> +++ b/src/packages/fff/fff-support/Makefile
> @@ -1,7 +1,7 @@
> include $(TOPDIR)/rules.mk
>
> PKG_NAME:=fff-support
> -PKG_RELEASE:=4
> +PKG_RELEASE:=5
>
> PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME)
>
> diff --git a/src/packages/fff/fff-support/files/usr/sbin/show_info b/src/packages/fff/fff-support/files/usr/sbin/show_info
> index d3367aec..cd3b4d2e 100755
> --- a/src/packages/fff/fff-support/files/usr/sbin/show_info
> +++ b/src/packages/fff/fff-support/files/usr/sbin/show_info
> @@ -159,13 +159,17 @@ echo "E-Mail Adresse: $(uci -q get 'fff.system.contact')"
>
> echo ""
> echo "-> BITRATEN-BEGRENZUNG"
> -if [ "$(uci -q get 'simple-tc.example.enabled')" = "1" ] ; then
> - echo "Aktiv: ja"
> +if [ -s "/etc/config/simple-tc" ]; then
> + if [ "$(uci -q get 'simple-tc.example.enabled')" = "1" ] ; then
> + echo "Aktiv: ja"
> + else
> + echo "Aktiv: nein"
> + fi
> + echo "Download-Limit: $(uci -q get 'simple-tc.example.limit_ingress') kBit/s"
> + echo "Upload-Limit: $(uci -q get 'simple-tc.example.limit_egress') kBit/s"
> else
> - echo "Aktiv: nein"
> + echo "simple-tc nicht installiert."
> fi
> -echo "Download-Limit: $(uci -q get 'simple-tc.example.limit_ingress') kBit/s"
> -echo "Upload-Limit: $(uci -q get 'simple-tc.example.limit_egress') kBit/s"
>
> if grep -q 'PORTORDER=' /etc/network.$board_name ; then
> portorder=$(awk -F= '/PORTORDER=/ { print $2 }' /etc/network.$board_name | sed "s/\"//g")
>
-------------- 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/20200315/40691dca/attachment.sig>
Mehr Informationen über die Mailingliste franken-dev