| Message ID | 1529875275-38841-1-git-send-email-freifunk@adrianschmutzler.de |
|---|---|
| State | Rejected |
| Headers | show |
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) for addr in $neighbor_addrs; do wget -T2 -t1 -qO "$file" "http://[${addr}%${ethmesh_dev}]:2342/keyxchangev2data" && return 0
Hi Am 24. Juni 2018 23:21:15 MESZ schrieb Adrian Schmutzler <freifunk@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@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
Hallo Tim, > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf > Of Tim Niemeyer > Sent: Montag, 25. Juni 2018 08:03 > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken- > dev@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@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@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
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@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(-)