idf-build-apps 2.8.1__tar.gz → 2.10.0__tar.gz

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/.pre-commit-config.yaml

@@ -16,8 +16,8 @@ repos:
           - license_header.txt        # defaults to: LICENSE.txt
           - --use-current-year
         exclude: 'idf_build_apps/vendors/'
-  - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: 'v0.9.5'
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: 'v0.11.6'
     hooks:
       - id: ruff
         args: ['--fix']

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 All notable changes to this project will be documented in this file.
 
+## v2.10.0 (2025-04-22)
+
+### Feat
+
+- support custom class load from CLI
+
+## v2.9.0 (2025-04-16)
+
+### Feat
+
+- record manifest_path that introduced the folder rule
+- support env var expansion in some fields
+
 ## v2.8.1 (2025-03-04)
 
 ### Fix

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: idf-build-apps
-Version: 2.8.1
+Version: 2.10.0
 Summary: Tools for building ESP-IDF related apps.
 Author-email: Fu Hanxi <fuhanxi@espressif.com>
 Requires-Python: >=3.7

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/docs/conf_common.py

@@ -1,4 +1,4 @@
-# SPDX-FileCopyrightText: 2023-2024 Espressif Systems (Shanghai) CO LTD
+# SPDX-FileCopyrightText: 2023-2025 Espressif Systems (Shanghai) CO LTD
 # SPDX-License-Identifier: Apache-2.0
 
 import os

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/docs/en/conf.py

@@ -1,4 +1,4 @@
-# SPDX-FileCopyrightText: 2024 Espressif Systems (Shanghai) CO LTD
+# SPDX-FileCopyrightText: 2024-2025 Espressif Systems (Shanghai) CO LTD
 # SPDX-License-Identifier: Apache-2.0
 
 import os

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/docs/en/explanations/config_rules.rst

@@ -146,6 +146,12 @@ When building the project for the ESP32 target, `sdkconfig files`_ with the ``.e
       B --> C --> D -- "populates sdkconfig file, then build" --> F
       B --> E -- "populates sdkconfig file, then build" --> G
 
+.. warning::
+
+   Standalone target-specific sdkconfig files are ignored. To make the target-specific sdkconfig files effective, the original sdkconfig file, (without the target name suffix) must be present.
+
+   For example, ``sdkconfig.ci.foo.esp32`` will only be taken into account while building with target ``esp32`` if ``sdkconfig.ci.foo`` is also present.
+
 Override In CLI
 ===============
 

idf_build_apps-2.10.0/docs/en/guides/custom_app.md

