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

Move flashLED & setLamp to start of file #41

Merged
merged 1 commit into from
Sep 16, 2020
Merged

Move flashLED & setLamp to start of file #41

merged 1 commit into from
Sep 16, 2020

Conversation

xtrinch
Copy link
Contributor

@xtrinch xtrinch commented Sep 16, 2020

If anyone tries to build and upload this with PlatformIO (esp32-cam-webserver.ino ->> main.cpp), they will be faced with compilation errors ("function not defined").
Moved the flashLED and setLamp to the start of the file to avoid those errors.

@easytarget easytarget merged commit 387dc07 into easytarget:master Sep 16, 2020
@easytarget easytarget added the bug Something isn't working label Sep 16, 2020
@easytarget
Copy link
Owner

:-) Thankyou!
I try to remember this should build in 'normal' make use, but the ArduinoIDE additional preprocessing makes it so easy to be lazy. But in keeping with the 'example' bit of the project I am only building and developing on the official IDE for now ;-)

I am, however, about to start an (unrelated) project that I intend to do as a PlatformIO one from scratch (and learn it that way), I might well come back after v3 and, at least, set up a Travis/PlatformIO build in parallel to the ArduinoIDE one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants