Re: [ovs-dev] [PATCH 1/4] ofproto-dpif-trace: Generalize syntax for ofproto/trace.

2018-01-24 Thread Yifeng Sun
Thanks for the patch, looks good to me. Tested-by: Yifeng Sun Reviewed-by: Yifeng Sun On Thu, Jan 18, 2018 at 2:45 PM, Ben Pfaff wrote: > ofproto/trace takes a bunch of options that have weird placement and > syntax. This commit changes the syntax so that the options can be placed > anywher

[ovs-dev] [PATCH 1/4] ofproto-dpif-trace: Generalize syntax for ofproto/trace.

2018-01-18 Thread Ben Pfaff
ofproto/trace takes a bunch of options that have weird placement and syntax. This commit changes the syntax so that the options can be placed anywhere and consistently use a double-dash option prefix. For compatibility, the previous syntax is also supported. An upcoming commit will add new optio