ESP32: disabling ARDUINO_OTA fails the buil #3083
Comments
|
OK, this is clearly a missing |
TD-er
added
Category: Build
|
From my investigation I found that there are missing includes in ESPEasayStorage.ino #ifdef ESP32 //MFD: These were missing when not using the ARDUINO_OTA lib I can add a pull request for this if you want. |
|
@fmuntean If you could make a PR for it, that would be great. |
|
@TD-er : this issue seems to be still here in 20200721 release: This is the environment configuration I was able to build before but can't anymore lib_deps = https://github.com/TD-er/ESPEasySerial.git#v2.0.1, https://github.com/lbernstone/Tone, MFRC522 |
|
It would help if you also gave the build errors. If you don't like to share it online, you can also email me. Is the build error related to P012_LCD ? I tested with your config and I ran into a build error there which I cannot yet explain. |
|
I noticed MD5Builder is part of the Arduino_OTA library, so if you ignore that lib then MD5Builder will not work. I will have a look at it. |
|
I will make a commit with a few changes I made. |
I don't have a custom pre ESP32 script. This is the one that is part of the ESPEasy project. I also just realized that I don't need it. |
|
You might have run into a different build error than me due to the fact that you might use different plugins. // This selects only the plugins I use in my prod boards
#ifdef MFD_PLUGIN_SET_PROD
# define PLUGIN_SET_NONE
# define USES_P001 // Switch
# define USES_P002 // ADC
# define USES_P003 // Pulse
# define USES_P014 // SI7021
# define USES_P017 // PN532 RFID reader
# define USES_P025 // ADS1115
# define USES_P026 // SysInfo
# define USES_P028 // BME280
# define USES_P033 // Dummy
# define USES_P069 // LM75A Temperature Sensor
#define USES_P097 // Touch (ESP32)
# define USES_P105 //RGBW Light
# define USES_P123 // SI7013 Temperature Humindity and ADC
// # define USES_P129 // RC522 RFID reader
# define USES_P200 // ADS1115 Thermistor
# define CONTROLLER_SET_NONE
# define USES_C005 // OpenHAB MQTT
// # define NOTIFIER_SET_NONE //MFD: setting this removes the notification page
# ifdef NOTIFIER_SET_STABLE
# undef NOTIFIER_SET_STABLE
# endif // ifdef NOTIFIER_SET_STABLE
# ifdef NOTIFIER_SET_TESTING
# undef NOTIFIER_SET_TESTING
# endif // ifdef NOTIFIER_SET_TESTING
# ifdef NOTIFIER_SET_EXPERIMENTAL
# undef NOTIFIER_SET_EXPERIMENTAL
# endif // ifdef NOTIFIER_SET_EXPERIMENTAL
# ifdef NOTIFIER_SET_NONE
# undef NOTIFIER_SET_NONE
# endif // ifdef NOTIFIER_SET_NONE
# define USES_N002 // Buzzer
#endif // ifdef MFD_PLUGIN_SET_PROD |
|
With the pending PR and the esp32 env for in your .ini file I can build it just fine with this plugin set you showed. |
Great: let's have this PR moved forward and I can test it locally. |
|
It has been merged. |
|
tested with latest and now builds successful. |
I am trying to disable the ARDUINO_OTA feature for ESP32 board by commenting out the
#define FEATURE_ARDUINO_OTA from inside the ESPEasy_common.h
The build fails with the following errors:
This was possible for the ESP8266 boards before.
The text was updated successfully, but these errors were encountered: