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

mail at adrianschmutzler.de mail at adrianschmutzler.de
Fr Mai 31 22:26:09 CEST 2019


Hallo,

 

das mit dem Namen ist bei mir auch reine Willkür, mir geht es nur darum dass sie dann gleich heißen und man einfacher diffen kann (bei mir ist die package schon in der FW). Ich kann meine dazu aber auch umbenennen.

 

Zwecks DNS löschen sehe ich halt keinen Grund, warum man alle DNS Server entfernen können wollte. Das war aber nur ein Vorschlag, ist mit hier total egal.

 

Grüße

 

Adrian

 

From: Fabian Bläse [mailto:fabian at blaese.de] 
Sent: Freitag, 31. Mai 2019 18:27
To: mail at adrianschmutzler.de; franken-dev at freifunk.net
Subject: Re: [PATCH 6/6] fff-dhcp: Add configuration scripts for dns

 

Hallo Adrian, 

On 25.05.19 21:23, mail at adrianschmutzler.de <mailto:mail at adrianschmutzler.de>  wrote: 
> Hallo Fabian, 
> 
> Reviewed-by: Adrian Schmutzler <freifunk at adrianschmutzler.de <mailto: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 <mailto: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 <mailto:gateway. at dns[0].server>  > /dev/null; then 
>> +            for f in $(uci get gateway. at dns[0].server <mailto:gateway. at dns[0].server> ); do 
> 
> Das kann man schön zusammenfassen: 
> 
>       if dnsservers=$(uci -q get gateway. at dns[0].server <mailto: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 HTML-Daten wurde abgetrennt...
URL: <https://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190531/4c734ab7/attachment.html>
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : openpgp-digital-signature.asc
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/20190531/4c734ab7/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev