alfred: Support interfaces IDs with more than two digits

Submitted by Adrian Schmutzler on May 21, 2018, 8:35 a.m.

Details

Message ID 000701d3f0de$a3b52500$eb1f6f00$@adrianschmutzler.de
State Superseded
Headers show

Commit Message

Adrian Schmutzler May 21, 2018, 8:35 a.m.
Hallo Robert,

 

danke für deinen Review.

 

Ich habe den Patch auch Upstream geschickt und dort ist er in einer besseren, allgemeineren Variante inzwischen akzeptiert:

 

https://github.com/openwrt-routing/packages/commit/135bc605b4cf91d40ac97f1129a6a5a950f3ff72

 

Sobald klar ist, wie es mit der Firmware weitergeht, würde ich den auch nochmal an die ML schicken.

 

Wenn man 18.06 nimmt, ist er bereits mit im routing Feed enthalten.

 

In der aktuellen Variante im Patchwork funktioniert er aber auch tadellos.

 

Beste Grüße

 

Adrian

 

From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of robert
Sent: Montag, 21. Mai 2018 09:25
To: franken-dev@freifunk.net
Subject: Re: [PATCH] alfred: Support interfaces IDs with more than two digits

 

Hi Adrian,

da der Interface Index tatsächlich auch mal 3stellig sein kann, ist der Patch notwendig.

Ich persönlich fände /^fe80.{37,38} [012389ab]/ schöner, ist aber egal.

Reviewed-by: Robert Langhgammer  <mailto:rlanghammer@web.de> <rlanghammer@web.de>
 
 

 

Am 19.03.2018 um 16:49 schrieb Adrian Schmutzler:

Signed-off-by: Adrian Schmutzler  <mailto:freifunk@adrianschmutzler.de> <freifunk@adrianschmutzler.de>
 
Tested-by: Adrian Schmutzler  <mailto:freifunk@adrianschmutzler.de> <freifunk@adrianschmutzler.de>
---
 ...ort-interface-IDs-with-more-than-two-digi.patch | 26 ++++++++++++++++++++++
 buildscript                                        |  3 ++-
 2 files changed, 28 insertions(+), 1 deletion(-)
 create mode 100644 build_patches/routing/0001-alfred-Support-interface-IDs-with-more-than-two-digi.patch

Patch hide | download patch | download mbox

diff --git a/build_patches/routing/0001-alfred-Support-interface-IDs-with-more-than-two-digi.patch b/build_patches/routing/0001-alfred-Support-interface-IDs-with-more-than-two-digi.patch
new file mode 100644
index 0000000..ce68d2f
--- /dev/null
+++ b/build_patches/routing/0001-alfred-Support-interface-IDs-with-more-than-two-digi.patch
@@ -0,0 +1,26 @@ 
+From b6da43f0f7ef835c994e20d20a42507606505f30 Mon Sep 17 00:00:00 2001
+From: Adrian Schmutzler  <mailto:freifunk@adrianschmutzler.de> <freifunk@adrianschmutzler.de>
+Date: Mon, 19 Mar 2018 14:16:54 +0100
+Subject: [PATCH] alfred: Support interface IDs with more than two digits
+
+Signed-off-by: Adrian Schmutzler  <mailto:freifunk@adrianschmutzler.de> <freifunk@adrianschmutzler.de>
+---
+ alfred/files/alfred.init | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/alfred/files/alfred.init b/alfred/files/alfred.init
+index 4c9a9e1..3b5fb3d 100755
+--- a/alfred/files/alfred.init
++++ b/alfred/files/alfred.init
+@@ -50,7 +50,7 @@ wait_for_ll_address()
+               # - on interface $iface
+               if awk '
+                       BEGIN { RET=1 }
+-                      /^fe80.{37} [012389ab]/ { if ($6 == "'"$iface"'") RET=0 }
++                      /^fe80.{37}[0-9a-f]? [012389ab]/ { if ($6 == "'"$iface"'") RET=0 }
+                       END { exit RET }
+               ' /proc/net/if_inet6; then
+                       return
+-- 
+2.7.4
+
diff --git a/buildscript b/buildscript
index 7abab0e..9c3cbb0 100755
--- a/buildscript
+++ b/buildscript
@@ -38,7 +38,8 @@  GLUON_PKGS="kmod-batman-adv-legacy micrond simple-tc uradvd"
 #official openwrt routing packages
 ROUTING=(routing
          https://git.openwrt.org/feed/routing.git
-         d11075cd40a88602bf4ba2b275f72100ddcb4767) # lede-17.01
+         d11075cd40a88602bf4ba2b275f72100ddcb4767
+         "0001-alfred-Support-interface-IDs-with-more-than-two-digi.patch") # lede-17.01
 ROUTING_PKGS="kmod-batman-adv batctl alfred"
 
 FFF=(fff)

Comments

Fabian Blaese June 13, 2018, 7:53 a.m.
Hallo Adrian,

auch das sollte man wohl backporten, bis wir auf ein aktuelleres OpenWRT wechseln.
Immerhin ist das wohl der Grund, warum Alfred ganz manchmal nicht startet, wenn ich das richtig sehe?

