diff mbox series

[01/13] libselinux: fix flake8 warnings in SWIG-generated code

Message ID 20180804194734.12577-2-nicolas.iooss@m4x.org (mailing list archive)
State Not Applicable
Headers show
Series Fix some issues found by flake8 | expand

Commit Message

Nicolas Iooss Aug. 4, 2018, 7:47 p.m. UTC 
flake8, a Python linter, complains:

    libselinux/src/selinux.py:100:1: F401 'errno' imported but unused
    libselinux/src/selinux.py:100:1: F401 'stat' imported but unused
    libselinux/src/selinux.py:100:14: E401 multiple imports on one line
    libselinux/src/selinux.py:130:16: E111 indentation is not a multiple of four
    libselinux/src/selinux.py:130:45: E231 missing whitespace after ','

Signed-off-by: Nicolas Iooss <nicolas.iooss@m4x.org>
---
 libselinux/src/selinuxswig_python.i | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/libselinux/src/selinuxswig_python.i b/libselinux/src/selinuxswig_python.i
index be17ceffe01f..2af7c7e0c038 100644
--- a/libselinux/src/selinuxswig_python.i
+++ b/libselinux/src/selinuxswig_python.i
@@ -13,7 +13,8 @@ 
 
 %pythoncode %{
 
-import shutil, os, errno, stat
+import shutil
+import os
 
 DISABLED = -1
 PERMISSIVE = 0
@@ -43,7 +44,7 @@  def chcon(path, context, recursive=False):
     if recursive:
         for root, dirs, files in os.walk(path):
             for name in files + dirs:
-               lsetfilecon(os.path.join(root,name), context)
+                lsetfilecon(os.path.join(root, name), context)
 
 def copytree(src, dest):
     """ An SELinux-friendly shutil.copytree method """