* [PATCH] Fix error message generated in routel script when have 'multicase' in routing list
@ 2021-03-15 8:30 B.Arenfeld
2021-03-17 6:07 ` Stephen Hemminger
0 siblings, 1 reply; 2+ messages in thread
From: B.Arenfeld @ 2021-03-15 8:30 UTC (permalink / raw
To: netdev
Hello :-)
I'm on an machine with Manjaro Linux.
I get some error messages from shift command in the routel script:
/usr/bin/routel: Zeile 48: shift: shift count out of range
Reason are lines from "ip route list table 0" starting with word "multicast" :
multicast ff00::/8 dev enp2s0 table local proto kernel metric 256 pref medium
I added the "multicast" word in routel script and everything is fine ;-)
Greetings
Burkhard Arenfeld
Signed-off-by: Burkhard Arenfeld <b.arenfeld@arcor.de>
--- routel.orig 2021-03-15 08:23:24.706677247 +0100
+++ routel 2021-03-15 08:23:20.293589911 +0100
@@ -25,7 +25,7 @@ ip route list table "$@" |
src=""
table=""
case $network in
- broadcast|local|unreachable) via=$network
+ broadcast|local|multicast|unreachable) via=$network
network=$1
shift
;;
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Fix error message generated in routel script when have 'multicase' in routing list
2021-03-15 8:30 [PATCH] Fix error message generated in routel script when have 'multicase' in routing list B.Arenfeld
@ 2021-03-17 6:07 ` Stephen Hemminger
0 siblings, 0 replies; 2+ messages in thread
From: Stephen Hemminger @ 2021-03-17 6:07 UTC (permalink / raw
To: B.Arenfeld; +Cc: netdev
On Mon, 15 Mar 2021 09:30:07 +0100
"B.Arenfeld" <b.arenfeld@arcor.de> wrote:
> Hello :-)
>
> I'm on an machine with Manjaro Linux.
> I get some error messages from shift command in the routel script:
> /usr/bin/routel: Zeile 48: shift: shift count out of range
>
> Reason are lines from "ip route list table 0" starting with word "multicast" :
>
> multicast ff00::/8 dev enp2s0 table local proto kernel metric 256 pref medium
>
> I added the "multicast" word in routel script and everything is fine ;-)
>
> Greetings
>
> Burkhard Arenfeld
>
>
> Signed-off-by: Burkhard Arenfeld <b.arenfeld@arcor.de>
>
> --- routel.orig 2021-03-15 08:23:24.706677247 +0100
> +++ routel 2021-03-15 08:23:20.293589911 +0100
> @@ -25,7 +25,7 @@ ip route list table "$@" |
> src=""
> table=""
> case $network in
> - broadcast|local|unreachable) via=$network
> + broadcast|local|multicast|unreachable) via=$network
> network=$1
> shift
> ;;
>
>
There are several patches which make it impossible to apply.
Serious: the patch doesn't apply the current version of ip/routel in git
has the same line already there? Not sure where you found the original but
the version in git was imported back in 2015 (from Bitkeeper) and already
has the same line. Did you get this from some vendor supplied out of date
distro?
Minor: the patch was not based at the normal place in the directory
tree. The standard practice is to do this at directory of the source tree.
In this case it would be in the iproute2 source directory. In that directory
the routel script is in ip sub directory.
Trivial: checkpatch whines that the name in From and Signed-off-by aren't
exact match. But that is just noise, and it looks ok.
Please compare with current iproute2 in git and see if your patch was backwards?
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-03-17 6:08 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-03-15 8:30 [PATCH] Fix error message generated in routel script when have 'multicase' in routing list B.Arenfeld
2021-03-17 6:07 ` Stephen Hemminger
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.