Gruß
Fabian

> On 21. May 2018, at 10:35, Adrian Schmutzler <mail@adrianschmutzler.de> wrote:
> 
> Hallo Robert,
> 
> danke für deinen Review.
> 
> Ich habe den Patch auch Upstream geschickt und dort ist er in einer besseren, allgemeineren Variante inzwischen akzeptiert:
> 
> https://github.com/openwrt-routing/packages/commit/135bc605b4cf91d40ac97f1129a6a5a950f3ff72 <https://github.com/openwrt-routing/packages/commit/135bc605b4cf91d40ac97f1129a6a5a950f3ff72>
> 
> Sobald klar ist, wie es mit der Firmware weitergeht, würde ich den auch nochmal an die ML schicken.
> 
> Wenn man 18.06 nimmt, ist er bereits mit im routing Feed enthalten.
> 
> In der aktuellen Variante im Patchwork funktioniert er aber auch tadellos.
> 
> Beste Grüße
> 
> Adrian
> 
> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of robert
> Sent: Montag, 21. Mai 2018 09:25
> To: franken-dev@freifunk.net
> Subject: Re: [PATCH] alfred: Support interfaces IDs with more than two digits
> 
> Hi Adrian,
> 
> da der Interface Index tatsächlich auch mal 3stellig sein kann, ist der Patch notwendig.
> 
> Ich persönlich fände /^fe80.{37,38} [012389ab]/ schöner, ist aber egal.
> 
> Reviewed-by: Robert Langhgammer <rlanghammer@web.de> <mailto:rlanghammer@web.de>
> 
> 
> 
> Am 19.03.2018 um 16:49 schrieb Adrian Schmutzler:
>> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> <mailto:freifunk@adrianschmutzler.de>
>> 
>> Tested-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> <mailto:freifunk@adrianschmutzler.de>
>> ---
>>  ...ort-interface-IDs-with-more-than-two-digi.patch | 26 ++++++++++++++++++++++
>>  buildscript                                        |  3 ++-
>>  2 files changed, 28 insertions(+), 1 deletion(-)
>>  create mode 100644 build_patches/routing/0001-alfred-Support-interface-IDs-with-more-than-two-digi.patch
>> 
>> diff --git a/build_patches/routing/0001-alfred-Support-interface-IDs-with-more-than-two-digi.patch b/build_patches/routing/0001-alfred-Support-interface-IDs-with-more-than-two-digi.patch
>> new file mode 100644
>> index 0000000..ce68d2f
>> --- /dev/null
>> +++ b/build_patches/routing/0001-alfred-Support-interface-IDs-with-more-than-two-digi.patch
>> @@ -0,0 +1,26 @@
>> +From b6da43f0f7ef835c994e20d20a42507606505f30 Mon Sep 17 00:00:00 2001
>> +From: Adrian Schmutzler <freifunk@adrianschmutzler.de> <mailto:freifunk@adrianschmutzler.de>
>> +Date: Mon, 19 Mar 2018 14:16:54 +0100
>> +Subject: [PATCH] alfred: Support interface IDs with more than two digits
>> +
>> +Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> <mailto:freifunk@adrianschmutzler.de>
>> +---
>> + alfred/files/alfred.init | 2 +-
>> + 1 file changed, 1 insertion(+), 1 deletion(-)
>> +
>> +diff --git a/alfred/files/alfred.init b/alfred/files/alfred.init
>> +index 4c9a9e1..3b5fb3d 100755
>> +--- a/alfred/files/alfred.init
>> ++++ b/alfred/files/alfred.init
>> +@@ -50,7 +50,7 @@ wait_for_ll_address()
>> +               # - on interface $iface
>> +               if awk '
>> +                       BEGIN { RET=1 }
>> +-                      /^fe80.{37} [012389ab]/ { if ($6 == "'"$iface"'") RET=0 }
>> ++                      /^fe80.{37}[0-9a-f]? [012389ab]/ { if ($6 == "'"$iface"'") RET=0 }
>> +                       END { exit RET }
>> +               ' /proc/net/if_inet6; then
>> +                       return
>> +--
>> +2.7.4
>> +
>> diff --git a/buildscript b/buildscript
>> index 7abab0e..9c3cbb0 100755
>> --- a/buildscript
>> +++ b/buildscript
>> @@ -38,7 +38,8 @@ GLUON_PKGS="kmod-batman-adv-legacy micrond simple-tc uradvd"
>>  #official openwrt routing packages
>>  ROUTING=(routing
>>           https://git.openwrt.org/feed/routing.git <https://git.openwrt.org/feed/routing.git>
>> -         d11075cd40a88602bf4ba2b275f72100ddcb4767) # lede-17.01
>> +         d11075cd40a88602bf4ba2b275f72100ddcb4767
>> +         "0001-alfred-Support-interface-IDs-with-more-than-two-digi.patch") # lede-17.01
>>  ROUTING_PKGS="kmod-batman-adv batctl alfred"
>> 
>>  FFF=(fff)
> 
> --
> franken-dev mailing list
> franken-dev@freifunk.net
> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net