| Message ID | 20230419154359.2656173-1-idosch@nvidia.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | ed328120f47af45f5dd2de2c4035ccaba4c22cf7 |
| Delegated to: | Stephen Hemminger |
| Headers | show |
| Series | [iproute2] ip: bridge_slave: Fix help message indentation | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Not a local patch |
On Wed, Apr 19, 2023 at 06:43:59PM +0300, Ido Schimmel wrote: > Use tabs instead of spaces to be consistent with the rest of the > options. > > Before: > > $ ip link help bridge_slave > Usage: ... bridge_slave [ fdb_flush ] > [...] > [ vlan_tunnel {on | off} ] > [ isolated {on | off} ] > [ locked {on | off} ] > [ mab {on | off} ] > [ backup_port DEVICE ] [ nobackup_port ] > > After: > > $ ip link help bridge_slave > Usage: ... bridge_slave [ fdb_flush ] > [...] > [ vlan_tunnel {on | off} ] > [ isolated {on | off} ] > [ locked {on | off} ] > [ mab {on | off} ] > [ backup_port DEVICE ] [ nobackup_port ] > > Fixes: 05f1164fe811 ("bridge: link: Add MAC Authentication Bypass (MAB) support") > Signed-off-by: Ido Schimmel <idosch@nvidia.com> Reviewed-by: Simon Horman <simon.horman@corigine.com>
Hello: This patch was applied to iproute2/iproute2.git (main) by Stephen Hemminger <stephen@networkplumber.org>: On Wed, 19 Apr 2023 18:43:59 +0300 you wrote: > Use tabs instead of spaces to be consistent with the rest of the > options. > > Before: > > $ ip link help bridge_slave > Usage: ... bridge_slave [ fdb_flush ] > [...] > [ vlan_tunnel {on | off} ] > [ isolated {on | off} ] > [ locked {on | off} ] > [ mab {on | off} ] > [ backup_port DEVICE ] [ nobackup_port ] > > [...] Here is the summary with links: - [iproute2] ip: bridge_slave: Fix help message indentation https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=ed328120f47a You are awesome, thank you!
diff --git a/ip/iplink_bridge_slave.c b/ip/iplink_bridge_slave.c index 43b429485502..66a67961957f 100644 --- a/ip/iplink_bridge_slave.c +++ b/ip/iplink_bridge_slave.c @@ -40,7 +40,7 @@ static void print_explain(FILE *f) " [ vlan_tunnel {on | off} ]\n" " [ isolated {on | off} ]\n" " [ locked {on | off} ]\n" - " [ mab {on | off} ]\n" + " [ mab {on | off} ]\n" " [ backup_port DEVICE ] [ nobackup_port ]\n" ); }
Use tabs instead of spaces to be consistent with the rest of the options. Before: $ ip link help bridge_slave Usage: ... bridge_slave [ fdb_flush ] [...] [ vlan_tunnel {on | off} ] [ isolated {on | off} ] [ locked {on | off} ] [ mab {on | off} ] [ backup_port DEVICE ] [ nobackup_port ] After: $ ip link help bridge_slave Usage: ... bridge_slave [ fdb_flush ] [...] [ vlan_tunnel {on | off} ] [ isolated {on | off} ] [ locked {on | off} ] [ mab {on | off} ] [ backup_port DEVICE ] [ nobackup_port ] Fixes: 05f1164fe811 ("bridge: link: Add MAC Authentication Bypass (MAB) support") Signed-off-by: Ido Schimmel <idosch@nvidia.com> --- ip/iplink_bridge_slave.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)