andrzej-kaczmarek commented on pull request #2686: URL: https://github.com/apache/mynewt-core/pull/2686#issuecomment-929466161
@vrahane I said explicitly that I do not like idea of having that package in **Mynewt tree**. I do not have any issue with adding APIs to support it, thus proposed callback, and I do not want to restrict anyone from using Memfault or any other tool with Mynewt, not even sure how you came to such a ridiculous conclusion. Once we add `os_coredump_cb`, adding required support to Memfault repository is trivial and imo it's the best way to do this: we **do not need to maintain** it and we have **proper** support that we can advertise e.g. on webpage. Otherwise, as proposed here, we have a package that requires a 3rd party repo that includes some fork of actual memfault-sdk repo... good luck with maintaining that. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@mynewt.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
