On Sep 24, 2013, at 9:09 PM, Ben Pfaff <b...@nicira.com> wrote:

> On Tue, Sep 24, 2013 at 04:41:43PM -0700, Justin Pettit wrote:
>> On Sep 24, 2013, at 2:11 PM, Ben Pfaff <b...@nicira.com> wrote:
>> 
>>> On Mon, Sep 23, 2013 at 02:36:39PM -0700, Justin Pettit wrote:
>>>> Signed-off-by: Justin Pettit <jpet...@nicira.com>
>>> 
>>> Should the tcpdump patch check for the old and the new OpenFlow port?
>>> 
>>> There are many remaining hits for 6633 in the tree.  I'd suggest
>>> updating at least lib/vconn-active.man, lib/vconn-passive.man,
>>> utilities/ovs-controller.8.in, and vswitchd/vswitch.xml to mention
>>> that the default port number will change.  (Please don't update
>>> lib/aes128.c despite the three hits ;-)
>>> 
>>> Acked-by: Ben Pfaff <b...@nicira.com>
>> 
>> Okay, how about the incremental at the end of this message?
> 
> Looks good, thanks.
> 
> I see one use of "when this form is used", which seems unnecessary in
> context.


Thanks.  I fixed this (and another typo you pointed out off-list) and pushed 
the series.

--Justin


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

Reply via email to