search

shining-man / bsc_fw

Firmware battery safety controller (BSC)
MIT License
98 stars 19 forks source link

Second include of `LITTLEFS.h` in main.cpp, may result in build error #66

Closed meikjaeckle closed 8 months ago

meikjaeckle commented 8 months ago

LITTLEFS.h is already included in line 34

Results in compile error if LITTLEFS lib is not available.

shining-man commented 8 months ago

You must replace LITTLEFS.h with LittleFS.h

meikjaeckle commented 8 months ago

Thanks, that's a good catch, I'm building on Linux. What I wanted to mention is that the header is once included inside the #ifdef USE_LittleFS and once just afterwards. I think this is just a left over from the last change. Not a big issue.

#ifdef USE_LittleFS
  #define SPIFFS LittleFS
  #include <LITTLEFS.h> 
#else
  #include <SPIFFS.h>
#endif
#include <LITTLEFS.h>
meikjaeckle commented 8 months ago

Fixed with version 0.5.3