{"action":"created","issue":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/8062","repository_url":"https://api.github.com/repos/RIOT-OS/RIOT","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/8062/labels{/name}","comments_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/8062/comments","events_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/8062/events","html_url":"https://github.com/RIOT-OS/RIOT/pull/8062","id":274559303,"number":8062,"title":"make: add board grouping","user":{"login":"kaspar030","id":4679640,"avatar_url":"https://avatars0.githubusercontent.com/u/4679640?v=4","gravatar_id":"","url":"https://api.github.com/users/kaspar030","html_url":"https://github.com/kaspar030","followers_url":"https://api.github.com/users/kaspar030/followers","following_url":"https://api.github.com/users/kaspar030/following{/other_user}","gists_url":"https://api.github.com/users/kaspar030/gists{/gist_id}","starred_url":"https://api.github.com/users/kaspar030/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kaspar030/subscriptions","organizations_url":"https://api.github.com/users/kaspar030/orgs","repos_url":"https://api.github.com/users/kaspar030/repos","events_url":"https://api.github.com/users/kaspar030/events{/privacy}","received_events_url":"https://api.github.com/users/kaspar030/received_events","type":"User","site_admin":false},"labels":[{"id":433669909,"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/contested","name":"contested","color":"d93f0b","default":false},{"id":23399100,"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/enhancement","name":"enhancement","color":"f17070","default":true},{"id":97338598,"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/make","name":"make","color":"fad8c7","default":false},{"id":214816571,"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Ready%20for%20CI%20build","name":"Ready for CI build","color":"bfdadc","default":false},{"id":100135107,"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Request%20for%20Comments","name":"Request for Comments","color":"23F823","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2017-11-16T15:35:35Z","updated_at":"2017-11-17T13:45:47Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/pulls/8062","html_url":"https://github.com/RIOT-OS/RIOT/pull/8062","diff_url":"https://github.com/RIOT-OS/RIOT/pull/8062.diff","patch_url":"https://github.com/RIOT-OS/RIOT/pull/8062.patch"},"body":"This PR adds infrastructure to group boards (e.g., by architecture, by bit-width, ...).\r\n\r\nAnother commit shows how this can be used within application makefiles for black/whitelisting."},"comment":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/comments/345247648","html_url":"https://github.com/RIOT-OS/RIOT/pull/8062#issuecomment-345247648","issue_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/8062","id":345247648,"user":{"login":"kaspar030","id":4679640,"avatar_url":"https://avatars0.githubusercontent.com/u/4679640?v=4","gravatar_id":"","url":"https://api.github.com/users/kaspar030","html_url":"https://github.com/kaspar030","followers_url":"https://api.github.com/users/kaspar030/followers","following_url":"https://api.github.com/users/kaspar030/following{/other_user}","gists_url":"https://api.github.com/users/kaspar030/gists{/gist_id}","starred_url":"https://api.github.com/users/kaspar030/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kaspar030/subscriptions","organizations_url":"https://api.github.com/users/kaspar030/orgs","repos_url":"https://api.github.com/users/kaspar030/repos","events_url":"https://api.github.com/users/kaspar030/events{/privacy}","received_events_url":"https://api.github.com/users/kaspar030/received_events","type":"User","site_admin":false},"created_at":"2017-11-17T13:45:47Z","updated_at":"2017-11-17T13:45:47Z","author_association":"CONTRIBUTOR","body":"> I would vote strictly against using the FEATURES_PROVIDED var for this in any case, is it does have a different semantic (has vs is)\r\n\r\n+1\r\n\r\n> I would prefer renaming the base var to BOARD_GROUP, as the semantics is around boards and not MCUs...\r\n\r\n+1\r\n\r\n> IMHO the run-time check for completeness in not needed, but should be moved to a static test. This way we can ensure, that the group file stays always consistent in master (and we might even want elaborate checks...\r\n\r\n+1\r\n\r\n> On top of the static checks, we could also add a script that can auto-create the groups file based on some predefined rules...\r\n\r\nSounds like an idea. Then again, we don't add/remove boards that often. If any PR that does that fails CI tests if a board is not added/removed, I think we should be good.\r\n"}}
{"actor":{"display_login":"kaspar030"}}