mirror of
https://github.com/espressif/esp-idf.git
synced 2024-10-05 20:47:46 -04:00
build & config: Don't include common components on early expansion
This commit is contained in:
parent
56bb81228c
commit
07827ea362
@ -222,6 +222,14 @@ function(__build_expand_requirements component_target)
|
||||
get_property(reqs TARGET ${component_target} PROPERTY REQUIRES)
|
||||
get_property(priv_reqs TARGET ${component_target} PROPERTY PRIV_REQUIRES)
|
||||
__component_get_property(component_name ${component_target} COMPONENT_NAME)
|
||||
__component_get_property(component_alias ${component_target} COMPONENT_ALIAS)
|
||||
idf_build_get_property(common_reqs __COMPONENT_REQUIRES_COMMON)
|
||||
list(APPEND reqs ${common_reqs})
|
||||
|
||||
if(reqs)
|
||||
list(REMOVE_DUPLICATES reqs)
|
||||
list(REMOVE_ITEM reqs ${component_alias} ${component_name})
|
||||
endif()
|
||||
|
||||
foreach(req ${reqs})
|
||||
depgraph_add_edge(${component_name} ${req} REQUIRES)
|
||||
|
@ -445,8 +445,7 @@ macro(project project_name)
|
||||
__component_get_target(main_target idf::main)
|
||||
__component_get_property(reqs ${main_target} REQUIRES)
|
||||
__component_get_property(priv_reqs ${main_target} PRIV_REQUIRES)
|
||||
idf_build_get_property(common_reqs __COMPONENT_REQUIRES_COMMON)
|
||||
if(reqs STREQUAL common_reqs AND NOT priv_reqs) #if user has not set any requirements
|
||||
if(NOT reqs AND NOT priv_reqs) #if user has not set any requirements
|
||||
if(test_components)
|
||||
list(REMOVE_ITEM build_components ${test_components})
|
||||
endif()
|
||||
|
@ -123,8 +123,6 @@ foreach(__component_target ${__component_targets})
|
||||
|
||||
__component_get_requirements()
|
||||
|
||||
list(APPEND __component_requires "${__common_reqs}")
|
||||
|
||||
# Remove duplicates and the component itself from its requirements
|
||||
__component_get_property(__component_alias ${__component_target} COMPONENT_ALIAS)
|
||||
__component_get_property(__component_name ${__component_target} COMPONENT_NAME)
|
||||
|
Loading…
x
Reference in New Issue
Block a user