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

[robert]

Hallo Fabian,

ich vermute mal, weil uniq ohne sort nicht funzt.

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/7fb9222c/attachment.sig>