From b95999aca7cc2d39ff6846627b8a0936f409465d Mon Sep 17 00:00:00 2001 From: "J. Nick Koston" Date: Fri, 17 Oct 2025 17:55:37 -1000 Subject: [PATCH 1/3] Update esphome/analyze_memory/__init__.py Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- esphome/analyze_memory/__init__.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/esphome/analyze_memory/__init__.py b/esphome/analyze_memory/__init__.py index 5ef9eab526..74299d4e95 100644 --- a/esphome/analyze_memory/__init__.py +++ b/esphome/analyze_memory/__init__.py @@ -295,7 +295,7 @@ class MemoryAnalyzer: cppfilt_cmd = "c++filt" _LOGGER.warning("Demangling %d symbols", len(symbols)) - _LOGGER.warning("objdump_path = %s", self.objdump_path) + _LOGGER.debug("objdump_path = %s", self.objdump_path) # Check if we have a toolchain-specific c++filt if self.objdump_path and self.objdump_path != "objdump": From 9a4288d81a02e7484e393f97a89fab856ae6e4e9 Mon Sep 17 00:00:00 2001 From: "J. Nick Koston" Date: Fri, 17 Oct 2025 17:56:41 -1000 Subject: [PATCH 2/3] Update script/determine-jobs.py Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- script/determine-jobs.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/script/determine-jobs.py b/script/determine-jobs.py index 5767ced859..26e91edbe1 100755 --- a/script/determine-jobs.py +++ b/script/determine-jobs.py @@ -26,7 +26,7 @@ The CI workflow uses this information to: - Skip or run Python linters (ruff, flake8, pylint, pyupgrade) - Determine which components to test individually - Decide how to split component tests (if there are many) -- Run memory impact analysis when exactly one component changes +- Run memory impact analysis whenever there are changed components (merged config), and also for core-only changes Usage: python script/determine-jobs.py [-b BRANCH] From a96cc5e6f20a8b7205a48ea38836fb22ff012239 Mon Sep 17 00:00:00 2001 From: "J. Nick Koston" Date: Fri, 17 Oct 2025 17:57:33 -1000 Subject: [PATCH 3/3] Update esphome/analyze_memory/__init__.py Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- esphome/analyze_memory/__init__.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/esphome/analyze_memory/__init__.py b/esphome/analyze_memory/__init__.py index 74299d4e95..3e85c4d869 100644 --- a/esphome/analyze_memory/__init__.py +++ b/esphome/analyze_memory/__init__.py @@ -294,24 +294,24 @@ class MemoryAnalyzer: # Try to find the appropriate c++filt for the platform cppfilt_cmd = "c++filt" - _LOGGER.warning("Demangling %d symbols", len(symbols)) + _LOGGER.info("Demangling %d symbols", len(symbols)) _LOGGER.debug("objdump_path = %s", self.objdump_path) # Check if we have a toolchain-specific c++filt if self.objdump_path and self.objdump_path != "objdump": # Replace objdump with c++filt in the path potential_cppfilt = self.objdump_path.replace("objdump", "c++filt") - _LOGGER.warning("Checking for toolchain c++filt at: %s", potential_cppfilt) + _LOGGER.info("Checking for toolchain c++filt at: %s", potential_cppfilt) if Path(potential_cppfilt).exists(): cppfilt_cmd = potential_cppfilt - _LOGGER.warning("✓ Using toolchain c++filt: %s", cppfilt_cmd) + _LOGGER.info("✓ Using toolchain c++filt: %s", cppfilt_cmd) else: - _LOGGER.warning( + _LOGGER.info( "✗ Toolchain c++filt not found at %s, using system c++filt", potential_cppfilt, ) else: - _LOGGER.warning( + _LOGGER.info( "✗ Using system c++filt (objdump_path=%s)", self.objdump_path )