fix(esp_https_ota): Add missing requirement to bootloader_support component

The `esp_https_ota.h` header includes `bootloader_common.h` from the
`bootloader_support` component, but the components CMakeLists.txt does not
depend on the `bootloader_support` component.
This causes compilation to fail when the header is included, unless the
component that the header belongs already depends on `bootloader_support`.

As the header is only needed in the `esp_https_ota.c` source file,
add a `PRIV_REQUIRES` for `bootloader_support` and move the include into the
.c file.
This commit is contained in:
Tom A. Wagner
2026-07-23 13:14:45 +02:00
parent 055ba9d3f9
commit a5cef637ae
3 changed files with 2 additions and 2 deletions

View File

@@ -8,6 +8,6 @@ idf_component_register(SRCS "src/esp_https_ota.c"
INCLUDE_DIRS "include"
REQUIRES esp_http_client esp_bootloader_format esp_app_format
esp_event esp_partition
PRIV_REQUIRES log app_update efuse)
PRIV_REQUIRES log app_update bootloader_support efuse)
idf_define_esp_err_codes(HEADERS include/esp_https_ota.h)

View File

@@ -7,7 +7,6 @@
#pragma once
#include <esp_http_client.h>
#include <bootloader_common.h>
#include "esp_app_desc.h"
#include "esp_bootloader_desc.h"
#include <sdkconfig.h>

View File

@@ -7,6 +7,7 @@
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <bootloader_common.h>
#include <esp_https_ota.h>
#include <esp_log.h>
#include <esp_ota_ops.h>