Pylint for common/repomaker.py
authorZhuoX Li <zhuox.li@intel.com>
Wed, 5 Mar 2014 11:52:43 +0000 (19:52 +0800)
committerLin A Yang <lin.a.yang@intel.com>
Tue, 25 Mar 2014 07:08:53 +0000 (09:08 +0200)
C: 82,4:create_dirs: Operator not preceded by a space
    repo_dirs=set([])
                 ^
                 C:102,8:gen_target_dirs: Comma not followed by a space
                         return [os.path.join(repo_dir,["packages", "debug"][is_debug], ftype)]
                                                              ^^

Change-Id: I0d2c1f634f66a8d08f10ad79923ed53e40d62830

common/repomaker.py

index db65098..61d8583 100644 (file)
@@ -79,7 +79,7 @@ def create_dirs(repo_dir, archs):
                                                   "noarch")))
     dirs.append(("source", None, os.path.join(repo_dir, "source")))
 
-    repo_dirs=set([])
+    repo_dirs = set([])
     # create and get repo_dirs
     for rtype, rarch, rdirname in dirs:
         if not os.path.exists(rdirname):
@@ -99,7 +99,7 @@ def gen_target_dirs(repo_dir, ftype, is_debug):
     if ftype ==  "src":
         return [os.path.join(repo_dir, "source")]
     else:
-        return [os.path.join(repo_dir,["packages", "debug"][is_debug], ftype)]
+        return [os.path.join(repo_dir, ["packages", "debug"][is_debug], ftype)]
 
 def move_or_hardlink(fpath, target_dirs, move=False):
     """Move or hardlink file to target directories."""