Build error on ESP32S3

[jrr]

Platform

ESP32

IDE / Tooling

PlatformIO

What happened?

In a platformio project targeting board = esp32s3_n16r8, a recent ESPAsyncWebServer version update provoked this new build error:

.pio/libdeps/app/ESPAsyncWebServer/src/WebHandlers.cpp: In member function 'virtual void AsyncStaticWebHandler::handleRequest(AsyncWebServerRequest*)':
.pio/libdeps/app/ESPAsyncWebServer/src/WebHandlers.cpp:241:34: error: format '%lx' expects argument of type 'long unsigned int', but argument 4 has type 'uint32_t' {aka 'unsigned int'} [-Werror=format=]
     snprintf(etag, sizeof(etag), "%08lx", etagValue);
                                  ^~~~~~~  ~~~~~~~~~
cc1plus: some warnings being treated as errors
*** [.pio/build/app/lib2bb/ESPAsyncWebServer/WebHandlers.cpp.o] Error 1

It looks like it was probably introduced in #303 .

Compiler version:

~/.platformio/packages/toolchain-xtensa-esp32s3/bin/xtensa-esp32s3-elf-g++ --version
xtensa-esp32s3-elf-g++ (crosstool-NG esp-2021r2-patch5) 8.4.0
Copyright (C) 2018 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

When I have some I'll try to contribute a repro and/or fix.

Stack Trace

n/a

Minimal Reproductible Example (MRE)

• create a platformio project targeting esp32s3_n16r8
• add ESP32Async/ESPAsyncWebServer@3.9.2
• pio run

I confirm that:

• I have read the documentation.
• I have searched for similar discussions.
• I have searched for similar issues.
• I have looked at the examples.
• I have upgraded to the lasted version of ESPAsyncWebServer (and AsyncTCP for ESP32).

[mathieucarbou]

This is not an error but a warning that your compiler flags are treating as an error. You are probably using an outdated plateform version. Use pioarduino.

[mathieucarbou]

@jrr also see: #352

[jrr]

Thanks for the quick attention! So it sounds like ESPAsyncWebServer only intends to support pioarduino, not platformio's old stale version?

(if so, that might be worth a big bold warning in the readme)

[mathieucarbou]

We do compile on arduin ocore 2 (if you check our CI - we compile on arduino 2), but arduino core 2 is not unsupported. Everybody should switch to 3. And yes, pio team stopped supporting the arduino platform so pioarduino is the way to go now.

[mathieucarbou]

(if so, that might be worth a big bold warning in the readme)

if you read the readme, we clearly show the right platform to use:

https://github.com/ESP32Async/ESPAsyncWebServer?tab=readme-ov-file#esp32--pioarduino

But like I said, we still compile and work on arduino core 2. But you have to deal with the warnings if some appear one day because we do not test on arduino core 2 anymore.

[mathieucarbou]

This PR will fix the warning one merged: #357