fix: error and warning raised by flake8

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
Signed-off-by: Alexis Masson <alexis.masson@st.com>

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`.
     """
     )
 
@@ -121,8 +122,20 @@ def get_boards(boardstxt):
         libpaths[line["location"]] = pathlib.Path(line["dir"])
 
 # platform lib path is already known, obviously, since that's where this script resides
-userlibs = pathlib.Path(libpaths["user"]).resolve()
-libs = [u.name for u in userlibs.iterdir() if u.is_dir()]
+userlibs = pathlib.Path(libpaths["user"])
+if userlibs.exists():
+    userlibs = userlibs.resolve()
+    libs = [u.name for u in userlibs.iterdir() if u.is_dir()]
+else:
+    print(
+        f"""Warning: Cannot find {userlibs}.
+    This has likely to do with your arduino-cli configuration.
+    Please refer to the following link for setup details:
+    https://arduino.github.io/arduino-cli/0.26/getting-started/#create-a-configuration-file
+    """
+    )
+    libs = list()
+
 corepath = pathlib.Path(__file__).parent.parent.parent.resolve()
 
 j2_env = Environment(

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()
 

cmake/scripts/sizereport.py

@@ -49,7 +49,10 @@
 
 print(
     f"""
-Sketch uses {allsz} bytes ({allsz/shargs.progmem:.0%}) of program storage space. Maximum is {shargs.progmem} bytes.
-Global variables use {datasz} bytes ({datasz/shargs.datamem:.0%}) of dynamic memory, leaving {shargs.datamem-datasz} bytes for local variables. Maximum is {shargs.datamem} bytes.
+Sketch uses {allsz} bytes ({allsz/shargs.progmem:.0%}) of program storage space. \
+Maximum is {shargs.progmem} bytes.
+Global variables use {datasz} bytes ({datasz/shargs.datamem:.0%}) of dynamic memory, \
+leaving {shargs.datamem-datasz} bytes for local variables. \
+Maximum is {shargs.datamem} bytes.
 """
 )