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

DEBUG log ondersteuning voor laatste ESPEasy build #134

Open
BasvanH opened this issue Jan 18, 2023 · 3 comments
Open

DEBUG log ondersteuning voor laatste ESPEasy build #134

BasvanH opened this issue Jan 18, 2023 · 3 comments

Comments

@BasvanH
Copy link
Contributor

BasvanH commented Jan 18, 2023

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
Copy link
Owner

Super! Ik ga dit weekend ernaar kijken.

@BasvanH
Copy link
Contributor Author

BasvanH commented Apr 3, 2023

Was je hier inmiddels aan toegekomen?

@BasvanH
Copy link
Contributor Author

BasvanH commented Oct 28, 2024

Zou je hier nog eens naar willen kijken en mogelijk je code willen updaten / aanpassen naar de laatste ESPEasy build?

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

2 participants