{"action":"created","issue":{"url":"https://api.github.com/repos/solettaproject/soletta/issues/908","labels_url":"https://api.github.com/repos/solettaproject/soletta/issues/908/labels{/name}","comments_url":"https://api.github.com/repos/solettaproject/soletta/issues/908/comments","events_url":"https://api.github.com/repos/solettaproject/soletta/issues/908/events","html_url":"https://github.com/solettaproject/soletta/pull/908","id":111900037,"number":908,"title":"Builders [v6]","user":{"login":"cmarcelo","id":10478,"avatar_url":"https://avatars.githubusercontent.com/u/10478?v=3","gravatar_id":"","url":"https://api.github.com/users/cmarcelo","html_url":"https://github.com/cmarcelo","followers_url":"https://api.github.com/users/cmarcelo/followers","following_url":"https://api.github.com/users/cmarcelo/following{/other_user}","gists_url":"https://api.github.com/users/cmarcelo/gists{/gist_id}","starred_url":"https://api.github.com/users/cmarcelo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmarcelo/subscriptions","organizations_url":"https://api.github.com/users/cmarcelo/orgs","repos_url":"https://api.github.com/users/cmarcelo/repos","events_url":"https://api.github.com/users/cmarcelo/events{/privacy}","received_events_url":"https://api.github.com/users/cmarcelo/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-10-16T19:56:11Z","updated_at":"2015-10-16T20:21:59Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/solettaproject/soletta/pulls/908","html_url":"https://github.com/solettaproject/soletta/pull/908","diff_url":"https://github.com/solettaproject/soletta/pull/908.diff","patch_url":"https://github.com/solettaproject/soletta/pull/908.patch"},"body":"Differences from v5:\r\n\r\n- Added notes indicating the code here is experimental. The goal of\r\n  it was to figure out the workflow we wanted. If/Once this is\r\n  settled, then worth focusing on hardening it (i.e. how can we\r\n  communicate using crypto, and so on).\r\n\r\n- Cleaned environment variables before calling compile in the\r\n  server. Starting with PATH, HOME and LANG=C. Should be tweaked\r\n  later as necessary.\r\n\r\n- New commit: try to fail early when running as server from wrong\r\n  location.\r\n\r\n- New commit: Improve test-builders.sh to actually use `sol` to\r\n  build things.\r\n\r\n- `sol` returns 1 when the build fails, and don't always print\r\n  the list of available targets (unnecessary).\r\n\r\n- Replace `-connect` options with an `-addr` option, that's used\r\n  both for connecting to and serving.\r\n\r\n- Added explanation of why platform-test builds always fail: they\r\n  are used to show the environment and debug the generic part of\r\n  compile scripts (generate C from FBP, pick the right files, etc.).\r\n\r\n- Added builders/.gitignore.\r\n"},"comment":{"url":"https://api.github.com/repos/solettaproject/soletta/issues/comments/148824097","html_url":"https://github.com/solettaproject/soletta/pull/908#issuecomment-148824097","issue_url":"https://api.github.com/repos/solettaproject/soletta/issues/908","id":148824097,"user":{"login":"lpereira","id":15001,"avatar_url":"https://avatars.githubusercontent.com/u/15001?v=3","gravatar_id":"","url":"https://api.github.com/users/lpereira","html_url":"https://github.com/lpereira","followers_url":"https://api.github.com/users/lpereira/followers","following_url":"https://api.github.com/users/lpereira/following{/other_user}","gists_url":"https://api.github.com/users/lpereira/gists{/gist_id}","starred_url":"https://api.github.com/users/lpereira/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lpereira/subscriptions","organizations_url":"https://api.github.com/users/lpereira/orgs","repos_url":"https://api.github.com/users/lpereira/repos","events_url":"https://api.github.com/users/lpereira/events{/privacy}","received_events_url":"https://api.github.com/users/lpereira/received_events","type":"User","site_admin":false},"created_at":"2015-10-16T20:21:59Z","updated_at":"2015-10-16T20:21:59Z","body":"LGTM since it's clearly marked as experimental, although I haven't tried it."}}