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

[Fabian Bläse]

Hallo Adrian

On 24.04.19 00:13, Adrian Schmutzler wrote:
> Hallo Fabian,
> 
> da bist du mir ja weit entgegen gekommen.
War mir wie gesagt auch selbst ein Dorn im Auge..

>> +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
> 
> Anmerkung:
> So wie du das jetzt nutzt, könnte man auf die Variablenzuweisung verzichten und einfach nur das uci get prüfen.
> Würde ich aber für nen Review auch so akzeptieren.
Könnte man, ja.
Schaden tuts jetzt aber auch nicht.

>> +		echo "WARNING: No location set!"
>> +	fi
>> +
>> +	## hoodname
>> +	if hood=$(uci -q get fff.system.hoodname); then
> 
> Hmm, hostname und Location sind Eigenschaften des Routers/Gerät, aber der Hoodname ist eine Eigenschaft der Hood. Den hätte ich daher nicht nach /etc/config/fff getan.
> Der gehört meines Erachtens ins hoodfile. Da du ja aber meintest, du wolltest vom Hoodfile unabhängig sein, wäre mir hier dann /etc/config/gateway noch lieber. Am liebsten wäre mir aber, wenn das hier gar nicht steht, sondern man das irgendwo prüft, wo man Sachen macht, die mit der Hood zu tun haben.
> Wenn du das trotzdem hier machen willst, könnte ich mit /etc/config/gateway leben. Zur Not kann man das später ja auch nochmal rumschieben, wenn man das größere Bild besser sieht.
Bisher weiß ich keinen besseren Platz.
Das ist eigentlich auch nur für den Nodewatcher und das Monitoring.

Lass das erstmal so übernehmen, bis zu einem ersten Release können wir das immer noch rumschieben, wie wir wollen.

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://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190424/a6b14b3f/attachment.sig>