diff mbox

[ovs-dev] ovs-tcpdump: Do not import unused "select" module.

Message ID 1479857546-12205-1-git-send-email-blp@ovn.org
State Accepted
Headers show

Commit Message

Ben Pfaff Nov. 22, 2016, 11:32 p.m. UTC 
Fixes the following flake8 warning:
    utilities/ovs-tcpdump.in:21:1: F401 'select' imported but unused

Signed-off-by: Ben Pfaff <blp@ovn.org>
---
This fixes a broken build.

 utilities/ovs-tcpdump.in | 1 -
 1 file changed, 1 deletion(-)

Comments

Joe Stringer Nov. 23, 2016, 5:13 a.m. UTC | #1 
On 22 November 2016 at 15:32, Ben Pfaff <blp@ovn.org> wrote:
> Fixes the following flake8 warning:
>     utilities/ovs-tcpdump.in:21:1: F401 'select' imported but unused
>
> Signed-off-by: Ben Pfaff <blp@ovn.org>

Acked-by: Joe Stringer <joe@ovn.org>
Ben Pfaff Nov. 23, 2016, 5:48 p.m. UTC | #2 
On Tue, Nov 22, 2016 at 09:13:35PM -0800, Joe Stringer wrote:
> On 22 November 2016 at 15:32, Ben Pfaff <blp@ovn.org> wrote:
> > Fixes the following flake8 warning:
> >     utilities/ovs-tcpdump.in:21:1: F401 'select' imported but unused
> >
> > Signed-off-by: Ben Pfaff <blp@ovn.org>
> 
> Acked-by: Joe Stringer <joe@ovn.org>

Thanks.  Applied to master and branch-2.6.
diff mbox

Patch

diff --git a/utilities/ovs-tcpdump.in b/utilities/ovs-tcpdump.in
index d984a39..538b3b4 100755
--- a/utilities/ovs-tcpdump.in
+++ b/utilities/ovs-tcpdump.in
@@ -18,7 +18,6 @@  import fcntl
 import netifaces
 import os
 import pwd
-import select
 import struct
 import subprocess
 import sys