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

[Christian Dresel]

hi

ich bin aktuell wegen etwas Zeitmangel nur am überfliegen, ich hab mir
den Code nicht weiter angeguckt aber auf Fragen antworte ich mal eben
gerne ;)

On 21.03.19 14:17, Fabian Bläse wrote:
> 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?

ich würde schon gerne die fdff:: behalten. "Später" will man vllt. mal
die Konfiguration des Gateways von SSH ins WebUI übertragen und WebUI
per fe80 geht bekanntlich nicht.

Gruß

Christian

>
> Gruß
> Fabian
>