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

[Tim Niemeyer]

Hi

Am 24. Juni 2018 23:21:15 MESZ schrieb Adrian Schmutzler <freifunk at adrianschmutzler.de>:
>There is no reason to sort MAC adresses, actually it is even
>making things worse as the original order might reflect the
>link quality.

Das wurde doch schon diskutiert und der Stand ist, dass die Link-Quality egal sein sollte. Habe da bisher kein anderes Argument gehört.


>Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
>
>---
>
>Changes in v2:
>- Also removed uniq
>---
> 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..c81ed23 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)

Du hast jetzt das uniq raus geworfen und somit werden im schlechtfall alle zweimal abgefragt, was mMn nicht gewollt ist.

Tim


> 
> 	for addr in $neighbor_addrs; do
>		wget -T2 -t1 -qO "$file"
>"http://[${addr}%${ethmesh_dev}]:2342/keyxchangev2data" && return 0