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

Build fails due to recent release of esp_insights (CON-1372) #1114

Closed
h2zero opened this issue Oct 10, 2024 · 5 comments
Closed

Build fails due to recent release of esp_insights (CON-1372) #1114

h2zero opened this issue Oct 10, 2024 · 5 comments

Comments

@h2zero
Copy link

h2zero commented Oct 10, 2024

Describe the bug
Compilation fails due to new version of esp_insights by commit

Environment

  • ESP-Matter Commit Id: v1.3.0-d76d4857

  • ESP-IDF Commit Id: v5.1.1-577-g6b1f40b9bf

  • SoC (eg: ESP32 or ESP32-C3): esp32

  • Device Logs (Please attach the log file): managed_components/espressif__esp_insights/src/esp_insights_cbor_encoder.c:176:22: error: 'SHA_SIZE' undeclared here

  • Host Machine OS: linux

  • Host Machine Python version: 3.10

I fixed this locally by adding espressif/esp_diag_data_store: "1.0.1" to my idf_component.yaml in the project folder but should be addressed here.

@github-actions github-actions bot changed the title Build fails due to recent release of esp_insights Build fails due to recent release of esp_insights (CON-1372) Oct 10, 2024
@shripad621git
Copy link
Contributor

shripad621git commented Oct 11, 2024

@h2zero , thanks for reporting the issue.It is a known reported issue in connectedhomeip. The fix for it is up in the connectedhomeip. We will get the fix out soon asap.

@h2zero
Copy link
Author

h2zero commented Oct 11, 2024

Thanks, but that doesn't resolve the compile issue when using esp-matter as a component. That same fix should be applied to the idf_component.yaml here and in the esp-matter component release as well.

@shubhamdp
Copy link
Contributor

@h2zero until we push the new esp-matter component to registry, please check #1115 (comment) that would help solve the problem till then.

@h2zero
Copy link
Author

h2zero commented Oct 11, 2024

@shubhamdp Thanks, I had already done something similar in my project as mentioned in the first post which will hopefully help others that will encounter this.

@shubhamdp
Copy link
Contributor

This was fixed in 704bed8

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

4 participants