[riot-notifications] [RIOT-OS/RIOT] buildsystem/CI: Use of `BOARD_INSUFFCIENT_MEMORY` becomes a maintainace burdon (#11128)
notifications at github.com
Thu Mar 7 15:50:50 CET 2019
Funny, we had the same discussion over lunch today 😄
My conclusion was that all these lists, i.e., `BOARD_WHITELIST`, `BOARD_BLACKLIST`, `BOARD_INSUFFICIENT_MEMORY`, `TEST_ON_CI_WHITELIST`, and so on, are only used by Murdock-CI and to make that pass without errors by avoid building+testing stuff that will likely fail. And also to safe exec time on the CI.
So ideally (IMHO) we should get rid of such manually maintained lists in RIOT, and e.g. let compiling or linking fail. Because, as you already pointed out, the lists are hard to maintain and once a board is on such a list it likely stays there, even if issues are fixed. Again, these lists mostly serve one purpose: to make Murdock not fail and to me that's not how a CI should run.
I'm in favour of option 1. My 2 cents ...
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