Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to 1.3.1 breaks my build (ESF-143) #112

Closed
pptz opened this issue Jun 27, 2024 · 3 comments
Closed

Update to 1.3.1 breaks my build (ESF-143) #112

pptz opened this issue Jun 27, 2024 · 3 comments

Comments

@pptz
Copy link

pptz commented Jun 27, 2024

Port

esp32

Target chip

esp32s3

Hardware Configuration

The problem is with the build, so hardware is not relevant.

Log output

/Users/dev/dev/nami-esp/nami-esp/components/nami/src/rcp_upgrade.c: In function 'flash_binary':
/Users/dev/dev/nami-esp/nami-esp/components/nami/src/rcp_upgrade.c:102:11: error: implicit declaration of function 'esp_loader_flash_verify'; did you mean 'esp_loader_flash_write'? [-Werror=implicit-function-declaration]
  102 |     err = esp_loader_flash_verify();
      |           ^~~~~~~~~~~~~~~~~~~~~~~
      |           esp_loader_flash_write
cc1: some warnings being treated as errors
ninja: build stopped: subcommand failed.

More Information

The call to esp_loader_flash_verify() fails to link with 1.3.1. It works with all previous versions.

@github-actions github-actions bot changed the title Update to 1.3.1 breaks my build Update to 1.3.1 breaks my build (ESF-143) Jun 27, 2024
@pptz
Copy link
Author

pptz commented Jun 27, 2024

Screenshot 2024-06-27 at 6 06 48 PM

This undefines MD5 and hence the mentioned function. Is this intended or accidental?

@DNedic
Copy link
Contributor

DNedic commented Jul 1, 2024

Hello @pptz , thank you for the submission, It looks like 4ad452d is the culprit due to the way that MD5_ENABLED is the only option we have not prefixed with SERIAL_FLASHER_, and the new add_option() function not accounting for that. We will fix this shortly.

@wnylei
Copy link

wnylei commented Jul 2, 2024

遇到了同样的问题, 由于 MD5_ENABLED 没有定义,导致报错 esp_loader_flash_verify 没有声明。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants