-
-
Notifications
You must be signed in to change notification settings - Fork 408
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
#ifdef HAL_ESP32_HAL_H_ #632
Comments
I rolled back OneBitDisplay to v1.5.0, this should work. |
With v1.5.0 it does not work either for me.
|
Sorry for my beginner question: How did you do this and where can I find this file? |
Please post platformio version and platform (Win/Linux). |
Core 5.0.0·Home 3.3.0 on macOS |
allright, i never tested pio 5.0.0, still sticking to 4.3.4 |
I can confim to this issue using following config: As workaround I tried following statement in OneBitDisplay.cpp |
I tried to load ESP32 BLE library by adding a lib_deps statement in platformio.ini. Still compiler error with OneBitDisplay. Checked, that the files (BLEDevice.h, etc.) are present in the .pio build environment, and yes, they are. Weird. Can anyone help? |
I can also confirm on: |
I got the code now compiling by modifing the file (Do not forget to press "Clear" before rebuilding or delete .pio\build\usb manually)
Maybe this information helps somehow? |
@pbriesch #include <SPI.h> #ifdef HAL_ESP32_HAL_H_ // added #ifdef HAL_ESP32_HAL_H_ The lines I added are between the include and the ifdef. Simply said: I undefd this define. Then it compiled and the code worked for me. My environment: |
Thanks, is working now! |
As described in my workaround there is a difference when building: Without the depends in library.properties platformio recognizes that OneBitDisplay lib depends on "ESP32 BLE Arduino"
Without that change platformio fails to recognize the dependecy and does not add "ESP32 BLE Arduino"
|
Okay, somehow the Library Dependency Finder (LDF) of Platform.io misses to detect the dependencies of OneBitDisplay. Add "lib_ldf_mode = deep" to your platformio.ini in the [env] section, like this Clean (or delete whole .pio folder), build again and it should compile now |
…isplay correctly and not end in compiler error
Fix for #632 - LDF will recognize BLE dependency of OneBitDisplay correctly
Thanks for investigation and fixing! Tested & Merged. |
with the latest update of PlatformIO I was no longer able to compile the code. The problem was the statement
#ifdef HAL_ESP32_HAL_H_
in the file OneBitDisplay.cpp.
This is te error-message:
I had to undefine it explicitly. Then it compiles and the code worked.
The text was updated successfully, but these errors were encountered: