[riot-notifications] [RIOT-OS/RIOT] buildsystem/CI: Use of `BOARD_INSUFFCIENT_MEMORY` becomes a maintainace burdon (#11128)
notifications at github.com
Fri Mar 8 20:41:43 CET 2019
> A change could increase code size so much that a couple of low end boards suddenly don't fit anymore.
The current approach is hardly solving this issue. Most tests and examples are already blacklisted for the lower end hardware anyway. And e.g. if a board still has 512B RAM free and one PR adds 511B more RAM requirements, this PR will likely get merged without anyone noticing. The next PR that adds 2B more and will however get all the blame.
To actually get feedback on which PRs do bloat RIOT, it would be better to provide statistics at the end of Murdock. E.g. did the number of link failures increase/decrease compared to the current master HEAD? What is the impact on the size `.bss`, `.data` and `.text` in average, what is the standard derivation and what are the outliers?
I'm aware this is something that will not happen over night. But considering the man power invested to keep `BOARDS_INSUFFICIENT_MEMORY` up to date, the time to develop this seems to be well spent.
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications