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

Fabian Bläse fabian at blaese.de
So Jun 24 10:23:19 CEST 2018


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: Message signed with OpenPGP
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20180624/b1b97489/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev