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

[Adrian Schmutzler]

Hallo Tim,

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
> Of Tim Niemeyer
> Sent: Montag, 25. Juni 2018 08:03
> To: Adrian Schmutzler <freifunk at adrianschmutzler.de>; franken-
> dev at freifunk.net
> Subject: Re: [PATCH v2] fff-hoods: Don't sort when getting hood file from
> ethernet
> 
> 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.

Genau, ich habe die in meinen Augen "unnötigen" Befehle entfernt. Wenn ihr das nicht wollt, ist es mir in dem Fall ziemlich egal, der Patch war ja jetzt nicht direkt aufwändig.

Grüße

Adrian

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