{"action":"created","issue":{"url":"https://api.github.com/repos/ros-industrial/industrial_ci/issues/21","repository_url":"https://api.github.com/repos/ros-industrial/industrial_ci","labels_url":"https://api.github.com/repos/ros-industrial/industrial_ci/issues/21/labels{/name}","comments_url":"https://api.github.com/repos/ros-industrial/industrial_ci/issues/21/comments","events_url":"https://api.github.com/repos/ros-industrial/industrial_ci/issues/21/events","html_url":"https://github.com/ros-industrial/industrial_ci/issues/21","id":132634603,"number":21,"title":"Adapt catkin_make?","user":{"login":"130s","id":1840401,"avatar_url":"https://avatars.githubusercontent.com/u/1840401?v=3","gravatar_id":"","url":"https://api.github.com/users/130s","html_url":"https://github.com/130s","followers_url":"https://api.github.com/users/130s/followers","following_url":"https://api.github.com/users/130s/following{/other_user}","gists_url":"https://api.github.com/users/130s/gists{/gist_id}","starred_url":"https://api.github.com/users/130s/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/130s/subscriptions","organizations_url":"https://api.github.com/users/130s/orgs","repos_url":"https://api.github.com/users/130s/repos","events_url":"https://api.github.com/users/130s/events{/privacy}","received_events_url":"https://api.github.com/users/130s/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ros-industrial/industrial_ci/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-02-10T08:26:22Z","updated_at":"2016-02-10T10:43:35Z","closed_at":null,"body":"From https://github.com/ros-industrial/industrial_core/pull/136#issuecomment-182246945\r\n\r\n> As @ipa-mdl already commented in ros-industrial/ros_canopen#155, this can / should be reverted once catkin_tools 0.4.0 is released. I'd rather we switch to catkin_make for the time being (don't know how feasible that is though, @130s?).\r\n\r\nIt's absolutely feasible and in fact I thought I offered to do so somewhere (I just can't find it now, but it's not a big deal), since I intend `industrial_ci` to work with multiple build tools (`catkin_make`, `ament` etc.) I don't think I'll be able to work on this this week though."},"comment":{"url":"https://api.github.com/repos/ros-industrial/industrial_ci/issues/comments/182308308","html_url":"https://github.com/ros-industrial/industrial_ci/issues/21#issuecomment-182308308","issue_url":"https://api.github.com/repos/ros-industrial/industrial_ci/issues/21","id":182308308,"user":{"login":"gavanderhoorn","id":4550046,"avatar_url":"https://avatars.githubusercontent.com/u/4550046?v=3","gravatar_id":"","url":"https://api.github.com/users/gavanderhoorn","html_url":"https://github.com/gavanderhoorn","followers_url":"https://api.github.com/users/gavanderhoorn/followers","following_url":"https://api.github.com/users/gavanderhoorn/following{/other_user}","gists_url":"https://api.github.com/users/gavanderhoorn/gists{/gist_id}","starred_url":"https://api.github.com/users/gavanderhoorn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gavanderhoorn/subscriptions","organizations_url":"https://api.github.com/users/gavanderhoorn/orgs","repos_url":"https://api.github.com/users/gavanderhoorn/repos","events_url":"https://api.github.com/users/gavanderhoorn/events{/privacy}","received_events_url":"https://api.github.com/users/gavanderhoorn/received_events","type":"User","site_admin":false},"created_at":"2016-02-10T10:43:35Z","updated_at":"2016-02-10T10:43:35Z","body":"@ipa-mdl wrote:\r\n> I am not sure if `catkin_make` is the right way to go, since it will not find bugs in dependencies etc. that might prevent release. `catkin_make_isolated --merge` might be a better option.\r\n\r\nI agree about misconfigured dependencies going unnoticed, but the context for my comment was that we're now changing *our* files to work-around a bug in a build tool not under our control. If I understand @130s correctly, he wanted to add support for additional build tools anyway, so if that could be exploited to add a temporary work-around for this particular issue with `catkin_tools`, I'd find that more palatable than what ros-industrial/industrial_core#136 is proposing.\r\n\r\nObviously, if there is support for additional builders, we could always decide to use `catkin_make_isolated` as well.\r\n\r\n> Another possible solution is to use latest catkin_tools from source in the repositories that need it\r\nThis could be interluded directly in .travis.yml: https://github.com/catkin/catkin_tools/blob/master/.travis.yml#L35\r\n\r\nHm. That would mean that we're going to roll out a CI setup that is tracking a development repository for its main build tool. I'm not sure I'd be so happy about that.\r\n"}}