On Mon, Apr 27, 2020 at 4:51 PM Benjamin Poirier <bpoir...@cumulusnetworks.com> wrote: > > Output the same names for vlan flags as the ones accepted in command input. > > Signed-off-by: Benjamin Poirier <bpoir...@cumulusnetworks.com> > ---
Benjamin, It's a good change, but this will break existing users ?. > bridge/vlan.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/bridge/vlan.c b/bridge/vlan.c > index 205851e4..08b19897 100644 > --- a/bridge/vlan.c > +++ b/bridge/vlan.c > @@ -398,10 +398,10 @@ static void print_vlan_flags(__u16 flags) > > open_json_array(PRINT_JSON, "flags"); > if (flags & BRIDGE_VLAN_INFO_PVID) > - print_string(PRINT_ANY, NULL, " %s", "PVID"); > + print_string(PRINT_ANY, NULL, " %s", "pvid"); > > if (flags & BRIDGE_VLAN_INFO_UNTAGGED) > - print_string(PRINT_ANY, NULL, " %s", "Egress Untagged"); > + print_string(PRINT_ANY, NULL, " %s", "untagged"); > close_json_array(PRINT_JSON, NULL); > } > > -- > 2.26.0 >