Re: [ovs-dev] [PATCH 3/3] ovs-ofctl.8.in: Fix minor formatting issues.

2016-09-19 Thread Justin Pettit 

> On Sep 19, 2016, at 8:26 AM, Ben Pfaff  wrote:
> 
> On Fri, Sep 16, 2016 at 11:27:02AM -0700, Justin Pettit wrote:
>> Signed-off-by: Justin Pettit 
> 
> Please drop the changes from "flow-based" to "flow\-based".  The rule
> for "-" versus "\-" in nroff isn't "put \ before every -", but instead
> more like "put \ before a literal - in sample syntax".  The fuller
> version of the rule is that - is a hyphen and \- is a minus sign; you
> want the former in "flow-based" and the latter in e.g. "--dry-run".
> 
> I'll get off my high horse now.
> 
> Anyway, except for those particular changes,
> Acked-by: Ben Pfaff 

Thanks for the explanation.  I made that change and pushed the series to master.

--Justin


___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 3/3] ovs-ofctl.8.in: Fix minor formatting issues.

2016-09-19 Thread Ben Pfaff 
On Fri, Sep 16, 2016 at 11:27:02AM -0700, Justin Pettit wrote:
> Signed-off-by: Justin Pettit 

Please drop the changes from "flow-based" to "flow\-based".  The rule
for "-" versus "\-" in nroff isn't "put \ before every -", but instead
more like "put \ before a literal - in sample syntax".  The fuller
version of the rule is that - is a hyphen and \- is a minus sign; you
want the former in "flow-based" and the latter in e.g. "--dry-run".

I'll get off my high horse now.

Anyway, except for those particular changes,
Acked-by: Ben Pfaff 
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH 3/3] ovs-ofctl.8.in: Fix minor formatting issues.

2016-09-16 Thread Justin Pettit 
Signed-off-by: Justin Pettit 
---
 utilities/ovs-ofctl.8.in | 10 +-
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/utilities/ovs-ofctl.8.in b/utilities/ovs-ofctl.8.in
index 249b28a..cf39fff 100644
--- a/utilities/ovs-ofctl.8.in
+++ b/utilities/ovs-ofctl.8.in
@@ -255,7 +255,7 @@ This command has limited usefulness, because ports often 
have no
 configured queues and because the OpenFlow protocol provides only very
 limited information about the configuration of a queue.
 .
-.IP "\fBdump\-ipfix\-bridge \fIswitch
+.IP "\fBdump\-ipfix\-bridge \fIswitch\fR"
 Prints to the console the statistics of bridge IPFIX for \fIswitch\fR.
 If bridge IPFIX is configured on the \fIswitch\fR, IPFIX statistics
 can be retrieved.  Otherwise, error message will be printed.
@@ -263,13 +263,13 @@ can be retrieved.  Otherwise, error message will be 
printed.
 This command uses an Open vSwitch extension that is only in Open
 vSwitch 2.6 and later.
 .
-.IP "\fBdump\-ipfix\-flow \fIswitch
-Prints to the console the statistics of flow-based IPFIX for
-\fIswitch\fR.  If flow-based IPFIX is configured on the \fIswitch\fR,
+.IP "\fBdump\-ipfix\-flow \fIswitch\fR"
+Prints to the console the statistics of flow\-based IPFIX for
+\fIswitch\fR.  If flow\-based IPFIX is configured on the \fIswitch\fR,
 statistics of all the collector set ids on the \fIswitch\fR will be
 printed.  Otherwise, print error message.
 .IP
-Refer to \fBovs-vswitchd.conf.db\fR(5) for more details on configuring
+Refer to \fBovs\-vswitchd.conf.db\fR(5) for more details on configuring
 flow based IPFIX and collector set ids.
 .IP
 This command uses an Open vSwitch extension that is only in Open
-- 
1.9.1

___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev