Augh, --unsafe-fixes got rid of my print statements! Dude!

Author: multiplemonomials

tools/pyproject.toml

@@ -38,7 +38,7 @@ dependencies = [
 
     # USB device detection on Linux
     "psutil; platform_system=='Linux'",
-    "pyudev; platform_system=='Linux'",
+    "pyudev>=0.24; platform_system=='Linux'", # 0.23.x seems to have issues with its Six dependency
 
     # USB device detection on Mac
     "beautifulsoup4; sys_platform == 'darwin'",
@@ -78,7 +78,7 @@ linters = [
     "platformio",
     "SCons",
     "psutil",
-    "pyudev",
+    "pyudev>=0.24",
     "beautifulsoup4",
     "lxml"
 ]

tools/python/mbed_platformio/build_mbed_ce.py

@@ -96,18 +96,27 @@ def is_cmake_reconfigure_required() -> bool:
     ninja_buildfile = BUILD_DIR / "build.ninja"
 
     if not cmake_cache_file.exists():
+        print(f"Mbed CE: Reconfigure required because CMake cache does not exist")
         return True
     if not CMAKE_API_REPLY_DIR.is_dir() or not any(CMAKE_API_REPLY_DIR.iterdir()):
+        print("Mbed CE: Reconfigure required because CMake API reply dir is missing")
         return True
     if not ninja_buildfile.exists():
+        print("Mbed CE: Reconfigure required because Ninja buildfile does not exist")
         return True
 
     # If the JSON files have 'Debug' in their names that means this project was previously configured as Debug.
     if not any(CMAKE_API_REPLY_DIR.glob(f"directory-*{CMAKE_BUILD_TYPE}*.json")):
+        print("Mbed CE: Reconfigure required because build type (debug / release) changed.")
         return True
 
     cache_file_mtime = cmake_cache_file.stat().st_mtime
-    return any(file.stat().st_mtime > cache_file_mtime for file in cmake_config_files)
+    for file in cmake_config_files:
+        if file.stat().st_mtime > cache_file_mtime:
+            print(f"Mbed CE: Reconfigure required because {file.name} was modified")
+            return True
+
+    return False
 
 
 def run_tool(command_and_args: list[str] | None = None) -> None:
@@ -121,6 +130,8 @@ def run_tool(command_and_args: list[str] | None = None) -> None:
         env.Exit(1)
 
     if int(ARGUMENTS.get("PIOVERBOSE", 0)):
+        print(result["out"])
+        print(result["err"])
         pass
 
 
@@ -135,6 +146,7 @@ def get_cmake_code_model(cmake_args: list) -> dict:
         create_default_project_files()
 
     if is_cmake_reconfigure_required():
+        print("Mbed CE: Configuring CMake build system...")
         cmake_command = [str(CMAKE_PATH), *cmake_args]
         run_tool(cmake_command)
 
@@ -254,6 +266,7 @@ def get_app_defines(app_config: dict) -> list[tuple[str, str]]:
     print("Error: Couldn't find code model generated by CMake", file=sys.stderr)
     env.Exit(1)
 
+print("Mbed CE: Reading CMake configuration...")
 target_configs = load_target_configurations(project_codemodel)
 
 framework_components_map = get_components_map(target_configs, ["STATIC_LIBRARY", "OBJECT_LIBRARY"], [])
@@ -300,6 +313,9 @@ def get_app_defines(app_config: dict) -> list[tuple[str, str]]:
 # Run Ninja to produce the linker script.
 # Note that this seems to execute CMake, causing the code model query to be re-done.
 # So, we have to do this after we are done using the results of said query.
+print("Mbed CE: Generating linker script...")
 project_ld_script = generate_project_ld_script()
 _ = env.Depends("$BUILD_DIR/$PROGNAME$PROGSUFFIX", str(project_ld_script))
 env.Append(LDSCRIPT_PATH=str(project_ld_script))
+
+print("Mbed CE: Build environment configured.")

tools/python/mbed_platformio/cmake_to_scons_converter.py

@@ -53,7 +53,10 @@ def prepare_build_envs(target_json: dict, default_env: Environment) -> list[Envi
     build_envs = []
     target_compile_groups = target_json.get("compileGroups", [])
     if not target_compile_groups:
-        pass
+        print(
+            "Warning! The `%s` component doesn't register any source files. "
+            "Check if sources are set in component's CMakeLists.txt!" % target_json["name"]
+        )
 
     for cg in target_compile_groups:
         includes = []

tools/python/mbed_tools/build/__init__.py

@@ -3,13 +3,9 @@
 # SPDX-License-Identifier: Apache-2.0
 #
 """
-Provides the core build system for Mbed OS, which relies on CMake and Ninja as underlying technologies.
+Provides the core Python parts of the build system for Mbed OS.
 
-The functionality covered in this package includes the following:
-
-- Execution of Mbed Pre-Build stages to determine appropriate configuration for Mbed OS and the build process.
-- Invocation of the build process for the command line tools and online build service.
-- Export of build instructions to third party command line tools and IDEs.
+This module contains the configuration generation functionality, which processes JSON files into a set of target flags.
 """
 
 from mbed_tools.build.config import generate_config as generate_config