[PATCH] fff-hoods: Don't sort when getting hood file from ethernet

[robert]

Hi,

es sollte echt egal sein ob sort -u drin ist oder nicht. Sobald ein
Hoodfile da ist, fliegt man aus der function. Und die link quality
zwischen Ethernet verkabelten Routern sollte auch egal sein. Wenn nicht,
ist auf Layer 1 was kaputt.

Hier ist das grep nach DUP unschoener.

Robert


Am 24.06.2018 um 10:23 schrieb Fabian Bläse:
> Hi
>
> Ich hab jetzt über eine Woche drüber nachgedacht, warum ich das damals erst sortiert hab, und keinen Grund dafür gefunden.
> Sollte wohl weg.
>
> Reviewed-by: Fabian Bläse <fabian at blaese.de>
>
> Fabian
>
>> On 16. Jun 2018, at 14:58, Adrian Schmutzler <freifunk at adrianschmutzler.de> wrote:
>>
>> There is no reason to sort MAC adresses, actually it is even
>> making things worse as the original order might reflect the
>> link quality.
>>
>> Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
>> ---
>> src/packages/fff/fff-hoods/files/usr/lib/functions/fff/hoodfile | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/src/packages/fff/fff-hoods/files/usr/lib/functions/fff/hoodfile b/src/packages/fff/fff-hoods/files/usr/lib/functions/fff/hoodfile
>> index 8c80802..65eb77a 100644
>> --- a/src/packages/fff/fff-hoods/files/usr/lib/functions/fff/hoodfile
>> +++ b/src/packages/fff/fff-hoods/files/usr/lib/functions/fff/hoodfile
>> @@ -67,7 +67,7 @@ getEthernetHoodfile() {
>> 		return 1
>> 	fi
>>
>> -	neighbor_addrs=$(ping6 -c2 ff02::1%"${ethmesh_dev}" | grep seq | grep DUP | cut -d " " -f4 | sed s/:$//g | sort -u)
>> +	neighbor_addrs=$(ping6 -c2 ff02::1%"${ethmesh_dev}" | grep seq | grep DUP | cut -d " " -f4 | sed s/:$//g | uniq)
>>
>> 	for addr in $neighbor_addrs; do
>> 		wget -T2 -t1 -qO "$file" "http://[${addr}%${ethmesh_dev}]:2342/keyxchangev2data" && return 0
>> --
>> 2.7.4
>>


-------------- 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/20180624/84dfe9b9/attachment.sig>