GitHub PullRequestEvent at 2014-11-12 15:29:50
Repository: contiki
Actor: simonduq
Payload: {"action":"opened","number":864,"pull_request":{"url":"https://api.github.com/repos/contiki-os/contiki/pulls/864","id":24300072,"html_url":"https://github.com/contiki-os/contiki/pull/864","diff_url":"https://github.com/contiki-os/contiki/pull/864.diff","patch_url":"https://github.com/contiki-os/contiki/pull/864.patch","issue_url":"https://api.github.com/repos/contiki-os/contiki/issues/864","number":864,"state":"open","locked":false,"title":"Cleanup network config","user":{"login":"simonduq","id":1425999,"avatar_url":"https://avatars.githubusercontent.com/u/1425999?v=3","gravatar_id":"","url":"https://api.github.com/users/simonduq","html_url":"https://github.com/simonduq","followers_url":"https://api.github.com/users/simonduq/followers","following_url":"https://api.github.com/users/simonduq/following{/other_user}","gists_url":"https://api.github.com/users/simonduq/gists{/gist_id}","starred_url":"https://api.github.com/users/simonduq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/simonduq/subscriptions","organizations_url":"https://api.github.com/users/simonduq/orgs","repos_url":"https://api.github.com/users/simonduq/repos","events_url":"https://api.github.com/users/simonduq/events{/privacy}","received_events_url":"https://api.github.com/users/simonduq/received_events","type":"User","site_admin":false},"body":"This pull requests proposes a general cleanup of the network layer configuration:\r\n* Homogenize config parameters: now using `CONTIKI_WITH_IPV6`, `CONTIKI_WITH_IPV4`, `CONTIKI_WITH_RIME` in makefiles, and `UIP_CONF_IPV6`, `UIP_CONF_IPV4` and `UIP_CONF_RIME` in c code. No more `UIP_CONF_*` in makefiles. No more `WITH_UIP` nor `WITH_UIP6` anywhere. IPv6 the new default.\r\n* Modularize compilation of the network layer: the modules `net/ip`, `net/ipv6`, `net/ipv4`, `net/rime` are now included in the compilation process only if used.\r\n* Cleanup IPv6 and RPL code: the c files in these modules don't need to have their entire content be conditionally compiled with `UIP_CONF_IPV6` anymore.\r\n\r\nAs a bonus, one can for instance disable packetbuf attributes and addresses depending on the current configuration (e.g. when IPv6 is used, Rime attributes can now be elided as rime won't be compiled at all). This commit https://github.com/simonduq/contiki/commit/f54e13099dfead536c99bcdda426f8eec5d8553c (not part of this PR, but will come next if this gets merged) saves 32 bytes of RAM per queuebuf for non-Rime systems.","created_at":"2014-11-12T15:29:49Z","updated_at":"2014-11-12T15:29:49Z","closed_at":null,"merged_at":null,"merge_commit_sha":null,"assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/contiki-os/contiki/pulls/864/commits","review_comments_url":"https://api.github.com/repos/contiki-os/contiki/pulls/864/comments","review_comment_url":"https://api.github.com/repos/contiki-os/contiki/pulls/comments/{number}","comments_url":"https://api.github.com/repos/contiki-os/contiki/issues/864/comments","statuses_url":"https://api.github.com/repos/contiki-os/contiki/statuses/440fb5b33e8c47d93ab78a6e66277d383af972ed","head":{"label":"simonduq:cleanup-network-config","ref":"cleanup-network-config","sha":"440fb5b33e8c47d93ab78a6e66277d383af972ed","user":{"login":"simonduq","id":1425999,"avatar_url":"https://avatars.githubusercontent.com/u/1425999?v=3","gravatar_id":"","url":"https://api.github.com/users/simonduq","html_url":"https://github.com/simonduq","followers_url":"https://api.github.com/users/simonduq/followers","following_url":"https://api.github.com/users/simonduq/following{/other_user}","gists_url":"https://api.github.com/users/simonduq/gists{/gist_id}","starred_url":"https://api.github.com/users/simonduq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/simonduq/subscriptions","organizations_url":"https://api.github.com/users/simonduq/orgs","repos_url":"https://api.github.com/users/simonduq/repos","events_url":"https://api.github.com/users/simonduq/events{/privacy}","received_events_url":"https://api.github.com/users/simonduq/received_events","type":"User","site_admin":false},"repo":{"id":6440493,"name":"contiki","full_name":"simonduq/contiki","owner":{"login":"simonduq","id":1425999,"avatar_url":"https://avatars.githubusercontent.com/u/1425999?v=3","gravatar_id":"","url":"https://api.github.com/users/simonduq","html_url":"https://github.com/simonduq","followers_url":"https://api.github.com/users/simonduq/followers","following_url":"https://api.github.com/users/simonduq/following{/other_user}","gists_url":"https://api.github.com/users/simonduq/gists{/gist_id}","starred_url":"https://api.github.com/users/simonduq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/simonduq/subscriptions","organizations_url":"https://api.github.com/users/simonduq/orgs","repos_url":"https://api.github.com/users/simonduq/repos","events_url":"https://api.github.com/users/simonduq/events{/privacy}","received_events_url":"https://api.github.com/users/simonduq/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/simonduq/contiki","description":"The official git repository for Contiki, the open source OS for the Internet of Things","fork":true,"url":"https://api.github.com/repos/simonduq/contiki","forks_url":"https://api.github.com/repos/simonduq/contiki/forks","keys_url":"https://api.github.com/repos/simonduq/contiki/keys{/key_id}","collaborators_url":"https://api.github.com/repos/simonduq/contiki/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/simonduq/contiki/teams","hooks_url":"https://api.github.com/repos/simonduq/contiki/hooks","issue_events_url":"https://api.github.com/repos/simonduq/contiki/issues/events{/number}","events_url":"https://api.github.com/repos/simonduq/contiki/events","assignees_url":"https://api.github.com/repos/simonduq/contiki/assignees{/user}","branches_url":"https://api.github.com/repos/simonduq/contiki/branches{/branch}","tags_url":"https://api.github.com/repos/simonduq/contiki/tags","blobs_url":"https://api.github.com/repos/simonduq/contiki/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/simonduq/contiki/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/simonduq/contiki/git/refs{/sha}","trees_url":"https://api.github.com/repos/simonduq/contiki/git/trees{/sha}","statuses_url":"https://api.github.com/repos/simonduq/contiki/statuses/{sha}","languages_url":"https://api.github.com/repos/simonduq/contiki/languages","stargazers_url":"https://api.github.com/repos/simonduq/contiki/stargazers","contributors_url":"https://api.github.com/repos/simonduq/contiki/contributors","subscribers_url":"https://api.github.com/repos/simonduq/contiki/subscribers","subscription_url":"https://api.github.com/repos/simonduq/contiki/subscription","commits_url":"https://api.github.com/repos/simonduq/contiki/commits{/sha}","git_commits_url":"https://api.github.com/repos/simonduq/contiki/git/commits{/sha}","comments_url":"https://api.github.com/repos/simonduq/contiki/comments{/number}","issue_comment_url":"https://api.github.com/repos/simonduq/contiki/issues/comments/{number}","contents_url":"https://api.github.com/repos/simonduq/contiki/contents/{+path}","compare_url":"https://api.github.com/repos/simonduq/contiki/compare/{base}...{head}","merges_url":"https://api.github.com/repos/simonduq/contiki/merges","archive_url":"https://api.github.com/repos/simonduq/contiki/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/simonduq/contiki/downloads","issues_url":"https://api.github.com/repos/simonduq/contiki/issues{/number}","pulls_url":"https://api.github.com/repos/simonduq/contiki/pulls{/number}","milestones_url":"https://api.github.com/repos/simonduq/contiki/milestones{/number}","notifications_url":"https://api.github.com/repos/simonduq/contiki/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/simonduq/contiki/labels{/name}","releases_url":"https://api.github.com/repos/simonduq/contiki/releases{/id}","created_at":"2012-10-29T12:55:12Z","updated_at":"2014-09-05T16:13:16Z","pushed_at":"2014-11-12T15:12:14Z","git_url":"git://github.com/simonduq/contiki.git","ssh_url":"git@github.com:simonduq/contiki.git","clone_url":"https://github.com/simonduq/contiki.git","svn_url":"https://github.com/simonduq/contiki","homepage":"http://www.contiki-os.org/","size":62291,"stargazers_count":1,"watchers_count":1,"language":"C","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":1,"forks":0,"open_issues":1,"watchers":1,"default_branch":"master"}},"base":{"label":"contiki-os:master","ref":"master","sha":"a488ac16cc40530a36315386b3fb5ded4e0f760d","user":{"login":"contiki-os","id":2511296,"avatar_url":"https://avatars.githubusercontent.com/u/2511296?v=3","gravatar_id":"","url":"https://api.github.com/users/contiki-os","html_url":"https://github.com/contiki-os","followers_url":"https://api.github.com/users/contiki-os/followers","following_url":"https://api.github.com/users/contiki-os/following{/other_user}","gists_url":"https://api.github.com/users/contiki-os/gists{/gist_id}","starred_url":"https://api.github.com/users/contiki-os/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/contiki-os/subscriptions","organizations_url":"https://api.github.com/users/contiki-os/orgs","repos_url":"https://api.github.com/users/contiki-os/repos","events_url":"https://api.github.com/users/contiki-os/events{/privacy}","received_events_url":"https://api.github.com/users/contiki-os/received_events","type":"Organization","site_admin":false},"repo":{"id":6365284,"name":"contiki","full_name":"contiki-os/contiki","owner":{"login":"contiki-os","id":2511296,"avatar_url":"https://avatars.githubusercontent.com/u/2511296?v=3","gravatar_id":"","url":"https://api.github.com/users/contiki-os","html_url":"https://github.com/contiki-os","followers_url":"https://api.github.com/users/contiki-os/followers","following_url":"https://api.github.com/users/contiki-os/following{/other_user}","gists_url":"https://api.github.com/users/contiki-os/gists{/gist_id}","starred_url":"https://api.github.com/users/contiki-os/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/contiki-os/subscriptions","organizations_url":"https://api.github.com/users/contiki-os/orgs","repos_url":"https://api.github.com/users/contiki-os/repos","events_url":"https://api.github.com/users/contiki-os/events{/privacy}","received_events_url":"https://api.github.com/users/contiki-os/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/contiki-os/contiki","description":"The official git repository for Contiki, the open source OS for the Internet of Things","fork":false,"url":"https://api.github.com/repos/contiki-os/contiki","forks_url":"https://api.github.com/repos/contiki-os/contiki/forks","keys_url":"https://api.github.com/repos/contiki-os/contiki/keys{/key_id}","collaborators_url":"https://api.github.com/repos/contiki-os/contiki/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/contiki-os/contiki/teams","hooks_url":"https://api.github.com/repos/contiki-os/contiki/hooks","issue_events_url":"https://api.github.com/repos/contiki-os/contiki/issues/events{/number}","events_url":"https://api.github.com/repos/contiki-os/contiki/events","assignees_url":"https://api.github.com/repos/contiki-os/contiki/assignees{/user}","branches_url":"https://api.github.com/repos/contiki-os/contiki/branches{/branch}","tags_url":"https://api.github.com/repos/contiki-os/contiki/tags","blobs_url":"https://api.github.com/repos/contiki-os/contiki/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/contiki-os/contiki/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/contiki-os/contiki/git/refs{/sha}","trees_url":"https://api.github.com/repos/contiki-os/contiki/git/trees{/sha}","statuses_url":"https://api.github.com/repos/contiki-os/contiki/statuses/{sha}","languages_url":"https://api.github.com/repos/contiki-os/contiki/languages","stargazers_url":"https://api.github.com/repos/contiki-os/contiki/stargazers","contributors_url":"https://api.github.com/repos/contiki-os/contiki/contributors","subscribers_url":"https://api.github.com/repos/contiki-os/contiki/subscribers","subscription_url":"https://api.github.com/repos/contiki-os/contiki/subscription","commits_url":"https://api.github.com/repos/contiki-os/contiki/commits{/sha}","git_commits_url":"https://api.github.com/repos/contiki-os/contiki/git/commits{/sha}","comments_url":"https://api.github.com/repos/contiki-os/contiki/comments{/number}","issue_comment_url":"https://api.github.com/repos/contiki-os/contiki/issues/comments/{number}","contents_url":"https://api.github.com/repos/contiki-os/contiki/contents/{+path}","compare_url":"https://api.github.com/repos/contiki-os/contiki/compare/{base}...{head}","merges_url":"https://api.github.com/repos/contiki-os/contiki/merges","archive_url":"https://api.github.com/repos/contiki-os/contiki/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/contiki-os/contiki/downloads","issues_url":"https://api.github.com/repos/contiki-os/contiki/issues{/number}","pulls_url":"https://api.github.com/repos/contiki-os/contiki/pulls{/number}","milestones_url":"https://api.github.com/repos/contiki-os/contiki/milestones{/number}","notifications_url":"https://api.github.com/repos/contiki-os/contiki/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/contiki-os/contiki/labels{/name}","releases_url":"https://api.github.com/repos/contiki-os/contiki/releases{/id}","created_at":"2012-10-24T05:59:36Z","updated_at":"2014-11-12T14:06:13Z","pushed_at":"2014-11-11T13:01:33Z","git_url":"git://github.com/contiki-os/contiki.git","ssh_url":"git@github.com:contiki-os/contiki.git","clone_url":"https://github.com/contiki-os/contiki.git","svn_url":"https://github.com/contiki-os/contiki","homepage":"http://www.contiki-os.org/","size":87065,"stargazers_count":1149,"watchers_count":1149,"language":"C","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":755,"mirror_url":null,"open_issues_count":170,"forks":755,"open_issues":170,"watchers":1149,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/contiki-os/contiki/pulls/864"},"html":{"href":"https://github.com/contiki-os/contiki/pull/864"},"issue":{"href":"https://api.github.com/repos/contiki-os/contiki/issues/864"},"comments":{"href":"https://api.github.com/repos/contiki-os/contiki/issues/864/comments"},"review_comments":{"href":"https://api.github.com/repos/contiki-os/contiki/pulls/864/comments"},"review_comment":{"href":"https://api.github.com/repos/contiki-os/contiki/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/contiki-os/contiki/pulls/864/commits"},"statuses":{"href":"https://api.github.com/repos/contiki-os/contiki/statuses/440fb5b33e8c47d93ab78a6e66277d383af972ed"}},"merged":false,"mergeable":null,"mergeable_state":"unknown","merged_by":null,"comments":0,"review_comments":0,"commits":3,"additions":473,"deletions":493,"changed_files":176}}