[PATCH] babeld: Automatically add redistribute filters

[Fabian Bläse]

Hey Robert,

On 14.08.19 12:40, robert wrote:
> Am 14.08.19 um 01:00 schrieb Fabian Bläse:
>> diff --git a/src/packages/fff/fff-babeld/files/lib/functions/fff/babel b/src/packages/fff/fff-babeld/files/lib/functions/fff/babel
>> index 0d19cef..5eb679b 100644
>> --- a/src/packages/fff/fff-babeld/files/lib/functions/fff/babel
>> +++ b/src/packages/fff/fff-babeld/files/lib/functions/fff/babel
>> @@ -86,3 +86,36 @@ babel_delete_interface() {
>>  
>>  	return 0
>>  }
>> +
>> +babel_add_redistribute_filter() {
>> +	[ "$#" -ne "1" ] && return 1
>> +
>> +	local prefix="$1"
>> +
>> +	config=$(uci add babeld filter)
>> +	uci set babeld.$config.type='redistribute'
>> +	uci set babeld.$config.ip="$prefix"
>> +	uci set babeld.$config.addedbyautoconfig='true'
>> +
>> +	return 0
>> +}
>> +
>> +babel_remove_custom_redistribute_filters() {
>> +	[ "$#" -ne "0" ] && return 1
> ich denke der Test ist nicht notwendig und könnte weg. Schadet aber auch
> nix.
Sicher ist sicher. :-)

>> +
>> +	remove_filters() {
>> +		local name="$1"
>> +
>> +		# check if filter was added by configuregateway
>> +		if ! [ "$(uci -q get babeld.$name.addedbyautoconfig)" = 'true' ]; then
>> +			return
>> +		fi
> 
> Könnte man das nicht vereinfachen und einfach auf Existenz testen:
> 
> uci -q get babeld.$name.addedbyautoconfig || return 
> 
> oder wenigstens das NOT reinziehen [ a != b ] - liest sich irgendwie besser :-)Ist mir an sich egal. Ich würde es aber so wie es ist nehmen, sonst brauchts ne v2 oder jemand muss das beim applien ändern.
Außer es ist dir sehr wichtig.

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://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190929/5030ce3d/attachment.sig>