[PATCH] fastd: Don't set obsolete batman-adv option

Christian Dresel fff at chrisi01.de
So Nov 11 00:35:02 CET 2018


Hi

hab mich mal auf dumm gestellt und bei Gluon umgehört:

https://github.com/freifunk-gluon/gluon/commit/2adf67edad86
https://git.open-mesh.org/batman-adv.git/commit/a00797d8fa8fd1471e8be1ac23d506f76d866aaa

Wenn ich das richtig verstehe, ist das im neuen Batman-adv also schon
mit drinnen.

Reviewed-by: Christian Dresel <fff at chrisi01.de>

mfg

Christian

Am 11.11.18 um 00:08 schrieb Fabian Bläse:
> Hey,
> 
> Christian hat gerade entdeckt, dass es da schonmal einen Patch und Diskussion gab:
> "[Patch] Remove batman option no_rebroadcast"
> https://pw.freifunk-franken.de/patch/900/
> 
> a) Die Option wurde nie bei batman-adv upstream genommen. Beim gluon-batman gibt es einen passenden Patch. Diesen haben wir auch früher applied.
> b) Ich glaube nicht, dass es vollkommen eskaliert, aber es würde bei dem zentralen Kram bestimmt einiges an Rauschen sparen.
> 
> Man müsste den Patch eigentlich auch auf das aktuelle BatmanAdv übernehmen können.
> Gibt es auch für compat15 ein gluon BatmanAdv repo?
> 
> Fabian
> 
> On 10.11.18 23:55, Tim Niemeyer wrote:
>> Hi
>>
>> Am Samstag, den 10.11.2018, 23:16 +0100 schrieb Fabian Bläse:
>>> The no_rebroadcast option has been introduced by
>>> a gluon patch in batman-adv-legacy. Switching to
>>> upstream batman with fff-batman-adv has made this
>>> option obsolete, so it can safely be removed.
>> Oh.. 
>>
>> a) Heißt das, die Option gibt es nicht mehr?
>>
>> b) Heißt das auch, dass es kein adäquates äquivalent zu dem Patch gibt?
>>
>> Wenn a): Reviewed-by: Tim Niemeyer <tim at tn-x.org>
>> Wenn b): Dann muss da eine Lösung her, weil es sonst einfach eskalieren
>> wird.
>>
>> Tim
>>
>>>
>>> Signed-off-by: Fabian Bläse <fabian at blaese.de>
>>> ---
>>>  src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd | 1 -
>>>  1 file changed, 1 deletion(-)
>>>
>>> diff --git a/src/packages/fff/fff-fastd/files/etc/uci-
>>> defaults/55_fff-fastd b/src/packages/fff/fff-fastd/files/etc/uci-
>>> defaults/55_fff-fastd
>>> index 72970bd..d53eb43 100644
>>> --- a/src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd
>>> +++ b/src/packages/fff/fff-fastd/files/etc/uci-defaults/55_fff-fastd
>>> @@ -22,6 +22,5 @@ EOF
>>>  ln -s /tmp/fastd_fff_peers /etc/fastd/fff/peers
>>>  echo "#!/bin/sh" > /etc/fastd/fff/up.sh
>>>  echo "ip link set up dev fffVPN" >> /etc/fastd/fff/up.sh
>>> -echo "echo enable >
>>> /sys/devices/virtual/net/fffVPN/batman_adv/no_rebroadcast" >>
>>> /etc/fastd/fff/up.sh
>>>  echo "batctl if add fffVPN" >> /etc/fastd/fff/up.sh
>>>  chmod +x /etc/fastd/fff/up.sh
> 

-------------- 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         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20181111/64a5e4f0/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev