Skip to content

Update library to 3.10.0 generate compiling error on ESP32 #175

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

Closed
hpsaturn opened this issue Jan 29, 2022 · 2 comments · Fixed by #176
Closed

Update library to 3.10.0 generate compiling error on ESP32 #175

hpsaturn opened this issue Jan 29, 2022 · 2 comments · Fixed by #176

Comments

@hpsaturn
Copy link
Contributor

Overview

Actual behavior:

Compiling error:

Compiling .pio/build/TTGO_T7/lib074/Update/HttpsOTAUpdate.cpp.o
In file included from .pio/libdeps/TTGO_T7/ESP8266 Influxdb/src/query/FluxTypes.cpp:28:0:
.pio/libdeps/TTGO_T7/ESP8266 Influxdb/src/query/FluxTypes.h: In constructor 'FluxString::FluxString(String, String, const char*)':
.pio/libdeps/TTGO_T7/ESP8266 Influxdb/src/query/FluxTypes.h:131:12: error: 'FluxString::value' will be initialized after [-Werror=reorder]
     String value;
            ^
.pio/libdeps/TTGO_T7/ESP8266 Influxdb/src/query/FluxTypes.h:127:17: error:   'const char* FluxString::_type' [-Werror=reorder]
     const char *_type;
                 ^
.pio/libdeps/TTGO_T7/ESP8266 Influxdb/src/query/FluxTypes.cpp:147:1: error:   when initialized here [-Werror=reorder]
 FluxString::FluxString(String rawValue, String value, const char *type):FluxBase(rawValue),value(value),_type(type)
 ^
Compiling .pio/build/TTGO_T7/lib074/Update/Updater.cpp.o
Compiling .pio/build/TTGO_T7/libc61/FS/FS.cpp.o

Steps to reproduce:
List the minimal actions needed to reproduce the behavior.

  1. Updated dependencie from 3.9.0 to 3.10.0
  2. Compiling over ESP32 Arduino Framework using PlatformIO

Expected behavior:

Backward compatibility

Specifications:

  • Library Version: 3.10.0
  • InfluxDB Version: NA
  • Board/ESP chip: ESP32 (lolin32)
  • Device Arduino SDK version: Espressif 32 3.4.0
@vlastahajek
Copy link
Collaborator

Thanks for the issue. I don't use PlatformIO. Arduino core for ESPs has only one compiler option to treat a specific warning as an error. I've pushed a fix that clears all warnings, so it should work for you.

Could you, please, add the full command line for compiling projects used by PlatformIO?

@hpsaturn
Copy link
Contributor Author

hpsaturn commented Jan 30, 2022

The current changes fixed the issue. Thanks.

Also I want say that PlatformIO is better than Arduino CLi and Arduino IDE, you should be consider that, and it is more easy too, because Pio includs Arduino Framework too.

Proof of that, I did a pull request that test the complete library over ESP32 and ESP8266 using continuous integration workflow with PlatformIO and GitHub Workflows. #177

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

Successfully merging a pull request may close this issue.

2 participants