memfault_platform_coredump_save_begin is documented as being weak no-op, but is not in ESP-IDF

[finger563]

Per the documentation and within the base sdk memfault_platform_coredump_save_begin() is listed as a weakly defined symbol that can be overridden by user code.

However, the esp-idf platform port overrides this here.

[noahp]

Hi @finger563 , you're correct! We'll update the SDK to make that change. Do you have a timeline for when you'd need an SDK release with the fix?

[finger563]

We have some platform integration deliverable timelines which are due by Nov 11, though of course the earlier the better for completing the integration and testing.

[noahp]

Thanks @finger563 ! We'll schedule an SDK release for this week, I'll post back here when it's shipped.

[gminn]

@finger563 This has been addressed in v1.16.0, the release that went out today. Please review the Changelog here for details, and let us know if you have any other questions!