[PATCH v3 2/3] gateway.d: Add configuration script for meta information

[Tim Niemeyer]

Hi

Am Dienstag, den 23.04.2019, 18:09 +0200 schrieb Fabian Bläse:
> This adds a configuration script for applying hostname,
> coordinates, hoodname and contact address.
> 
> Signed-off-by: Fabian Bläse <fabian at blaese.de>
> ---
> Changes in v3:
> - Use meta information from /etc/config/fff
> - rename 01-meta to 10-meta
> ---
>  .../fff-gateway/files/etc/gateway.d/10-meta   | 37
> +++++++++++++++++++
>  1 file changed, 37 insertions(+)
>  create mode 100644 src/packages/fff/fff-
> gateway/files/etc/gateway.d/10-meta
> 
> diff --git a/src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta 
> b/src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta
> new file mode 100644
> index 0000000..3b8c9d2
> --- /dev/null
> +++ b/src/packages/fff/fff-gateway/files/etc/gateway.d/10-meta
> @@ -0,0 +1,37 @@
> +configure() {
> +	## set hostname
> +	if name=$(uci -q get fff.system.hostname); then
> +		uci -q set system. at system[0].hostname="$name"
> +	else
> +		echo "WARNING: No hostname set!"
> +	fi
> +
> +	## set contact
> +	if ! contact=$(uci -q get fff.system.contact); then
> +		echo "WARNING: No contact set!"
> +	fi
> +
> +	## set location
> +	if ! lat=$(uci -q get fff.system.latitude) || ! long=$(uci
> -q get fff.system.longitude); then
> +		echo "WARNING: No location set!"
> +	fi
> +
> +	## hoodname
> +	if hood=$(uci -q get fff.system.hoodname); then
> +		echo "Setting hood name: $hood"
> +		uci -q set "system. at system[0].hood=$hood"
> +	else
> +		echo "WARNING: No hoodname set!"
> +	fi
Ich verstehe nicht, wofür wir zu dieser Zeit ein hoodname brauchen.
Auch im dritten Patch taucht das nicht auf.

Hostname, Kontakt, Location ergeben für mich noch Sinn, da man diese
vielleicht später mal in Richtung Monitoring kippen möchte. Aber
hoodname? Hat ja eigentlich nix mit dem Router zu tun.

Wenn da noch ein Patch in der Schublade schlummert, wo das dann
benötigt wird, wäre es sauberer das auch erst später hinzuzufügen. Ich
würde mich aber auch mit dieser unschönheit zufrieden geben, wenn das
einfach so in den Gesamtplan passt. Wenn da allerdings nichts mehr
kommt, dann sollte es mMn eher raus.

Tim

> +}
> +
> +apply() {
> +	uci commit system
> +	uci commit fff
> +	echo "$(uci get "system. at system[0].hostname")" >
> /proc/sys/kernel/hostname
> +}
> +
> +revert() {
> +	uci revert system
> +	uci revert fff
> +}
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: This is a digitally signed message part
URL         : <https://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190508/29a66ae5/attachment.sig>