espressif/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf.git synced 2026-04-27 19:13:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge branch 'contrib/github_pr_17594_v5.4' into 'release/v5.4'

Browse Source 
fix(depgraph): Add double quotes around dependency names (GitHub PR) (v5.4)

See merge request espressif/esp-idf!42493
This commit is contained in:
Roland Dobai
2025-10-27 16:30:50 +01:00
parent b7748ca2f9 f5fccb7f3d
commit dfd7fd391d
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tools/cmake/depgraph.cmake
+2 -2
View File
@@ -35,10 +35,10 @@ function(depgraph_add_edge dep_from dep_to)
# However, show which components are "common" by adding an edge from a node named "common".
# If necessary, add a new build property to customize this behavior.
if(NOT dep_from IN_LIST common_reqs)
idf_build_set_property(__BUILD_COMPONENT_DEPGRAPH "common -> ${dep_to}" APPEND)
idf_build_set_property(__BUILD_COMPONENT_DEPGRAPH "\"common\" -> \"${dep_to}\"" APPEND)
endif()
else()
idf_build_set_property(__BUILD_COMPONENT_DEPGRAPH "${dep_from} -> ${dep_to} ${attr}" APPEND)
idf_build_set_property(__BUILD_COMPONENT_DEPGRAPH "\"${dep_from}\" -> \"${dep_to}\" ${attr}" APPEND)
endif()
endfunction()
tools/test_apps/system/g1_components/check_dependencies.py
+2 -2
View File
@@ -26,8 +26,8 @@ def parse_dependencies(file_path: str) -> Tuple[Dict[str, List[str]], List[str]]
parts = line.split(' -> ')
if (len(parts) >= 2):
source = parts[0]
target = parts[1].split()[0] # Extracting the target component
source = parts[0].strip('"')
target = parts[1].split()[0].strip('"') # Extracting the target component
logging.debug(f'Parsed dependency: {source} -> {target}')
# Check that g1/g0 dependencies are either on the list of expected violations