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

robert rlanghammer at web.de
Di Jun 25 14:08:22 CEST 2019


Hi Fabian,

die beiden exit 1 nimmst du ja raus. Bei den anderen wäre ein echo
"irgendwas" schön.  Es hat allerdings nichts mit der Funktionalität zu
tun. Darum schon mal ein

Reviewed-by: Robert Langhammer <rlanghammer at web.de>

Am 23.06.19 um 12:27 schrieb Fabian Bläse:
> Hallo,
>
> On 22.06.19 21:51, Fabian Bläse wrote:
>> diff --git a/src/packages/fff/fff-babeld/files/etc/gateway.d/40-babel b/src/packages/fff/fff-babeld/files/etc/gateway.d/40-babel
>> index cc1cf5d..4c94a12 100644
>> --- a/src/packages/fff/fff-babeld/files/etc/gateway.d/40-babel
>> +++ b/src/packages/fff/fff-babeld/files/etc/gateway.d/40-babel
>> @@ -1,4 +1,5 @@
>>  . /lib/functions.sh
>> +. /lib/functions/fff/babel
>>  
>>  #load board specific properties
>>  BOARD="$(uci get board.model.name)"
>> @@ -14,10 +15,9 @@ configure() {
>>  				# remove interface
>>  				uci -q del network.$name
>>  				# remove iif-rules
>> -				uci -q del network.${name}_rule
>> -				uci -q del network.${name}_rule6
>> +				babel_delete_iifrules "$name" || exit 1
> hier
>
>>  				# remove babel interface
>> -				uci -q del babeld.$name
>> +				babel_delete_interface "$name" || exit 1
> und hier darf kein "|| exit 1" stehen, da es kein Fehler ist, wenn die Konfiguration bereits nicht existiert.
>
> 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/20190625/3c5816f0/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev