vekexasia / comfoair-esp32

Interact with Zehnder Comfoair with esp32
Other
116 stars 38 forks source link

Deprecated warning for esp_intr.h #67

Closed wauswaus closed 4 months ago

wauswaus commented 4 months ago

During compile in ESPHome version 2024.2.2: `Compiling .pioenvs/esp-growatt-modbus/lib2a8/esp32_can/esp32_can_builtin_lowlevel.cpp.o In file included from .piolibdeps/esp-growatt-modbus/esp32_can/src/esp32_can_builtin_lowlevel.cpp:32: /data/cache/platformio/packages/framework-arduinoespressif32/tools/sdk/esp32/include/esp_hw_support/include/esp_intr.h:8:2: warning: #warning esp_intr.h is deprecated, please include esp_intr_alloc.h instead [-Wcpp]

warning esp_intr.h is deprecated, please include esp_intr_alloc.h instead`

vekexasia commented 4 months ago

hey, thanks for reporting. do you feel up for a pullrequest?

On Wed, Mar 6, 2024 at 8:12 PM wauswaus @.***> wrote:

During compile in ESPHome version 2024.2.2: Compiling .pioenvs/esp-growatt-modbus/lib2a8/esp32_can/esp32_can_builtin_lowlevel.cpp.o In file included from .piolibdeps/esp-growatt-modbus/esp32_can/src/esp32_can_builtin_lowlevel.cpp:32: /data/cache/platformio/packages/framework-arduinoespressif32/tools/sdk/esp32/include/esp_hw_support/include/esp_intr.h:8:2: warning: #warning esp_intr.h is deprecated, please include esp_intr_alloc.h instead [-Wcpp] #warning esp_intr.h is deprecated, please include esp_intr_alloc.h instead

— Reply to this email directly, view it on GitHub https://github.com/vekexasia/comfoair-esp32/issues/67, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABQ6S5CWQCB55VEAEMZS73YW5TBVAVCNFSM6AAAAABEJUIHFGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE3TEMRTGEYTEMI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

--

Andrea Baccega [image: @. [image: @. [image: Personal site] http://www.andreabaccega.com/ [image: Linkedin Profile] http://it.linkedin.com/in/andreabaccega [image: Facebook] https://www.facebook.com/andrea.baccega [image: Google+] https://plus.google.com/109217393200753135791 [image: Twitter] http://twitter.com/veke87 [image: StackOverflow] http://stackoverflow.com/users/314407/andrea-baccega [image: GitHub] https://github.com/vekexasia

wauswaus commented 4 months ago

phoe, never done that before but i'll give at a shot when i've got my stuff up and running

wauswaus commented 4 months ago

mmm looks like its not in your code but somewhere in the libs. Sorry for the trouble...