fix: error and warning raised by flake8

Signed-off-by: Frederic Pillon <[email protected]>

Author: fpistm

cmake/scripts/.flake8

@@ -0,0 +1,2 @@
+[flake8]
+max-line-length = 88

cmake/scripts/ccwrapper.py

@@ -22,7 +22,7 @@
 
 shargs = parser.parse_args()
 
-logline = re.compile("^\.* .+$")  # a series of dots, a space, a filename
+logline = re.compile(r"^\.* .+$")  # a series of dots, a space, a filename
 
 
 proc = subprocess.run(shargs.cmd, capture_output=True, encoding="ascii")

cmake/scripts/cmake_core.py

@@ -1,10 +1,9 @@
 #!/usr/bin/env python3
 
 import argparse
-import sys
 from pathlib import Path
 from jinja2 import Environment, FileSystemLoader
-from cmake_gen import *
+from cmake_gen import config_for_bareflat, render
 
 script_path = Path(__file__).parent.resolve()
 

cmake/scripts/cmake_easy_setup.py

@@ -46,7 +46,8 @@
     print(
         """
     Warning: you did not specify which board you were targeting;
-    please review the generated CMakeLists.txt to remove the placeholder value before calling `cmake`.
+    please review the generated CMakeLists.txt to remove the placeholder
+    value before calling `cmake`.
     """
     )
 

cmake/scripts/cmake_libs.py

@@ -3,7 +3,7 @@
 import argparse
 from pathlib import Path
 from jinja2 import Environment, FileSystemLoader
-from cmake_gen import *
+from cmake_gen import autoconfig, render
 
 script_path = Path(__file__).parent.resolve()
 

cmake/scripts/cmake_updater_hook.py

@@ -1,7 +1,8 @@
 #!/usr/bin/env python3
 
 """
-This file centralizes all the operations needed to regenerate the CMakeLists.txt scattered along this repo.
+This file centralizes all the operations needed to regenerate the CMakeLists.txt
+scattered along this repo.
 Hint: it would be a good practice to run it before committing...
 """
 

cmake/scripts/cmake_variant.py

@@ -3,7 +3,7 @@
 import argparse
 from pathlib import Path
 from jinja2 import Environment, FileSystemLoader
-from cmake_gen import *
+from cmake_gen import config_for_bareflat, render
 
 script_path = Path(__file__).parent.resolve()
 

cmake/scripts/generate_header.py

@@ -2,7 +2,6 @@
 
 import argparse
 import pathlib
-import subprocess
 
 parser = argparse.ArgumentParser()
 parser.add_argument(

cmake/scripts/parse_boards.py

@@ -25,7 +25,7 @@ def __getattr__(self, attr):
         return self.__getitem__(attr)
 
     def __setattr__(self, attr, val):
-        return self.__setitem__(item, attr)
+        return self.__setitem__(val, attr)
 
     def set_default_entries(self, mothercfg):
         for k, v in mothercfg.items():
@@ -51,9 +51,13 @@ def evaluate_entries(self, wrt=None):
                 self[k].evaluate_entries(wrt)
 
 
+def default_reject(x):
+    return False
+
+
 def parse_file(infile, reject=None):
     if reject is None:
-        reject = lambda x: False
+        reject = default_reject
 
     config = Configuration()