search

arnemauer / Ducobox-ESPEasy-Plugin

Plugin voor ESPEasy om een Ducobox Silent / Focus / WTW Energy Comfort aan te sturen en sensoren uit te lezen. Er zijn twee plugins: een ducobox RF gateway en ducobox serial gateway.
75 stars 16 forks source link

DEBUG log ondersteuning voor laatste ESPEasy build #134

Open BasvanH opened 1 year ago

BasvanH commented 1 year ago

Tijdens het oplossen van #125 liep ik tegen het probleem aan dat ik build fouten kreeg op LOG_LEVEL_DEBUG. Sinds de november 5 release van ESPEasy is er een global variable gemaakt genaamd BUILD_NO_DEBUG. Deze bepaald of een debug log gedeelte wel of niet in de build moet worden meegenomen om zo ruimte te besparen. Standaard wordt met de ESP8266 1M en 4M build de debug logs blokken niet meer meegenomen. Dit betekend dat de log plekken in de Ducobox plugin moeten worden nagelopen of deze echt voor debug zijn of toch meer informationeel. Voor informationeel moet het dan gewijzigd worden naar LOG_LEVEL_INFO, en voor echt debug (wat dus standaard niet meer in de build zal meegaan) zal er een wrap rondom dat gedeelte heen moeten.

# ifndef BUILD_NO_DEBUG
// De debug log code
# endif // ifndef BUILD_NO_DEBUG

Ik kan niet beoordelen wel stuk dus INFO of DEBUG moet zijn, dus @arnemauer als je wilt dat de Ducobox plugin ondersteuning voor die versie en later gaat krijgen is dit een aandachtspunt.

Ik heb simpelweg alles naar LOG_LEVEL_INFO omgezet en dat werkt prima. De plugin werkt dan (na toepassing van #125) helemaal zonder problemen.

arnemauer commented 1 year ago

Super! Ik ga dit weekend ernaar kijken.

BasvanH commented 1 year ago

Was je hier inmiddels aan toegekomen?