{"action":"created","issue":{"url":"https://api.github.com/repos/denoland/deno/issues/450","repository_url":"https://api.github.com/repos/denoland/deno","labels_url":"https://api.github.com/repos/denoland/deno/issues/450/labels{/name}","comments_url":"https://api.github.com/repos/denoland/deno/issues/450/comments","events_url":"https://api.github.com/repos/denoland/deno/issues/450/events","html_url":"https://github.com/denoland/deno/issues/450","id":347415691,"node_id":"MDU6SXNzdWUzNDc0MTU2OTE=","number":450,"title":"Introduce Makefile into build process","user":{"login":"NewLunarFire","id":10763734,"node_id":"MDQ6VXNlcjEwNzYzNzM0","avatar_url":"https://avatars3.githubusercontent.com/u/10763734?v=4","gravatar_id":"","url":"https://api.github.com/users/NewLunarFire","html_url":"https://github.com/NewLunarFire","followers_url":"https://api.github.com/users/NewLunarFire/followers","following_url":"https://api.github.com/users/NewLunarFire/following{/other_user}","gists_url":"https://api.github.com/users/NewLunarFire/gists{/gist_id}","starred_url":"https://api.github.com/users/NewLunarFire/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NewLunarFire/subscriptions","organizations_url":"https://api.github.com/users/NewLunarFire/orgs","repos_url":"https://api.github.com/users/NewLunarFire/repos","events_url":"https://api.github.com/users/NewLunarFire/events{/privacy}","received_events_url":"https://api.github.com/users/NewLunarFire/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-08-03T14:21:07Z","updated_at":"2018-08-03T16:17:25Z","closed_at":"2018-08-03T16:17:25Z","author_association":"NONE","body":"Hi @ry ,\r\n\r\nAs the project advances, I was wondering if introducing a Makefile in the build process would be a good idea.\r\n\r\nAdvantages:\r\n- It would be simple to rebuild deno and could check for specific file dependencies to know if it should run setup.py, build.py or other scripts that could appear in the future.\r\n- It would also allow to specify different build configurations (debug, release, testing?).\r\n- It a standard and widely used method for this workflow. Building/Re-building becomes as simple as typing make\r\n\r\nDisadvantages:\r\n- It is a really Unix method and never works well on a Windows environment without adding other tools to the mix, like Cygwin\r\n- It adds another step to the build process\r\n\r\nI'm on the fence about this. The build process as is isn't really complicated as of now, so there's no absolute need for this. If it's something you would like to have I will gladly open a PR. It could also be introduced further down the road."},"comment":{"url":"https://api.github.com/repos/denoland/deno/issues/comments/410303501","html_url":"https://github.com/denoland/deno/issues/450#issuecomment-410303501","issue_url":"https://api.github.com/repos/denoland/deno/issues/450","id":410303501,"node_id":"MDEyOklzc3VlQ29tbWVudDQxMDMwMzUwMQ==","user":{"login":"ry","id":80,"node_id":"MDQ6VXNlcjgw","avatar_url":"https://avatars1.githubusercontent.com/u/80?v=4","gravatar_id":"","url":"https://api.github.com/users/ry","html_url":"https://github.com/ry","followers_url":"https://api.github.com/users/ry/followers","following_url":"https://api.github.com/users/ry/following{/other_user}","gists_url":"https://api.github.com/users/ry/gists{/gist_id}","starred_url":"https://api.github.com/users/ry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ry/subscriptions","organizations_url":"https://api.github.com/users/ry/orgs","repos_url":"https://api.github.com/users/ry/repos","events_url":"https://api.github.com/users/ry/events{/privacy}","received_events_url":"https://api.github.com/users/ry/received_events","type":"User","site_admin":false},"created_at":"2018-08-03T16:17:25Z","updated_at":"2018-08-03T16:17:25Z","author_association":"COLLABORATOR","body":"Hi @NewLunarFire - make would be redundant, we already use ninja."}}
{"actor":{"display_login":"ry"}}