@@ -0,0 +1,80 @@
+# Custom App Classes
+
+`idf-build-apps` allows you to create custom app classes by subclassing the base `App` class. This is useful when you need to implement custom build logic or handle special project types.
+
+## Creating a Custom App Class
+
+Here's an example of creating a custom app class:
+
+```python
+from idf_build_apps import App
+from idf_build_apps.constants import BuildStatus
+import os
+from typing import Literal  # Python 3.8+ only. from typing_extensions import Literal for earlier versions
+
+class CustomApp(App):
+    build_system: Literal['custom'] = 'custom'  # Must be unique to identify your custom app type
+
+    def build(self, *args, **kwargs):
+        # Implement your custom build logic here
+        os.makedirs(self.build_path, exist_ok=True)
+        with open(os.path.join(self.build_path, 'dummy.txt'), 'w') as f:
+            f.write('Custom build successful')
+        self.build_status = BuildStatus.SUCCESS
+        print('Custom build successful')
+
+    @classmethod
+    def is_app(cls, path: str) -> bool:
+        # Implement logic to determine if a path contains your custom app type
+        return True
+```
+
+## Using Custom App Classes
+
+You can use custom app classes in two ways:
+
+### Via CLI
+
+```shell
+idf-build-apps build -p /path/to/app --target esp32 --build-system custom:CustomApp
+```
+
+Where `custom:CustomApp` is in the format `module:class`. The module must be in your Python path.
+
+### Via Python API
+
+```python
+from idf_build_apps import find_apps
+
+apps = find_apps(
+    paths=['/path/to/app'],
+    target='esp32',
+    build_system=CustomApp,
+)
+
+for app in apps:
+    app.build()
+```
+
+## Important Notes
+
+- Your custom app class must subclass `App`
+- The `build_system` attribute must be unique to identify your app type
+- You must implement the `is_app()` class method to identify your app type
+- For JSON serialization support, you need to pass your custom class to `json_to_app()` when deserializing
+
+## Example: JSON Serialization
+
+```python
+from idf_build_apps import json_to_app
+
+# Serialize
+json_str = custom_app.to_json()
+
+# Deserialize
+deserialized_app = json_to_app(json_str, extra_classes=[CustomApp])
+```
+
+## Available Methods and Properties
+
+Please refer to the [API reference of the class `App`](https://docs.espressif.com/projects/idf-build-apps/en/latest/references/api/idf_build_apps.html#idf_build_apps.app.App)

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/docs/en/index.rst

@@ -16,8 +16,9 @@ This documentation is for idf-build-apps. idf-build-apps is a tool that allows d
 .. toctree::
    :maxdepth: 1
    :caption: Guides
+   :glob:
 
-   guides/1.x_to_2.x
+   guides/*
 
 .. toctree::
    :maxdepth: 1

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/docs/en/references/config_file.rst

@@ -97,6 +97,18 @@ This indicates that in the configuration file, you should specify it with the na
        "bar",
    ]
 
+******************************
+ Expand Environment Variables
+******************************
+
+Some configuration options support environment variables. You can use environment variables in the configuration file by using the syntax ``${VAR_NAME}`` or ``$VAR_NAME``. Undeclared environment variables will be replaced with an empty string. For exmaple:
+
+.. code:: toml
+
+   collect_app_info_filename = "app_info_${CI_JOB_NAME_SLUG}"
+
+when the environment variable ``CI_JOB_NAME_SLUG`` is set to ``my_job``, the ``collect_app_info_filename`` will be expanded to ``app_info_my_job``. When the environment variable is not set, the value will be ``app_info_``.
+
 *************************
  CLI Argument Precedence
 *************************

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/idf_build_apps/__init__.py

@@ -8,7 +8,7 @@ Tools for building ESP-IDF related apps.
 # ruff: noqa: E402
 # avoid circular imports
 
-__version__ = '2.8.1'
+__version__ = '2.10.0'
 
 from .session_args import (
     SessionArgs,

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/idf_build_apps/__main__.py

@@ -1,4 +1,4 @@
-# SPDX-FileCopyrightText: 2022-2023 Espressif Systems (Shanghai) CO LTD
+# SPDX-FileCopyrightText: 2022-2025 Espressif Systems (Shanghai) CO LTD
 # SPDX-License-Identifier: Apache-2.0
 
 from .main import (

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/idf_build_apps/args.py

@@ -14,6 +14,7 @@ from copy import deepcopy
 from dataclasses import dataclass
 from io import TextIOWrapper
 from pathlib import Path
+from string import Template
 from typing import Any
 
 from pydantic import AliasChoices, Field, computed_field, field_validator
@@ -112,6 +113,29 @@ def get_meta(f: FieldInfo) -> t.Optional[FieldMetadata]:
     return None
 
 
+def expand_vars(v: t.Optional[str]) -> t.Optional[str]:
+    """
+    Expand environment variables in the string. If the variable is not found, use an empty string.
+
+    :param v: string to expand
+    :return: expanded string or None if the input is None
+    """
+    if v is None:
+        return None
+
+    unknown_vars: t.Dict[str, str] = dict()
+    while True:
+        try:
+            v = Template(v).substitute(os.environ, **unknown_vars)
+        except KeyError as e:
+            LOGGER.debug('Environment variable %s not found. use empty string', e)
+            unknown_vars[e.args[0]] = ''
+        else:
+            break
+
+    return v
+
+
 class BaseArguments(BaseSettings):
     """Base settings class for all settings classes"""
 
@@ -154,7 +178,7 @@ class BaseArguments(BaseSettings):
                     if method == ValidateMethod.TO_LIST:
                         v = to_list(v)
                     elif method == ValidateMethod.EXPAND_VARS:
-                        v = os.path.expandvars(v)
+                        v = expand_vars(v)
                     else:
                         raise NotImplementedError(f'Unknown validate method: {method}')
 
@@ -390,10 +414,9 @@ class FindBuildArguments(DependencyDrivenBuildArguments):
         default='all',  # type: ignore
     )
     build_system: t.Union[str, t.Type[App]] = field(
-        FieldMetadata(
-            choices=['cmake', 'make'],
-        ),
-        description='Filter the apps by build system. By default set to "cmake"',
+        None,
+        description='Filter the apps by build system. By default set to "cmake". '
+        'Can be either "cmake", "make" or a custom App class path in format "module:class"',
         default='cmake',  # type: ignore
     )
     recursive: bool = field(
@@ -689,9 +712,10 @@ class BuildArguments(FindBuildArguments):
         FieldMetadata(
             deprecates={'collect_size_info': {}},
             hidden=True,
+            validate_method=[ValidateMethod.EXPAND_VARS],
         ),
         description='Record size json filepath of the built apps to the specified file. '
-        'Each line is a json string. Can expand placeholders @p',
+        'Each line is a json string. Can expand placeholders @p. Support environment variables.',
         validation_alias=AliasChoices('collect_size_info_filename', 'collect_size_info'),
         default=None,  # type: ignore
         exclude=True,  # computed field is used
@@ -700,9 +724,10 @@ class BuildArguments(FindBuildArguments):
         FieldMetadata(
             deprecates={'collect_app_info': {}},
             hidden=True,
+            validate_method=[ValidateMethod.EXPAND_VARS],
         ),
         description='Record serialized app model of the built apps to the specified file. '
-        'Each line is a json string. Can expand placeholders @p',
+        'Each line is a json string. Can expand placeholders @p. Support environment variables.',
         validation_alias=AliasChoices('collect_app_info_filename', 'collect_app_info'),
         default=None,  # type: ignore
         exclude=True,  # computed field is used
@@ -711,8 +736,10 @@ class BuildArguments(FindBuildArguments):
         FieldMetadata(
             deprecates={'junitxml': {}},
             hidden=True,
+            validate_method=[ValidateMethod.EXPAND_VARS],
         ),
-        description='Path to the junitxml file to record the build results. Can expand placeholder @p',
+        description='Path to the junitxml file to record the build results. Can expand placeholder @p. '
+        'Support environment variables.',
         validation_alias=AliasChoices('junitxml_filename', 'junitxml'),
         default=None,  # type: ignore
         exclude=True,  # computed field is used

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/idf_build_apps/autocompletions.py

@@ -1,4 +1,4 @@
-# SPDX-FileCopyrightText: 2024 Espressif Systems (Shanghai) CO LTD
+# SPDX-FileCopyrightText: 2024-2025 Espressif Systems (Shanghai) CO LTD
 # SPDX-License-Identifier: Apache-2.0
 
 import os

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/idf_build_apps/finder.py

@@ -1,4 +1,4 @@
-# SPDX-FileCopyrightText: 2022-2024 Espressif Systems (Shanghai) CO LTD
+# SPDX-FileCopyrightText: 2022-2025 Espressif Systems (Shanghai) CO LTD
 # SPDX-License-Identifier: Apache-2.0
 
 import logging

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/idf_build_apps/junit/__init__.py

@@ -1,4 +1,4 @@
-# SPDX-FileCopyrightText: 2023-2024 Espressif Systems (Shanghai) CO LTD
+# SPDX-FileCopyrightText: 2023-2025 Espressif Systems (Shanghai) CO LTD
 # SPDX-License-Identifier: Apache-2.0
 
 from .report import (

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/idf_build_apps/junit/utils.py

@@ -1,4 +1,4 @@
-# SPDX-FileCopyrightText: 2023-2024 Espressif Systems (Shanghai) CO LTD
+# SPDX-FileCopyrightText: 2023-2025 Espressif Systems (Shanghai) CO LTD
 # SPDX-License-Identifier: Apache-2.0
 
 import os

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/idf_build_apps/main.py

@@ -4,6 +4,7 @@
 # SPDX-License-Identifier: Apache-2.0
 
 import argparse
+import importlib
 import json
 import logging
 import os
@@ -90,13 +91,24 @@ def find_apps(
 
     app_cls: t.Type[App]
     if isinstance(find_arguments.build_system, str):
-        # backwards compatible
         if find_arguments.build_system == 'cmake':
             app_cls = CMakeApp
         elif find_arguments.build_system == 'make':
             app_cls = MakeApp
+        elif ':' in find_arguments.build_system:
+            # Custom App class in format "module:class"
+            try:
+                module_path, class_name = find_arguments.build_system.split(':')
+                module = importlib.import_module(module_path)
+                app_cls = getattr(module, class_name)
+                if not issubclass(app_cls, App):
+                    raise ValueError(f'Class {class_name} must be a subclass of App')
+            except (ValueError, ImportError, AttributeError) as e:
+                raise ValueError(f'Invalid custom App class path: {find_arguments.build_system}. Error: {e!s}')
         else:
-            raise ValueError('Only Support "make" and "cmake"')
+            raise ValueError(
+                'build_system must be either "cmake", "make" or a custom App class path in format "module:class"'
+            )
     else:
         app_cls = find_arguments.build_system
 

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/idf_build_apps/manifest/manifest.py

@@ -89,7 +89,10 @@ class FolderRule:
         disable_test: t.Optional[t.List[t.Dict[str, t.Any]]] = None,
         depends_components: t.Optional[t.List[t.Union[str, t.Dict[str, t.Any]]]] = None,
         depends_filepatterns: t.Optional[t.List[t.Union[str, t.Dict[str, t.Any]]]] = None,
+        manifest_filepath: t.Optional[str] = None,
     ) -> None:
+        self._manifest_filepath = manifest_filepath
+
         self.folder = os.path.abspath(folder)
 
         def _clause_to_if_clause(clause: t.Dict[str, t.Any]) -> IfClause:
@@ -168,6 +171,10 @@ class FolderRule:
     def __repr__(self) -> str:
         return f'FolderRule({self.folder})'
 
+    @property
+    def by_manifest_file(self) -> t.Optional[str]:
+        return self._manifest_filepath
+
     def _enable_build(self, target: str, config_name: str) -> bool:
         if self.enable:
             res = False
@@ -309,7 +316,7 @@ class Manifest:
                     LOGGER.warning(msg)
 
             try:
-                rules.append(FolderRule(folder, **folder_rule if folder_rule else {}))
+                rules.append(FolderRule(folder, **folder_rule if folder_rule else {}, manifest_filepath=str(path)))
             except InvalidIfClause as e:
                 raise InvalidManifest(f'Invalid manifest file {path}: {e}')
 

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/idf_build_apps/session_args.py

@@ -1,4 +1,4 @@
-# SPDX-FileCopyrightText: 2023-2024 Espressif Systems (Shanghai) CO LTD
+# SPDX-FileCopyrightText: 2023-2025 Espressif Systems (Shanghai) CO LTD
 # SPDX-License-Identifier: Apache-2.0
 
 import logging

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/idf_build_apps/yaml/__init__.py

@@ -1,4 +1,4 @@
-# SPDX-FileCopyrightText: 2024 Espressif Systems (Shanghai) CO LTD
+# SPDX-FileCopyrightText: 2024-2025 Espressif Systems (Shanghai) CO LTD
 # SPDX-License-Identifier: Apache-2.0
 
 from .parser import (

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/idf_build_apps/yaml/parser.py

@@ -1,4 +1,4 @@
-# SPDX-FileCopyrightText: 2024 Espressif Systems (Shanghai) CO LTD
+# SPDX-FileCopyrightText: 2024-2025 Espressif Systems (Shanghai) CO LTD
 # SPDX-License-Identifier: Apache-2.0
 
 import typing as t

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/pyproject.toml

@@ -64,7 +64,7 @@ idf-build-apps = "idf_build_apps:main.main"
 
 [tool.commitizen]
 name = "cz_conventional_commits"
-version = "2.8.1"
+version = "2.10.0"
 tag_format = "v$version"
 version_files = [
     "idf_build_apps/__init__.py",

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/setup.py

@@ -38,7 +38,7 @@ entry_points = \
 {'console_scripts': ['idf-build-apps = idf_build_apps:main.main']}
 
 setup(name='idf-build-apps',
-      version='2.8.1',
+      version='2.10.0',
       description='Tools for building ESP-IDF related apps.',
       author=None,
       author_email='Fu Hanxi <fuhanxi@espressif.com>',

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/tests/test_args.py

@@ -15,6 +15,7 @@ from idf_build_apps.args import (
     DependencyDrivenBuildArguments,
     FindArguments,
     FindBuildArguments,
+    expand_vars,
 )
 from idf_build_apps.constants import IDF_BUILD_APPS_TOML_FN, PREVIEW_TARGETS, SUPPORTED_TARGETS
 from idf_build_apps.main import main
@@ -408,3 +409,12 @@ dry_run = false
         assert test_suite.attrib['errors'] == '0'
         assert test_suite.attrib['skipped'] == '1'
         assert test_suite.findall('testcase')[0].attrib['name'] == 'bar/build'
+
+
+def test_expand_vars(monkeypatch):
+    assert expand_vars('Value is $TEST_VAR') == 'Value is '
+    monkeypatch.setenv('TEST_VAR', 'test_value')
+    assert expand_vars('Value is $TEST_VAR') == 'Value is test_value'
+    assert expand_vars('Value is $TEST_VAR and $NON_EXISTING_VAR') == 'Value is test_value and '
+    assert expand_vars('No variables here') == 'No variables here'
+    assert expand_vars('') == ''

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/tests/test_build.py

@@ -3,9 +3,11 @@
 
 import os
 import shutil
+import subprocess
 from copy import (
     deepcopy,
 )
+from pathlib import Path
 from xml.etree import (
     ElementTree,
 )
@@ -20,6 +22,7 @@ from idf_build_apps import (
     find_apps,
 )
 from idf_build_apps.app import (
+    App,
     CMakeApp,
 )
 from idf_build_apps.args import BuildArguments
@@ -27,6 +30,7 @@ from idf_build_apps.constants import (
     IDF_PATH,
     BuildStatus,
 )
+from idf_build_apps.utils import Literal
 
 
 @pytest.mark.skipif(not shutil.which('idf.py'), reason='idf.py not found')
@@ -205,6 +209,105 @@ class TestBuild:
         assert test_suite.findall('testcase')[0].attrib['name'] == 'foo/bar/build'
 
 
+class CustomClassApp(App):
+    build_system: Literal['custom_class'] = 'custom_class'  # type: ignore
+
+    def build(self, *args, **kwargs):
+        # For testing, we'll just create a dummy build directory
+        if not self.dry_run:
+            os.makedirs(self.build_path, exist_ok=True)
+            with open(os.path.join(self.build_path, 'dummy.txt'), 'w') as f:
+                f.write('Custom build successful')
+        self.build_status = BuildStatus.SUCCESS
+        print('Custom build successful')
+
+    @classmethod
+    def is_app(cls, path: str) -> bool:  # noqa: ARG003
+        return True
+
+
+@pytest.mark.skipif(not shutil.which('idf.py'), reason='idf.py not found')
+class TestBuildWithCustomApp:
+    custom_app_code = """
+from idf_build_apps import App
+import os
+from idf_build_apps.constants import BuildStatus
+from idf_build_apps.utils import Literal
+
+class CustomApp(App):
+    build_system: Literal['custom'] = 'custom'
+
+    def build(self, *args, **kwargs):
+        # For testing, we'll just create a dummy build directory
+        if not self.dry_run:
+            os.makedirs(self.build_path, exist_ok=True)
+            with open(os.path.join(self.build_path, 'dummy.txt'), 'w') as f:
+                f.write('Custom build successful')
+        self.build_status = BuildStatus.SUCCESS
+        print('Custom build successful')
+
+    @classmethod
+    def is_app(cls, path: str) -> bool:
+        return True
+    """
+
+    @pytest.fixture(autouse=True)
+    def _setup(self, tmp_path: Path, monkeypatch):
+        os.chdir(tmp_path)
+
+        test_app = tmp_path / 'test_app'
+
+        test_app.mkdir()
+        (test_app / 'main' / 'main.c').parent.mkdir(parents=True)
+        (test_app / 'main' / 'main.c').write_text('void app_main() {}')
+
+        # Create a custom app module
+        custom_module = tmp_path / 'custom.py'
+        custom_module.write_text(self.custom_app_code)
+
+        monkeypatch.setenv('PYTHONPATH', os.getenv('PYTHONPATH', '') + os.pathsep + str(tmp_path))
+
+        return test_app
+
+    def test_custom_app_cli(self, tmp_path):
+        subprocess.run(
+            [
+                'idf-build-apps',
+                'build',
+                '-p',
+                'test_app',
+                '--target',
+                'esp32',
+                '--build-system',
+                'custom:CustomApp',
+            ],
+            check=True,
+        )
+
+        assert (tmp_path / 'test_app' / 'build' / 'dummy.txt').exists()
+        assert (tmp_path / 'test_app' / 'build' / 'dummy.txt').read_text() == 'Custom build successful'
+
+    def test_custom_app_function(self, tmp_path):
+        # Import the custom app class
+        # Find and build the app using the imported CustomApp class
+        apps = find_apps(
+            paths=['test_app'],
+            target='esp32',
+            build_system=CustomClassApp,
+        )
+
+        assert len(apps) == 1
+        app = apps[0]
+        assert isinstance(app, CustomClassApp)
+        assert app.build_system == 'custom_class'
+
+        # Build the app
+        app.build()
+        assert app.build_status == BuildStatus.SUCCESS
+        assert (tmp_path / 'test_app' / 'build' / 'dummy.txt').exists()
+        assert (tmp_path / 'test_app' / 'build' / 'dummy.txt').read_text() == 'Custom build successful'
+
+
 def test_build_apps_collect_files_when_no_apps_built(tmp_path):
     os.chdir(tmp_path)
 

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/tests/test_cmd.py

@@ -1,4 +1,4 @@
-# SPDX-FileCopyrightText: 2024 Espressif Systems (Shanghai) CO LTD
+# SPDX-FileCopyrightText: 2024-2025 Espressif Systems (Shanghai) CO LTD
 # SPDX-License-Identifier: Apache-2.0
 import os
 import sys

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/tests/test_finder.py

@@ -1,7 +1,6 @@
 # SPDX-FileCopyrightText: 2022-2025 Espressif Systems (Shanghai) CO LTD
 # SPDX-License-Identifier: Apache-2.0
 
-import logging
 import os
 import tempfile
 from pathlib import (
@@ -443,7 +442,6 @@ class TestFindWithSdkconfigFiles:
             if not os.path.isfile(default_sdkconfig_path):
                 with open(default_sdkconfig_path, 'w') as fw:
                     fw.write('CONFIG_IDF_TARGET="linux"')
-                    logging.info('Created temp %s %s', DEFAULT_SDKCONFIG, default_sdkconfig_path)
                 _app = app
                 _default_sdkconfig_path = default_sdkconfig_path
                 break
@@ -456,7 +454,6 @@ class TestFindWithSdkconfigFiles:
         finally:
             try:
                 os.remove(_default_sdkconfig_path)
-                logging.info('Removed temp %s %s', DEFAULT_SDKCONFIG, _default_sdkconfig_path)
             except:  # noqa
                 pass
 

{idf_build_apps-2.8.1 → idf_build_apps-2.10.0}/tests/test_manifest.py

@@ -535,6 +535,28 @@ baz:
     }
 
 
+def test_folder_rule_introduced_by(tmp_path):
+    yaml_file = tmp_path / 'test.yml'
+    yaml_file.write_text(
+        """
+foo:
+  enable:
+    - if: IDF_TARGET == "esp32"
+    - if: IDF_TARGET == "esp32c3"
+bar:
+  enable:
+    - if: IDF_TARGET == "esp32"
+baz:
+  enable:
+    - if: IDF_TARGET == "esp32"
+""",
+        encoding='utf8',
+    )
+
+    manifest = Manifest.from_file(yaml_file)
+    assert manifest.most_suitable_rule('baz').by_manifest_file == str(yaml_file)
+
+
 class TestIfParser:
     def test_invalid_if_statement(self):
         statement = '1'

idf_build_apps-2.8.1/.github/workflows/check-pre-commit.yml

@@ -1,18 +0,0 @@
-name: pre-commit check
-
-on:
-  pull_request:
-
-jobs:
-  pre-commit:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-      - uses: actions/setup-python@v5
-      - id: changed-files
-        uses: tj-actions/changed-files@v45
-      - uses: pre-commit/action@v3.0.1
-        with:
-          extra_args: --files ${{ steps.changed-files.outputs.all_changed_files }}