espressif / esp-insights

ESP Insights: A remote diagnostics/observability framework for connected devices
Apache License 2.0
101 stars 27 forks source link

Issues regarding migrating ESP-Insights from idf-V4.4 to idf-V5.0 #22

Closed Munjalnyk closed 1 year ago

Munjalnyk commented 1 year ago

there are errors with deprecated file name for eg. sntp.h has been upgraded as esp_sntp.h and there many more file names such that. Anyone has figured it out or process is still in development. can some one provide migrating guide??

vikramdattu commented 1 year ago

Hi @yorocoboy1 thanks for reporting. We will take a look and fix this. Ideally, there should not be a need to make any explicit changes to make Insights work with IDF 5.0 as we have added compatibility changes and we keep fixing it as IDF is changing.
The error you're getting are because you are treating deprecation warnings as errors. Please check if you are adding stricter compiler flags, -Werror=deprecated-declarations in this case.

vikramdattu commented 1 year ago

cc @sanketwadekar

shahpiyushv commented 1 year ago

This should have been sorted by now. Closing this.