Skip to content

Build error with platformio/framework-arduinoespressif32 @ ~3.10006.0 #5014

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
gin66 opened this issue Apr 5, 2021 · 2 comments
Closed

Build error with platformio/framework-arduinoespressif32 @ ~3.10006.0 #5014

gin66 opened this issue Apr 5, 2021 · 2 comments

Comments

@gin66
Copy link

gin66 commented Apr 5, 2021

Build error with framework-arduinoespressif32 @ ~3.10006.0 as per this log. Works well with framework-arduinoespressif32 @ 3.10005.210308 (see log ):

Tool Manager: Installing platformio/framework-arduinoespressif32 @ ~3.10006.0
123
Downloading...
124
Unpacking...
125
Tool Manager: framework-arduinoespressif32 @ 3.10006.210326 has been installed!
126
/home/runner/.platformio/packages/framework-arduinoespressif32/cores/esp32/esp32-hal-spi.c: In function 'spiTransferBytesNL':
127
/home/runner/.platformio/packages/framework-arduinoespressif32/cores/esp32/esp32-hal-spi.c:922:39: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
128
                 uint8_t * last_out8 = &result[c_longs-1];
129
                                       ^
130
/home/runner/.platformio/packages/framework-arduinoespressif32/cores/esp32/esp32-hal-spi.c:923:40: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
131
                 uint8_t * last_data8 = &last_data;
132
                                        ^
133
cc1: all warnings being treated as errors
134
*** [.pio/build/esp32/FrameworkArduino/esp32-hal-spi.c.o] Error 1
@me-no-dev
Copy link
Member

you would need to lower the log/debug level of the build cc1: all warnings being treated as errors is the problem

@gin66
Copy link
Author

gin66 commented Apr 5, 2021

Not a good idea to lower warning level. Never mind. The latest (huge) commit from #4996 fixes this problem by adding a (uint8_t *) before the &. So the next release will resolve the issue.

@gin66 gin66 closed this as completed Apr 5, 2021
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

No branches or pull requests

2 participants