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 when log level set to none. Observed the same issue with esp matter examples as well (Tried with example matter light) (CON-918) #758

Closed
aggaddam opened this issue Nov 30, 2023 · 5 comments

Comments

@aggaddam
Copy link

aggaddam commented Nov 30, 2023

Describe the bug
Build fails when the log level is set to none and observed the same issue with esp matter examples as well (Tried with example matter light). Here is the error that occurred during the build process

Environment

  • ESP-Matter Commit Id: 68e99de
  • ESP-IDF Commit Id: v5.1.1 Stable Release
  • SoC (eg: ESP32 or ESP32-C3): ESP32-C3
  • Device Logs (Please attach the log file):
include_esp_phy_override  -lphy  -lbtbb  -lphy  -lbtbb  esp-idf/esp_phy/libesp_phy.a  -u vfs_include_syscalls_impl && :
/home/agaddam/.espressif/tools/riscv32-esp-elf/esp-12.2.0_20230208/riscv32-esp-elf/bin/../lib/gcc/riscv32-esp-elf/12.2.0/../../../../riscv32-esp-elf/bin/ld: esp-idf/esp_wifi/libesp_wifi.a(esp_adapter.c.obj): in function `esp_log_writev_wrapper':
/home/agaddam/Documents/test/li_ac_matter_fw/targets/simplifi/AC/li_esp32matter_sdk/esp-idf/components/esp_wifi/esp32c3/esp_adapter.c:372: undefined reference to `__wrap_esp_log_writev'
/home/agaddam/.espressif/tools/riscv32-esp-elf/esp-12.2.0_20230208/riscv32-esp-elf/bin/../lib/gcc/riscv32-esp-elf/12.2.0/../../../../riscv32-esp-elf/bin/ld: esp-idf/esp_wifi/libesp_wifi.a(esp_adapter.c.obj): in function `esp_log_write_wrapper':
/home/agaddam/Documents/test/li_ac_matter_fw/targets/simplifi/AC/li_esp32matter_sdk/esp-idf/components/esp_wifi/esp32c3/esp_adapter.c:360: undefined reference to `__wrap_esp_log_writev'
/home/agaddam/.espressif/tools/riscv32-esp-elf/esp-12.2.0_20230208/riscv32-esp-elf/bin/../lib/gcc/riscv32-esp-elf/12.2.0/../../../../riscv32-esp-elf/bin/ld: esp-idf/esp_http_client/libesp_http_client.a(esp_http_client.c.obj): in function `esp_http_client_read':
/home/agaddam/Documents/test/li_ac_matter_fw/targets/simplifi/AC/li_esp32matter_sdk/esp-idf/components/esp_http_client/esp_http_client.c:1164: undefined reference to `__wrap_esp_log_write'
/home/agaddam/.espressif/tools/riscv32-esp-elf/esp-12.2.0_20230208/riscv32-esp-elf/bin/../lib/gcc/riscv32-esp-elf/12.2.0/../../../../riscv32-esp-elf/bin/ld: /home/agaddam/Documents/test/li_ac_matter_fw/targets/simplifi/AC/li_esp32matter_sdk/esp-idf/components/esp_http_client/esp_http_client.c:1164: undefined reference to `__wrap_esp_log_write'
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.
ninja failed with exit code 1, output of the command is in the /home/agaddam/Documents/test/li_ac_matter_fw/targets/simplifi/AC/build/log/idf_py_stderr_output_14766 and /home/agaddam/Documents/test/li_ac_matter_fw/targets/simplifi/AC/build/log/idf_py_stdout_output_14766


MAKE FAIL

  • Host Machine OS: Ubuntu
  • Commissioner app and versions if present: NA
  • Commissioner's logs if present: NA

Any additional details
...

@github-actions github-actions bot changed the title Build fails when log level set to none. Observed the same issue with esp matter examples as well (Tried with example matter light) Build fails when log level set to none. Observed the same issue with esp matter examples as well (Tried with example matter light) (CON-918) Nov 30, 2023
@sayondeep
Copy link
Contributor

@aggaddam Can you please enable "Use external log wrapper" and try again..
It can be enabled at Component config -> Diagnostics -> Use external log wrapper

@aggaddam
Copy link
Author

@sayondeep It worked. Thank you!

Can i always enable this setting Component config -> Diagnostics -> Use external log wrapper irrespective of log level state?

@shubhamdp
Copy link
Contributor

@aggaddam We have identified the fix and we'll be out soon. The fix is in esp-insights framework. It is used in connectedhomeip. After the fix is out, we need to bump the esp-insights version in connectedhomeip and then update the submodule here, ideally.

Untill them please use the work-around.

@shubhamdp
Copy link
Contributor

@aggaddam We have fixed the problem in espressif/esp-insights@2990bb3 and you can remove the workaround.

I was wrong about the second part where we update connectedhomeip and then esp-matter... I just works without any changes.

Also, please close the issue once you verify it.

@dhrishi
Copy link
Collaborator

dhrishi commented Dec 8, 2023

Closing. Please re-open in case of any issues

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