SDK 2.0 - make time() function WEAK + SNTP tweaks

[DarkSide]

Hello,
In one of my projects I'm using DS3231 clock module and I want to make time() function to return time from this module. The problem is that in new SDK time() function already defined in libmain - that causing linker error. So, please make it weak to allow user to provide it's own implementation.
Futhermore, it would be good if you provide a callback for user when SNTP receive time. Currently to check is SNTP time was updated I need to constantly poll sntp_get_current_timestamp() until it return correct time.

[eriksl]

+1

Even better, just make all symbols in library weak, like it should, with a library.

[DarkSide]

Is this forum alive? Will I hear any response from Espressif related person?

[pratik]

Hello,
I will forward this suggestion to the SDK developers. Making internal functions weak causes issues with debugging because libraries are not provided with complete source code. Also, interdependent components inside the SDK get messed up a lot. But will definitely get this through to the developers. Thanks for the feedback!