# Mark 691508 as fixed upstream.
tags 691508 + fixed-upstream
thanks

On Fri, Dec 07, 2012 at 10:05:50PM +0000, Kyle Mestery (kmestery) wrote:
> On Dec 4, 2012, at 2:51 PM, Ben Pfaff <[email protected]> wrote:
> > On Mon, Oct 29, 2012 at 09:34:37AM -0700, Ben Pfaff wrote:
> >> Until now, a command like "ovs-vsctl --may-exist add-br br0" yielded a
> >> confusing error message.  Users had to realize that the correct form was
> >> "ovs-vsctl -- --may-exist add-br br0", but instead they often reported a
> >> bug or gave up in frustration.  Even though the behavior was documented, it
> >> was counterintuitive.
> >> 
> >> This commit allows command-specific options to be mixed with global
> >> options, making both forms of the command listed above equally acceptable.
> >> 
> >> CC: [email protected]
> >> Reported-by: Adam Heath <[email protected]>
> >> Signed-off-by: Ben Pfaff <[email protected]>
> > 
> > This has confused many users--including me--over the years, so I'd
> > like to get it reviewed.
> 
> I agree, this will be super handy to have in (even I've been confused by this 
> in the past).
> 
> Acked-by: Kyle Mestery <[email protected]>

Thanks Kyle, I pushed this to master.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to