Hi Donald

I had added the command in zebra_vty.c to dump the mbgp routing table in
RTM. I should have renamed it to show IP route mbgp or something else but
we could remove this command as I believe Cisco also does not show the mbgp
routing table. Morever show ip mroute is the apt command for displaying
mcast routing table.If that be the case I can send a patch for it to be
removed

Thanks
  Balaji
On May 24, 2015 6:17 AM, "Donald Sharp" <[email protected]> wrote:

> In pim_cmd.c we have:
>
> DEFUN (show_ip_mroute,
>        show_ip_mroute_cmd,
>        "show ip mroute",
>        SHOW_STR
>        IP_STR
>        MROUTE_STR)
>
> In zebra_vty.c we have:
>
> DEFUN (show_ip_mroute,
>        show_ip_mroute_cmd,
>        "show ip mroute",
>        SHOW_STR
>        IP_STR
>        "IP Multicast routing table\n")
>
> Both of these commands display radically different information.
>
> When we parse the files( via extract.pl ) for compiling vtysh, both of
> these commands conflict due to the same name and the pim_cmd.c file wins
> the resolution.  Before I send a patch in to address this, what should be
> the correct rename?  I suggest 'show ip mroute pim' for pim_cmd.c and leave
> 'show ip mroute' for zebra_vty.c.  Do people have an opinion?
>
> donald
>
>
>
> _______________________________________________
> Quagga-dev mailing list
> [email protected]
> https://lists.quagga.net/mailman/listinfo/quagga-dev
>
>
_______________________________________________
Quagga-dev mailing list
[email protected]
https://lists.quagga.net/mailman/listinfo/quagga-dev

Reply via email to