GrKoR / esphome_aux_ac_component

ESPHome component for AUX based air conditioners. Direct wifi control of HVAC by ESPHome and Home Assistant. HVAC status and command feedback are available.
Other
177 stars 42 forks source link

При попытке обновиться: #72

Closed arsenal3008 closed 10 months ago

arsenal3008 commented 1 year ago
Compiling /data/rovex/.pioenvs/rovex/src/main.cpp.o
In file included from src/esphome/core/controller.h:29,
                 from src/esphome/components/api/api_server.h:4,
                 from src/esphome/components/api/api_connection.h:6,
                 from src/esphome.h:3,
                 from src/main.cpp:3:
src/esphome/components/climate/climate.h: In constructor 'esphome::climate::Climate::Climate()':
src/esphome/components/climate/climate.h:167:7: warning: 'esphome::climate::Climate::away' is deprecated: away is deprecated, use preset instead [-Wdeprecated-declarations]
  167 | class Climate : public EntityBase {
      |       ^~~
src/esphome/components/climate/climate.h:193:8: note: declared here
  193 |   bool away{false};
      |        ^~~~
src/esphome/components/climate/climate.h:167:7: warning: 'esphome::climate::Climate::away' is deprecated: away is deprecated, use preset instead [-Wdeprecated-declarations]
  167 | class Climate : public EntityBase {
      |       ^~~
src/esphome/components/climate/climate.h:193:8: note: declared here
  193 |   bool away{false};
      |        ^~~~
In file included from src/esphome/components/aux_ac/automation.h:3,
                 from src/esphome.h:15,
                 from src/main.cpp:3:
src/esphome/components/aux_ac/aux_ac.h: In constructor 'esphome::aux_ac::AirCon::AirCon()':
src/esphome/components/aux_ac/aux_ac.h:724:7: note: synthesized method 'esphome::climate::Climate::Climate()' first required here
  724 | class AirCon : public esphome::Component, public esphome::climate::Climate {
      |       ^~
src/main.cpp: In function 'void setup()':
src/main.cpp:382:31: note: synthesized method 'esphome::aux_ac::AirCon::AirCon()' first required here
  382 |   aux_id = new aux_ac::AirCon();
      |                               ^
GrKoR commented 1 year ago

Бегло глянул. Вообще нигде в коде не использую Away(), о котором пишется в логах. Запрет Away(), как и кучи других методов, был несколько лет назад. Тогда всё сделал с учетом их требований. Такое ощущение, что в последней версии ESPHome что-то поменяли внутри и сами наступили на эти грабли. Должны бы поправить в ближайших релизах.

@arsenal3008 , если после следующих релизов ESPHome уведомление останется, тегни меня, пожалуйста. Буду глубже вникать. Пока выглядит так, что в компоненте всё правильно. На работу сейчас не влияет, и похоже не повлияет в будущем.

GrKoR commented 10 months ago

закрыл, так как проблема видимо ушла с обновдением ESPHome