-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
Virtual functions in client.h are out of line with standard Arduino API #2755
Comments
I've got the same problem. After downgrading to 1.0.1 it works again. |
HardwareBoard: WeMos WiFi and Bluetooth Battery DescriptionWe are working on multi communication device (Wifi, Ethernet, RS232, Bluetooth). Last week we cannot compile source codes, because there is some errors in Ethernet library, described bellow. I tried to compile some basic Arduino examples (WebClient) in Arduino IDE and I got same resolution.
This was also mentioned in Arduino Libraries Issues |
We're dealing with the exactly same issue. Could you please, please 👍 look at it? :) |
Same issue here. |
Same issue with ArduinoHTTPClient and Adafruit IO Arduino, @me-no-dev will this change be reverted in a future 1.0.3 release similar to what happened with the Arduino ESP8266 core. |
We fixed this incompatibility by downgrading Espressif32 platform core from 1.8.0 to 1.7.0. Just put line bellow into platformio.ini
If some other libraries was updated and some errors will be show, then try downgrade libraries by the same way or put the old libraries code directly into the project directory and update #include directives. Good Luck
|
@thujer Yep, you can downgrade to retain the old feature. Point is - the function signatures should be in-line with the Arduino API in a new release of the ESP32 BSP. |
@me-no-dev Any updates on this? |
This is required to make Ethernet library compile (see espressif/arduino-esp32#2755)
Hi Guys! We're also trying to implement an IoT solution that require an updated library when using TinyGSM, so it would be a bloody shame to have to abandon 2 libraries (and perhaps even more) just because this one won't play ball. Especially since several other things already depend on this. I hope you can reconsider and prioritize this issue. Cheers, |
I had to implement my own HttpClient (it is very simple, with support only for GET). Like @eabase said, why don't you prioritize this issue? It is simple to solve and it would help a lot of users!! |
@me-no-dev Any updates on this issue? |
Hi, ArduinoHttpClient library has the same problem, please review to correct this..are a lot a libraries with problems |
The Problem still exists with 1.0.3-rc3 |
Sorry guys! I get hundreds of emails daily and it's not possible to read them all. Please, stop by gitter (link in the readme) and drop a message if something is seriously wrong. I try to keep an eye there. Issue is fixed and will be in 1.0.3 final. |
@SRGDamia1 the 1.0.3 final update was released!!! |
1.0.3 works with Adafruit IO Arduino and Adafruit MQTT, thanks @me-no-dev. |
Thanks! |
There was a regression of this bug in #10202 Examples of breakage caused: |
It breaks also arduino-libraries/ArduinoMqttClient#112 |
Hardware:
Board: ESP32
Core Installation version: 1.0.2
IDE name: PlatformIO
Flash Frequency: N/A
PSRAM enabled: N/A
Upload Speed: N/A
Computer OS: Linux (TravisCI)
Description:
I am working on TinyGSM - a cross-platform GSM client. Because the client.h file in the current ESP32 core does not match the standard Arduino API (https://github.com/arduino/ArduinoCore-API/blob/master/api/Client.h) the library can no longer be compiled by esp32 users.
NOTE: The ESP8266 core introduced the same problem in version 2.5.0. They are fixing it for 2.5.1 (esp8266/Arduino#5969)
Sketch: (leave the backquotes for code formatting)
Debug Messages:
Unable to compile because of undefined virtual functions:
This was also mentioned in #2704.
The text was updated successfully, but these errors were encountered: