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]> > _______________________________________________ > dev mailing list > [email protected] > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
