{"action":"created","issue":{"url":"https://api.github.com/repos/matthiasbeyer/imag/issues/618","repository_url":"https://api.github.com/repos/matthiasbeyer/imag","labels_url":"https://api.github.com/repos/matthiasbeyer/imag/issues/618/labels{/name}","comments_url":"https://api.github.com/repos/matthiasbeyer/imag/issues/618/comments","events_url":"https://api.github.com/repos/matthiasbeyer/imag/issues/618/events","html_url":"https://github.com/matthiasbeyer/imag/issues/618","id":169312481,"number":618,"title":"Improve Makefile","user":{"login":"matthiasbeyer","id":427866,"avatar_url":"https://avatars.githubusercontent.com/u/427866?v=3","gravatar_id":"","url":"https://api.github.com/users/matthiasbeyer","html_url":"https://github.com/matthiasbeyer","followers_url":"https://api.github.com/users/matthiasbeyer/followers","following_url":"https://api.github.com/users/matthiasbeyer/following{/other_user}","gists_url":"https://api.github.com/users/matthiasbeyer/gists{/gist_id}","starred_url":"https://api.github.com/users/matthiasbeyer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/matthiasbeyer/subscriptions","organizations_url":"https://api.github.com/users/matthiasbeyer/orgs","repos_url":"https://api.github.com/users/matthiasbeyer/repos","events_url":"https://api.github.com/users/matthiasbeyer/events{/privacy}","received_events_url":"https://api.github.com/users/matthiasbeyer/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/matthiasbeyer/imag/labels/complexity/medium","name":"complexity/medium","color":"5319e7"},{"url":"https://api.github.com/repos/matthiasbeyer/imag/labels/kind/enhancement","name":"kind/enhancement","color":"eb6420"},{"url":"https://api.github.com/repos/matthiasbeyer/imag/labels/kind/infrastructure","name":"kind/infrastructure","color":"eb6420"},{"url":"https://api.github.com/repos/matthiasbeyer/imag/labels/meta/assigned","name":"meta/assigned","color":"d4c5f9"},{"url":"https://api.github.com/repos/matthiasbeyer/imag/labels/meta/importance/low","name":"meta/importance/low","color":"d4c5f9"},{"url":"https://api.github.com/repos/matthiasbeyer/imag/labels/meta/WIP","name":"meta/WIP","color":"d4c5f9"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2016-08-04T07:50:50Z","updated_at":"2016-08-06T12:31:35Z","closed_at":null,"body":"At the moment, we simply `cargo build` all the things and copy them to `./out`. A better way would be to let the `out/imag-foo` binary depend on the `imag-foo/target/debug/imag-foo` binary and define the job for this one by calling `cargo build`.\r\n\r\nThis way we wouldn't rebuild the world each time when we build all the things but only one binary changed, right?"},"comment":{"url":"https://api.github.com/repos/matthiasbeyer/imag/issues/comments/238021192","html_url":"https://github.com/matthiasbeyer/imag/issues/618#issuecomment-238021192","issue_url":"https://api.github.com/repos/matthiasbeyer/imag/issues/618","id":238021192,"user":{"login":"mario-kr","id":7137799,"avatar_url":"https://avatars.githubusercontent.com/u/7137799?v=3","gravatar_id":"","url":"https://api.github.com/users/mario-kr","html_url":"https://github.com/mario-kr","followers_url":"https://api.github.com/users/mario-kr/followers","following_url":"https://api.github.com/users/mario-kr/following{/other_user}","gists_url":"https://api.github.com/users/mario-kr/gists{/gist_id}","starred_url":"https://api.github.com/users/mario-kr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mario-kr/subscriptions","organizations_url":"https://api.github.com/users/mario-kr/orgs","repos_url":"https://api.github.com/users/mario-kr/repos","events_url":"https://api.github.com/users/mario-kr/events{/privacy}","received_events_url":"https://api.github.com/users/mario-kr/received_events","type":"User","site_admin":false},"created_at":"2016-08-06T12:31:35Z","updated_at":"2016-08-06T12:31:35Z","body":"Ok, I'm gonna make/keep following targets:\r\nall : imag-*\r\nimag-% : ---- cargo build $(toml)\r\ninstall : install-imag-*\r\ninstall-imag-% :  ---- cargo install --force $(toml)\r\ninstall-dev : install-dev-imag-*\r\ninstall-dev-imag-% : imag-% prep ---- cp target/debug/imag-% out/\r\nprep : ---- mkdir out\r\n// remove doc target, is already done and may cause merge conflict\r\nPlease keep in mind that this is not actual code.\r\nI don't know if the dependency of install-dev-imag-% is possible to do that way (I don't think so), so I might need to leave out this target, and do the copying in install-dev directly.\r\nDoes this sound about right?"}}
{"actor":{"display_login":"mario-kr"}}