On Mon, Oct 29, 2012 at 03:03:13PM -0500, Adam Heath wrote: > On 10/29/2012 11:34 AM, 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. > > Tbh, I would actually prefer to have command-specific options that > appear in the global area issue an error. > > == > one-vsctl: Found command-specific --may-exist in global area, please > use -- instead. > == > > See (1) for a reason why I'd prefer it the way I suggested above. > > 1: http://lwn.net/Articles/520198/ > you might need to wait a few days for it to become publically > available.
OK, it's available now, and I skimmed over it, but the analogy to ovs-vsctl isn't really obvious to me. Can you elaborate? Thanks, Ben. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
