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

[Adrian Schmutzler]

Hallo Fabian,

 

kann ich mit leben, hängt für mich aber das Etikett „bitte ändern“ dran.

 

Reviewed-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>

 

Grüße

 

Adrian

 

From: Fabian Bläse [mailto:fabian at blaese.de] 
Sent: Mittwoch, 24. April 2019 10:26
To: Adrian Schmutzler <mail at adrianschmutzler.de>; franken-dev at freifunk.net
Subject: Re: [PATCH v3 2/3] gateway.d: Add configuration script for meta information

 

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 <mailto:system. at system[0].hostname=%22$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 HTML-Daten wurde abgetrennt...
URL: <https://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190424/318c675a/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         : <https://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190424/318c675a/attachment.sig>