# 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 <b...@nicira.com> 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: 691...@bugs.debian.org
> >> Reported-by: Adam Heath <doo...@brainfood.com>
> >> Signed-off-by: Ben Pfaff <b...@nicira.com>
> > 
> > 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 <kmest...@cisco.com>

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

Reply via email to