{"action":"created","issue":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4913","repository_url":"https://api.github.com/repos/RIOT-OS/RIOT","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4913/labels{/name}","comments_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4913/comments","events_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4913/events","html_url":"https://github.com/RIOT-OS/RIOT/pull/4913","id":136977543,"number":4913,"title":"make: Use makefile wildcard function instead of shell ls","user":{"login":"gebart","id":317506,"avatar_url":"https://avatars.githubusercontent.com/u/317506?v=3","gravatar_id":"","url":"https://api.github.com/users/gebart","html_url":"https://github.com/gebart","followers_url":"https://api.github.com/users/gebart/followers","following_url":"https://api.github.com/users/gebart/following{/other_user}","gists_url":"https://api.github.com/users/gebart/gists{/gist_id}","starred_url":"https://api.github.com/users/gebart/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gebart/subscriptions","organizations_url":"https://api.github.com/users/gebart/orgs","repos_url":"https://api.github.com/users/gebart/repos","events_url":"https://api.github.com/users/gebart/events{/privacy}","received_events_url":"https://api.github.com/users/gebart/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/cleanup","name":"cleanup","color":"FF7c7c"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/make","name":"make","color":"bfdadc"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/osx","name":"osx","color":"207de5"}],"state":"open","locked":false,"assignee":{"login":"thomaseichinger","id":56618,"avatar_url":"https://avatars.githubusercontent.com/u/56618?v=3","gravatar_id":"","url":"https://api.github.com/users/thomaseichinger","html_url":"https://github.com/thomaseichinger","followers_url":"https://api.github.com/users/thomaseichinger/followers","following_url":"https://api.github.com/users/thomaseichinger/following{/other_user}","gists_url":"https://api.github.com/users/thomaseichinger/gists{/gist_id}","starred_url":"https://api.github.com/users/thomaseichinger/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thomaseichinger/subscriptions","organizations_url":"https://api.github.com/users/thomaseichinger/orgs","repos_url":"https://api.github.com/users/thomaseichinger/repos","events_url":"https://api.github.com/users/thomaseichinger/events{/privacy}","received_events_url":"https://api.github.com/users/thomaseichinger/received_events","type":"User","site_admin":false},"milestone":null,"comments":1,"created_at":"2016-02-27T22:41:37Z","updated_at":"2016-03-01T11:44:24Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/pulls/4913","html_url":"https://github.com/RIOT-OS/RIOT/pull/4913","diff_url":"https://github.com/RIOT-OS/RIOT/pull/4913.diff","patch_url":"https://github.com/RIOT-OS/RIOT/pull/4913.patch"},"body":"I had forgot to PR this branch before..\r\nAlternative to #4424 "},"comment":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/comments/190684647","html_url":"https://github.com/RIOT-OS/RIOT/pull/4913#issuecomment-190684647","issue_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4913","id":190684647,"user":{"login":"cgundogan","id":739129,"avatar_url":"https://avatars.githubusercontent.com/u/739129?v=3","gravatar_id":"","url":"https://api.github.com/users/cgundogan","html_url":"https://github.com/cgundogan","followers_url":"https://api.github.com/users/cgundogan/followers","following_url":"https://api.github.com/users/cgundogan/following{/other_user}","gists_url":"https://api.github.com/users/cgundogan/gists{/gist_id}","starred_url":"https://api.github.com/users/cgundogan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cgundogan/subscriptions","organizations_url":"https://api.github.com/users/cgundogan/orgs","repos_url":"https://api.github.com/users/cgundogan/repos","events_url":"https://api.github.com/users/cgundogan/events{/privacy}","received_events_url":"https://api.github.com/users/cgundogan/received_events","type":"User","site_admin":false},"created_at":"2016-03-01T11:44:23Z","updated_at":"2016-03-01T11:44:23Z","body":"I am wondering if it's really necessary to put each Makefile change into a single commit? I probably would reduce this to 3 commits: one for native, one containing all boards, and one for desvirt.\r\nThe change to the board Makefiles is straight forward and if it breaks for one it should break for all. Reverting a single commit is simpler than reverting them all, and the commit history doesn't look that bloated. But that's just a matter of taste (:"}}