On Wed, Feb 09, 2011 at 12:29:12PM -0800, Ethan Jackson wrote:
> Looks good.  Couple of minor comments.
> 
> > -\fBca\-cert\fR
> > +\fica-cert\fR
> > ?Specifies a PEM file containing the CA certificate used to verify that
> > ?the virtual switch is connected to a trustworthy controller.
> > ?.PP
> 
> \fi should be \fI here.

Thanks, I've fixed that up.

> > -Port commands:\n\
> > +Port commands (a bond is considered to be a single port):\n\
> > ? list-ports BRIDGE ? ? ? ? ? print the names of all the ports on BRIDGE\n\
> > ? add-port BRIDGE PORT ? ? ? ?add network device PORT to BRIDGE\n\
> > ? add-bond BRIDGE PORT IFACE... ?add bonded port PORT in BRIDGE from 
> > IFACES\n\
> > ? del-port [BRIDGE] PORT ? ? ?delete PORT (which may be bonded) from 
> > BRIDGE\n\
> > ? port-to-br PORT ? ? ? ? ? ? print name of bridge that contains PORT\n\
> > -A bond is considered to be a single port.\n\
> > +.\n\
> 
> The period before the \n is unnecessary

I've just deleted that line now since there was already a blank line
after it.  Thanks for the review.

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev_openvswitch.org

Reply via email to