[ovs-dev] [PATCH] Fix the build error caused by flake8 check failure

Ben Pfaff blp at ovn.org
Wed Nov 23 18:04:24 UTC 2016


No problem.  In any case, it's fixed now.

On Wed, Nov 23, 2016 at 05:23:12PM +0530, Numan Siddique wrote:
> Sorry for the patch. Didn't notice that there's already a patch from Ben to
> fix this - https://patchwork.ozlabs.org/patch/697945/
> 
> 
> Thanks
> Numan
> 
> 
> 
> On Wed, Nov 23, 2016 at 2:48 PM, Numan Siddique <nusiddiq at redhat.com> wrote:
> 
> > utilities/ovs-tcpdump.in:21:1: F401 'select' imported but unused
> > Makefile:5645: recipe for target 'flake8-check' failed
> >
> > Fixes: b47d91a74073 ("ovs-tcpdump: should't halt when dumping traffic.")
> > Signed-off-by: Numan Siddique <nusiddiq at redhat.com>
> > ---
> >  utilities/ovs-tcpdump.in | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > 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
> > --
> > 2.7.4
> >
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> >
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev


More information about the dev mailing list