diff mbox

[ovs-dev,1/2] utilities: ovs-ctl: Drop duplicate line

Message ID 20160701093126.10453-2-mchandras@suse.de
State Accepted
Headers show

Commit Message

Markos Chandras July 1, 2016, 9:31 a.m. UTC 
The --ovs-vswitchd-wrapper=WRAPPER line is referenced twice so drop it.

Signed-off-by: Markos Chandras <mchandras@suse.de>
---
 utilities/ovs-ctl.in | 1 -
 1 file changed, 1 deletion(-)

Comments

Aaron Conole July 2, 2016, 2:23 a.m. UTC | #1 
Markos Chandras <mchandras@suse.de> writes:

> The --ovs-vswitchd-wrapper=WRAPPER line is referenced twice so drop it.
>
> Signed-off-by: Markos Chandras <mchandras@suse.de>
> ---
>  utilities/ovs-ctl.in | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/utilities/ovs-ctl.in b/utilities/ovs-ctl.in
> index b2a97fc..8ec825b 100755
> --- a/utilities/ovs-ctl.in
> +++ b/utilities/ovs-ctl.in
> @@ -574,7 +574,6 @@ Less important options for "start", "restart" and "force-reload-kmod":
>  Debugging options for "start", "restart" and "force-reload-kmod":
>    --ovsdb-server-wrapper=WRAPPER
>    --ovs-vswitchd-wrapper=WRAPPER
> -  --ovs-vswitchd-wrapper=WRAPPER
>       run specified daemon under WRAPPER (either 'valgrind' or 'strace')
>  
>  File location options:

This one looks good.
Ben Pfaff July 3, 2016, 7:08 p.m. UTC | #2 
On Fri, Jul 01, 2016 at 10:31:25AM +0100, Markos Chandras wrote:
> The --ovs-vswitchd-wrapper=WRAPPER line is referenced twice so drop it.
> 
> Signed-off-by: Markos Chandras <mchandras@suse.de>

Applied, thanks!
diff mbox

Patch

diff --git a/utilities/ovs-ctl.in b/utilities/ovs-ctl.in
index b2a97fc..8ec825b 100755
--- a/utilities/ovs-ctl.in
+++ b/utilities/ovs-ctl.in
@@ -574,7 +574,6 @@  Less important options for "start", "restart" and "force-reload-kmod":
 Debugging options for "start", "restart" and "force-reload-kmod":
   --ovsdb-server-wrapper=WRAPPER
   --ovs-vswitchd-wrapper=WRAPPER
-  --ovs-vswitchd-wrapper=WRAPPER
      run specified daemon under WRAPPER (either 'valgrind' or 'strace')
 
 File location options: