| Message ID | 1529153900-2609-1-git-send-email-freifunk@adrianschmutzler.de |
|---|---|
| State | Superseded |
| 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..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
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@blaese.de> Fabian > On 16. Jun 2018, at 14:58, Adrian Schmutzler <freifunk@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@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 >
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@blaese.de> > > Fabian > >> On 16. Jun 2018, at 14:58, Adrian Schmutzler <freifunk@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@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 >>
Hallo Robert, .. ähm. Hust. Ja. Ich muss mein Review wohl zurücknehmen.. Fabian > On 24. Jun 2018, at 10:49, robert <rlanghammer@web.de> wrote: > > Hallo Fabian, > > ich vermute mal, weil uniq ohne sort nicht funzt. > > Robert
Hi, es sollte echt egal sein ob sort -u drin ist oder nicht. Sobald ein Hoodfile da ist, fliegt man aus der function. Und die link quality zwischen Ethernet verkabelten Routern sollte auch egal sein. Wenn nicht, ist auf Layer 1 was kaputt. Hier ist das grep nach DUP unschoener. 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@blaese.de> > > Fabian > >> On 16. Jun 2018, at 14:58, Adrian Schmutzler <freifunk@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@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 >>
Moin, ist es auch. An sich wäre eine Lösung ohne sort nicht verkehrt gewesen, wenn das sort nicht unbedingt nötig wäre. Aber so würde ich es dann doch einfach so lassen. Gruß Fabian > On 24. Jun 2018, at 11:20, robert <rlanghammer@web.de> wrote: > > Hi, > > es sollte echt egal sein ob sort -u drin ist oder nicht. Sobald ein > Hoodfile da ist, fliegt man aus der function. Und die link quality > zwischen Ethernet verkabelten Routern sollte auch egal sein. Wenn nicht, > ist auf Layer 1 was kaputt. > > Hier ist das grep nach DUP unschoener. > > Robert
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> --- src/packages/fff/fff-hoods/files/usr/lib/functions/fff/hoodfile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)