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

[Fabian Bläse]

Hallo Robert,

wie macht man denn sowas schön, dass man echo und exit gemeinsam hinter das || packt? Hast du da ne Idee?
Ansonsten würde ich das ist ein if verkapseln.

Gruß
Fabian

On 25.06.19 14:08, robert wrote:
> 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/d2e08d2b/attachment.sig>