On Mon, Mar 11, 2024 at 11:27:34PM +0100, Alexander Zubkov via Bird-users wrote: > Hi all, > > I noticed in the new version 'show bfd sessions' was extended with > 'all' option. But I also noticed that in case of 'show protocols', > 'show ospf state|topology' this option comes before [name], but for > 'show bfd sessions' it is the other way. I think it would be better to > be consistent here and use 'all' keyword before [name] too.
Hi In fact, for 'show bfd sessions' you can use the option 'all' either before or after [name], as there is no fixed order of options for this command. In documentation, there is an order in which they appear, e.g.: show bfd sessions [<name>] [address <ip|prefix>] [(interface|dev) "<name>"] [ipv4|ipv6] [direct|multihop] [all] but that is just simplification, they can be used in any order. In general, there are commands that have a few options and depend on fixed order (e.g. 'show protocols' or 'show ospf neighbors'), and there are commands that have more complex options, use key/value scheme and allow options in arbitrary order (e.g. 'show route', 'show ospf lsadb', or 'show bfd sessions'). I think that the first scheme is lazy and non-extensible, and most commands should be converted to the second scheme. > I also noticed some incompleteness of the "Remote control" > documentation now. I.e. it lists: > show protocols [all] > When it is actually: > show protocols [all] [<protocol>|"<pattern>"] Yes, that is missing. > Also "show route ... [options]" does not have clear description of > what these options are. Not sure if you mean interactive help or reference documentation, as in the documentation they are described: https://bird.network.cz/?get_doc&v=20&f=bird-4.html#cli-show-route (but of course, it could always be improved.) > It seems there are ways to improve this part of the documentation. I > can offer a hand to examine it in more details what else might be > missing and maybe prepare patches. You are welcome. -- Elen sila lumenn' omentielvo Ondrej 'Santiago' Zajicek (email: [email protected]) "To err is human -- to blame it on a computer is even more so."
