[PATCH v2 1/2] fff-babeld: Move common babeld procedures into functions

[Adrian Schmutzler]

Hallo Fabian,

gefällt mir konzeptionell ganz gut.

Für ein Review müsste ich das erstmal mit meinem Code zusammenpfriemeln, dafür habe ich gerade keine Zeit. Sehe aber kein Problem, das mit dem vorhandenen Review zu mergen.

Ein Sache ist mir aber noch aufgefallen:

> - 
> -             # peer_ip6 
> -             if peer_ip6=$(uci -q get gateway. at gateway[0].peer_ip6); then 
> -                     uci set network.$name.ip6addr="$peer_ip6" 
> -             fi 
> +             uci -q delete "network.$name.ipaddr" 
> +             uci -q delete "network.$name.ip6addr" 
> +             babel_add_peeraddr "network.$name.ipaddr" 
> +             babel_add_peer6addr "network.$name.ip6addr" 

Während der Rest deines Patches kosmetisch ist, werden hier die beiden "uci delete" hinzugefügt. Dies ist (wenn ich nichts übersehen habe) die einzige funktionale Änderung im Code. Diese sollte daher separat stattfinden.

Grüße

Adrian
-------------- 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/20190701/8b1349b2/attachment.sig>