Skip to content

Commit 1fb02a9

Browse files
fix(websocket): fix esp_event dependency management
Move esp_event from PRIV_REQUIRES to REQUIRES, as it is included by esp_websocket_client.h, so any code including that header also requires esp_event. This resolves the following build error for files not explicitly requiring esp_event: ninja: build stopped: subcommand failed. Compilation failed because esp_websocket_client.h (in "espressif__esp_websocket_client" component) includes esp_event.h, provided by esp_event component(s). However, esp_event component(s) is in the private requirements list of "espressif__esp_websocket_client". To fix this, move esp_event from PRIV_REQUIRES into REQUIRES list of idf_component_register call in ...\managed_components\espressif__esp_websocket_client\CMakeLists.txt. ninja failed with exit code 1, output of the command is in...
1 parent b5be844 commit 1fb02a9

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

components/esp_websocket_client/CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,6 @@ if(${IDF_TARGET} STREQUAL "linux")
1616
else()
1717
idf_component_register(SRCS "esp_websocket_client.c"
1818
INCLUDE_DIRS "include"
19-
REQUIRES lwip esp-tls tcp_transport http_parser
20-
PRIV_REQUIRES esp_timer esp_event)
19+
REQUIRES lwip esp-tls tcp_transport http_parser esp_event
20+
PRIV_REQUIRES esp_timer)
2121
endif()

0 commit comments

Comments
 (0)