[PATCH v2 3/3] gateway.d: Add scripts for network configuration

[Fabian Bläse]

Hallo,

On 20.03.19 16:25, Adrian Schmutzler wrote:
>> +configure() {
>> +	# ipaddr
>> +	#remove old ipaddr
>> +	uci -q del network.mesh.ipaddr
>> +	#set new ipaddr
>> +	if ipaddr=$(uci -q get gateway. at client[0].ipaddr); then
>> +		for ip in $ipaddr; do
>> +			uci add_list network.mesh.ipaddr=$ip
>> +		done
>> +	else
>> +		echo "WARNING: No client ipaddr set!"
>> +	fi
> 
> Ich würde hier lieber ip4addr statt ipaddr lesen. Mindestens in der warning (ggf. auch direkt "IPv4 address"), man könnte aber auch überlegen, ob man es "jetzt noch" generell macht: Man könnte z.B. auch in der gateway-config ip4addr als option verwenden. Hat dann den Nachteil, dass es nicht konsistent mit der network-config ist, wäre aber ggf. einfacher für den Nutzer.
Ich tatsächlich auch. Aber ich würde da auf jeden Fall eher bei dem bleiben, wie es auch bei OpenWRT verwendet wird.
Die Fehlermeldung kann man anpassen.

> Wo wird in der GW-Firmware die fe80::1 denn gesetzt? Bei V2 ist es in der configurehood:
> https://github.com/adrianschmutzler/fff-firmware/blob/gwmod/src/packages/fff/fff-hoods/files/usr/sbin/configurehood#L57
Nirgends. Überrest von Früher.

Auch diese hart da reinkodierte fdff sollte wohl eher weg. Ich würde diesen ganzen Kram bei der Gatewayfirmware (inklusive configurenetwork Skript) bei der Gatewayfirmware komplett weglassen und lieber eine Standard-Konfiguration fürs Gateway mitbringen.
Initiale Konfiguration kann dann über fe80 oder so funktionieren.
Was meint ihr dazu?

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/20190321/751de889/attachment.sig>