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

Dependencies: Update platform SDK for node-wifi-mqtt.ino program #71

Open
amotl opened this issue May 29, 2023 · 0 comments
Open

Dependencies: Update platform SDK for node-wifi-mqtt.ino program #71

amotl opened this issue May 29, 2023 · 0 comments

Comments

@amotl
Copy link
Member

amotl commented May 29, 2023

Original: #58 (comment)

Problem

Updating the ESP8266 SDK on the node-wifi-mqtt.ino program to espressif8266@^3 with GH-67 failed at 1. It is probably our fault, because we are using other outdated libraries, or such.

Observations

After looking into the details, it turns out joaolopesf/SerialDebug might trip the build, as we receive similar error messages like outlined in JoaoLopesF/SerialDebug#20 when updating to espressif8266@^3.

SerialDebug.cpp:2384:46: error: reference to 'byte' is ambiguous
SerialDebug.cpp:2384:46: error: 'byte' has not been declared
SerialDebug.cpp:2486:61: error: reference to 'byte' is ambiguous
SerialDebug.cpp:5608:21: error: expected primary-expression before '(' token

Regarding this SerialDebug library, I can find those comments in the changelog of the node-wifi-mqtt.ino program we are trying to build here:

 2017-04-07 Silence debug output by using SerialDebugger to reduce battery drain even more.
 2021-05-02 Modernize code by using the SerialDebug library by Joao Lopes and
            add build environment based on PlatformIO.

Is it sane? Do we need it for the purposes outlined there, to reduce battery drain even more? Can it be safely removed? Maybe there are better techniques "to turn off logging at runtime" in a similar manner. On the other hand, using the SerialDebug library may have been utter nonsense ;].

With kind regards,
Andreas.

P.S.: Those commands should get anyone interested to the right spot quickly.

git clone https://github.com/hiveeyes/arduino hiveeyes-arduino --branch amo/update-esp8266-sdk
cd hiveeyes-arduino/backdoor/node-wifi-mqtt
make

Footnotes

  1. https://github.com/hiveeyes/arduino/actions/runs/5113724917/jobs/9193256917?pr=67#step:8:130

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

1 participant