ESP8266SdFat and SDFS linked when not used

[TD-er]

Similar to #6691
The PlatformIO inspect tool shows both libraries ESP8266SdFat and SDFS are linked in the binary and thus taking both around 20k of flash size. (debug build, release size may differ)

As far as I can see, all includes for SD.h in my code are in #ifdef wrappers and I cannot see where the file SD.h is included in the code of esp8266/Arduino.

Right now, I am using https://github.com/Jason2866/platform-espressif8266.git#core_2_6_0 as platform (has the latest build toolchain included).

Does any of you have an idea why it is linked?

[Jason2866]

Not linked in Tasmota using https://github.com/Jason2866/platform-espressif8266.git#core_2_6_0 as platform

[TD-er]

@Jason2866 You are not using SPIFFS in Tasmota, right?

[Jason2866]

Ahhh, sorry overlooked (read not exactly...)
It is NOT the SPIFFS / LittleFs stuff. It is about SDFS and ESP8266SdFat

[TD-er]

It is NOT the SPIFFS / LittleFs stuff. It is about SDFS and ESP8266SdFat

Yep, but I wanted to check if there may be a link somewhere.
Like I said, I did check the source of this repo also and could not find an include of SD.h

[d-a-v]

SDFS.h replaces SD.h

[TD-er]

SDFS.h replaces SD.h

Do you have any idea what these include and if it is needed when you don't use the SD functionality?

[d-a-v]

I find none.
Can you copy-paste the linker line, and run it without SDFS library/object files, in the hope the linker will tell where the symbols are missing ?

[TD-er]

I just renamed the files SD.h and SDFS.h so they would not be compiled and then the build completes just fine but without the libraries.

Clean build with the original file names:

Clean build with the 2 .h files renamed:

So I wonder, what could be wrong here? (maybe this is a PlatformIO bug? Linker bug?)

The binaries are actually smaller with these files renamed. (roughly 17k I think)

[devyte]

PlatformIO bug

Maybe retest building with Arduino IDE?

[TD-er]

That's easier said than done.
The only way I know I could check then is by comparing sizes.
And is the build toolchain the same?

[Jason2866]

@TD-er Build chain for platformIO used (with my setup for core 2.6.0) is
https://github.com/earlephilhower/esp-quick-toolchain/releases/tag/2.5.0-4

[TD-er]

Just as a test, I added those libraries to the platformIO lib_ignore list, which does help in no longer including it.

lib_ignore                = SD(esp8266), SDFS

[Jason2866]

In Tasmota the behaviour you have does not occur

[Misiu]

@TD-er adding those libraries to lib_ignore is more of a workaround. Maybe this is a bug in PlatformIO or build chain?