[PATCH 6/6] fff-dhcp: Add configuration scripts for dns

[Fabian Bläse]

Hallo Adrian,

On 25.05.19 21:23, mail at adrianschmutzler.de wrote:
> Hallo Fabian,
> 
> Reviewed-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> 
> mit ein paar Vorschlägen unten.
> 
> Ich habe die Datei bei mir 40-dns genannt, wenn du Lust hast das anzugleichen wäre das praktisch für mich.
Warum?
Zugegeben, ich hab mir bei der Nummerierung nicht allzu viele Gedanken gemacht, aber warum 40?

>> diff --git a/src/packages/fff/fff-dhcp/files/etc/gateway.d/35-dns
>> b/src/packages/fff/fff-dhcp/files/etc/gateway.d/35-dns
>> new file mode 100644
>> index 0000000..8ffd440
>> --- /dev/null
>> +++ b/src/packages/fff/fff-dhcp/files/etc/gateway.d/35-dns
>> @@ -0,0 +1,21 @@
>> +configure() {
>> +	## dns
>> +	uci -q del dhcp. at dnsmasq[0].server
> 
> Das habe ich ins if gezogen, dann bleibt alles beim alten, wenn der Block in der config fehlt. Ist aber reine Geschmackssache.
Bei mir halt immer die Idee, dass alles weg kommt, was der Nutzer nicht explizit haben will.
Daher würde ich das so lassen.

>> +	if uci -q get gateway. at dns[0].server > /dev/null; then
>> +		for f in $(uci get gateway. at dns[0].server); do
> 
> Das kann man schön zusammenfassen:
> 
> 	if dnsservers=$(uci -q get gateway. at dns[0].server); then
> 		for f in $dnsservers; do
Jo. Ich mach dann ne v2.

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/20190531/6239b84d/attachment.sig>