mirror of
https://github.com/espressif/esp-idf.git
synced 2026-04-27 11:03:11 +00:00
fix(esp_wifi): Update esp_wifi dependencies on non-WiFi targets for cmakev2
This commit updates the dependencies list for the esp_wifi component when compiled for non-WiFi targets. The reason for this change is because cmakev2 explicitly includes the esp_wifi component even on non-WiFi targets leading to compilation errors due to missing dependencies.
This commit is contained in:
@@ -15,10 +15,26 @@ if( NOT CONFIG_ESP_WIFI_ENABLED
|
||||
"src/wifi_netif.c"
|
||||
"src/wifi_default_ap.c")
|
||||
|
||||
# In build system v2, idf_component_optional_requires() includes the target
|
||||
# component into the build immediately, so these stub sources are compiled
|
||||
# even on targets where Wi-Fi is not supported. The stubs need esp_event and
|
||||
# esp_netif headers, so those dependencies must be declared here for v2.
|
||||
#
|
||||
# In build system v1, idf_component_optional_requires() only links a
|
||||
# component that is already part of the build. On non-Wi-Fi targets, this
|
||||
# component is never pulled in, so the stubs are never compiled and the
|
||||
# dependencies are not needed. Additionally, the CMAKE_BUILD_EARLY_EXPANSION
|
||||
# guard ensures that the stubs are not compiled for v1.
|
||||
set(priv_reqs "")
|
||||
if(IDF_BUILD_V2)
|
||||
set(priv_reqs "esp_event" "esp_netif")
|
||||
endif()
|
||||
|
||||
# This component provides "esp_wifi" "wifi_apps/nan_app" headers if WiFi not enabled
|
||||
# (implementation supported optionally in a managed component esp_wifi_remote)
|
||||
idf_component_register(SRCS "${srcs}"
|
||||
INCLUDE_DIRS "include" "wifi_apps/nan_app/include")
|
||||
INCLUDE_DIRS "include" "wifi_apps/nan_app/include"
|
||||
PRIV_REQUIRES ${priv_reqs})
|
||||
add_subdirectory(remote) # wifi-remote on esp32p4/h2 (no wifi)
|
||||
return()
|
||||
endif()
|
||||
|
||||
Reference in New Issue
Block a user