[ovs-dev] ofproto: Fix typo in manpage fragment.
diff mbox series

Message ID 20200319230256.3286590-1-blp@ovn.org
State New
Headers show
Series
  • [ovs-dev] ofproto: Fix typo in manpage fragment.
Related show

Commit Message

Ben Pfaff March 19, 2020, 11:02 p.m. UTC
There was a missing ] and an extra space.

Signed-off-by: Ben Pfaff <blp@ovn.org>
---
 ofproto/ofproto-unixctl.man | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

0-day Robot March 19, 2020, 11:58 p.m. UTC | #1
Bleep bloop.  Greetings Ben Pfaff, I am a robot and I have tried out your patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


checkpatch:
WARNING: Line is 110 characters long (recommended limit is 79)
#22 FILE: ofproto/ofproto-unixctl.man:12:
.IQ "\fBofproto/trace\-packet\-out\fR [\fIoptions\fR] \fIbridge\fR \fIbr_flow\fR [\fIpacket\fR] \fIactions\fR"

Lines checked: 28, Warnings: 1, Errors: 0


Please check this out.  If you feel there has been an error, please email aconole@redhat.com

Thanks,
0-day Robot
Numan Siddique March 20, 2020, 9 a.m. UTC | #2
On Fri, Mar 20, 2020 at 4:33 AM Ben Pfaff <blp@ovn.org> wrote:
>
> There was a missing ] and an extra space.
>
> Signed-off-by: Ben Pfaff <blp@ovn.org>

Acked-by: Numan Siddique <numans@ovn.org>

Numan

> ---
>  ofproto/ofproto-unixctl.man | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/ofproto/ofproto-unixctl.man b/ofproto/ofproto-unixctl.man
> index 925752343e87..095afd57cc55 100644
> --- a/ofproto/ofproto-unixctl.man
> +++ b/ofproto/ofproto-unixctl.man
> @@ -9,7 +9,7 @@ that may be used on \fBofproto/trace\fR.
>  .IP "\fBofproto/trace\fR [\fIoptions\fR] [\fIdpname\fR] \fIodp_flow\fR [\fIpacket\fR]
>  .IQ "\fBofproto/trace\fR [\fIoptions\fR] \fIbridge\fR \fIbr_flow\fR [\fIpacket\fR]]
>  .IQ "\fBofproto/trace\-packet\-out\fR [\fIoptions\fR] [\fIdpname\fR] \fIodp_flow\fR [\fIpacket\fR] \fIactions\fR"
> -.IQ "\fBofproto/trace\-packet\-out\fR [\fIoptions\fR \fIbridge\fR \fIbr_flow\fR  [\fIpacket\fR] \fIactions\fR"
> +.IQ "\fBofproto/trace\-packet\-out\fR [\fIoptions\fR] \fIbridge\fR \fIbr_flow\fR [\fIpacket\fR] \fIactions\fR"
>  Traces the path of an imaginary packet through \fIswitch\fR and
>  reports the path that it took.  The initial treatment of the packet
>  varies based on the command:
> --
> 2.24.1
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
Ben Pfaff March 20, 2020, 3:51 p.m. UTC | #3
On Fri, Mar 20, 2020 at 02:30:22PM +0530, Numan Siddique wrote:
> On Fri, Mar 20, 2020 at 4:33 AM Ben Pfaff <blp@ovn.org> wrote:
> >
> > There was a missing ] and an extra space.
> >
> > Signed-off-by: Ben Pfaff <blp@ovn.org>
> 
> Acked-by: Numan Siddique <numans@ovn.org>

Thanks, applied to master.

Patch
diff mbox series

diff --git a/ofproto/ofproto-unixctl.man b/ofproto/ofproto-unixctl.man
index 925752343e87..095afd57cc55 100644
--- a/ofproto/ofproto-unixctl.man
+++ b/ofproto/ofproto-unixctl.man
@@ -9,7 +9,7 @@  that may be used on \fBofproto/trace\fR.
 .IP "\fBofproto/trace\fR [\fIoptions\fR] [\fIdpname\fR] \fIodp_flow\fR [\fIpacket\fR]
 .IQ "\fBofproto/trace\fR [\fIoptions\fR] \fIbridge\fR \fIbr_flow\fR [\fIpacket\fR]]
 .IQ "\fBofproto/trace\-packet\-out\fR [\fIoptions\fR] [\fIdpname\fR] \fIodp_flow\fR [\fIpacket\fR] \fIactions\fR"
-.IQ "\fBofproto/trace\-packet\-out\fR [\fIoptions\fR \fIbridge\fR \fIbr_flow\fR  [\fIpacket\fR] \fIactions\fR"
+.IQ "\fBofproto/trace\-packet\-out\fR [\fIoptions\fR] \fIbridge\fR \fIbr_flow\fR [\fIpacket\fR] \fIactions\fR"
 Traces the path of an imaginary packet through \fIswitch\fR and
 reports the path that it took.  The initial treatment of the packet
 varies based on the command: