Next version of espressif-esp32 will fail to compile

[schlaegerz]

I'm trying out the newest espressif esp32 libraries and this library fails to compile

#include <hwcrypto/sha.h>
is being moved to
#include <esp32/sha.h>

I don't quite know what the solution should be, but just thought I would put this here so a decision can be made before this gets updated for most people.

[Links2004]

the travis job is building against https://github.com/espressif/arduino-esp32.git master, and has not shown any include problems, can you give more details?

[chemicstry]

There is idf-release/v4.0 branch, which is based on esp-idf v4 and it has changed include path as indicated above. It will fail to compile once it is released in master. Unfortunately, I couldn't find a way to test for idf >= 4 version, because older idfs don't have version defines.

Platform.io already uses esp-idf v4 and compiling arduinoWebSockets fails

EDIT: I found a way to test for IDF version and support both:

#include "esp_system.h"

#if ESP_IDF_VERSION_MAJOR >= 4
#include <esp32/aes.h>
#else
#include <hwcrypto/aes.h>
#endif

I will submit a PR when I have time if it's not fixed by then.