{"action":"created","issue":{"url":"https://api.github.com/repos/cpp-netlib/cpp-netlib/issues/549","labels_url":"https://api.github.com/repos/cpp-netlib/cpp-netlib/issues/549/labels{/name}","comments_url":"https://api.github.com/repos/cpp-netlib/cpp-netlib/issues/549/comments","events_url":"https://api.github.com/repos/cpp-netlib/cpp-netlib/issues/549/events","html_url":"https://github.com/cpp-netlib/cpp-netlib/pull/549","id":99540051,"number":549,"title":"Boost include directories are now tied to targets directly","user":{"login":"rcdailey","id":1768054,"avatar_url":"https://avatars.githubusercontent.com/u/1768054?v=3","gravatar_id":"","url":"https://api.github.com/users/rcdailey","html_url":"https://github.com/rcdailey","followers_url":"https://api.github.com/users/rcdailey/followers","following_url":"https://api.github.com/users/rcdailey/following{/other_user}","gists_url":"https://api.github.com/users/rcdailey/gists{/gist_id}","starred_url":"https://api.github.com/users/rcdailey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rcdailey/subscriptions","organizations_url":"https://api.github.com/users/rcdailey/orgs","repos_url":"https://api.github.com/users/rcdailey/repos","events_url":"https://api.github.com/users/rcdailey/events{/privacy}","received_events_url":"https://api.github.com/users/rcdailey/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-08-06T22:43:45Z","updated_at":"2015-08-19T08:14:30Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cpp-netlib/cpp-netlib/pulls/549","html_url":"https://github.com/cpp-netlib/cpp-netlib/pull/549","diff_url":"https://github.com/cpp-netlib/cpp-netlib/pull/549.diff","patch_url":"https://github.com/cpp-netlib/cpp-netlib/pull/549.patch"},"body":"Boost is required for cppnetlib & related projects. Thus, it should transitively provided include directories to its own source as well as the include directories to its dependencies (Boost in this case).\r\n\r\nThat way if there are other CMake targets that depend on cppnetlib but not Boost directly, it will still be able to compile cppnetlib headers that include boost headers.\r\n\r\nThe include directories are tied to respective targets via the new target_include_directories() CMake command. This requires a newer version of CMake (v3.0 minimum)."},"comment":{"url":"https://api.github.com/repos/cpp-netlib/cpp-netlib/issues/comments/132486380","html_url":"https://github.com/cpp-netlib/cpp-netlib/pull/549#issuecomment-132486380","issue_url":"https://api.github.com/repos/cpp-netlib/cpp-netlib/issues/549","id":132486380,"user":{"login":"deanberris","id":839088,"avatar_url":"https://avatars.githubusercontent.com/u/839088?v=3","gravatar_id":"","url":"https://api.github.com/users/deanberris","html_url":"https://github.com/deanberris","followers_url":"https://api.github.com/users/deanberris/followers","following_url":"https://api.github.com/users/deanberris/following{/other_user}","gists_url":"https://api.github.com/users/deanberris/gists{/gist_id}","starred_url":"https://api.github.com/users/deanberris/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/deanberris/subscriptions","organizations_url":"https://api.github.com/users/deanberris/orgs","repos_url":"https://api.github.com/users/deanberris/repos","events_url":"https://api.github.com/users/deanberris/events{/privacy}","received_events_url":"https://api.github.com/users/deanberris/received_events","type":"User","site_admin":false},"created_at":"2015-08-19T08:14:29Z","updated_at":"2015-08-19T08:14:29Z","body":"I actually don't know much about installing cmake this way. :frowning: I would like to support this change, but unfortunately I think maybe CMake 3.0 is still too new? The risk here is that if I merge this change in, Travis CI would just be totally useless."}}