Thanks for the review. I will send a v2 patch with your suggestions.

On Tue, Aug 7, 2018 at 4:06 PM, Ben Pfaff <b...@ovn.org> wrote:

> On Mon, Aug 06, 2018 at 05:21:37PM -0700, Ashish Varma wrote:
> > Even though there are no runtime management commands supported by
> > ovs-testcontroller, the '--unixctl' section of the man page refers to
> > 'RUNTIME MANAGEMENT COMMANDS'. This message which refers to the runtime
> > management commands section is common for all '--unixctl' option.
> > (via 'lib/unixctl.man')
> >
> > Added an empty section in the man page for 'RUNTIME MANAGEMENT COMMANDS'
> to
> > avoid confusion to the reader of the man page.
> >
> > Signed-off-by: Ashish Varma <ashishvarma....@gmail.com>
>
> Thanks for improving the OVS documentation!
>
> However, unixctl is useless if there are no commands.  It's probably
> better to remove the support for it.
>
> Thanks,
>
> Ben.
>
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to