{"action":"created","issue":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2961","repository_url":"https://api.github.com/repos/RobotLocomotion/drake","labels_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2961/labels{/name}","comments_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2961/comments","events_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2961/events","html_url":"https://github.com/RobotLocomotion/drake/pull/2961","id":167979685,"number":2961,"title":"Provide bindings/swig iff we are using swig_matlab","user":{"login":"jwnimmer-tri","id":17596505,"avatar_url":"https://avatars.githubusercontent.com/u/17596505?v=3","gravatar_id":"","url":"https://api.github.com/users/jwnimmer-tri","html_url":"https://github.com/jwnimmer-tri","followers_url":"https://api.github.com/users/jwnimmer-tri/followers","following_url":"https://api.github.com/users/jwnimmer-tri/following{/other_user}","gists_url":"https://api.github.com/users/jwnimmer-tri/gists{/gist_id}","starred_url":"https://api.github.com/users/jwnimmer-tri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jwnimmer-tri/subscriptions","organizations_url":"https://api.github.com/users/jwnimmer-tri/orgs","repos_url":"https://api.github.com/users/jwnimmer-tri/repos","events_url":"https://api.github.com/users/jwnimmer-tri/events{/privacy}","received_events_url":"https://api.github.com/users/jwnimmer-tri/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"mwoehlke-kitware","id":6474409,"avatar_url":"https://avatars.githubusercontent.com/u/6474409?v=3","gravatar_id":"","url":"https://api.github.com/users/mwoehlke-kitware","html_url":"https://github.com/mwoehlke-kitware","followers_url":"https://api.github.com/users/mwoehlke-kitware/followers","following_url":"https://api.github.com/users/mwoehlke-kitware/following{/other_user}","gists_url":"https://api.github.com/users/mwoehlke-kitware/gists{/gist_id}","starred_url":"https://api.github.com/users/mwoehlke-kitware/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mwoehlke-kitware/subscriptions","organizations_url":"https://api.github.com/users/mwoehlke-kitware/orgs","repos_url":"https://api.github.com/users/mwoehlke-kitware/repos","events_url":"https://api.github.com/users/mwoehlke-kitware/events{/privacy}","received_events_url":"https://api.github.com/users/mwoehlke-kitware/received_events","type":"User","site_admin":false},"assignees":[{"login":"mwoehlke-kitware","id":6474409,"avatar_url":"https://avatars.githubusercontent.com/u/6474409?v=3","gravatar_id":"","url":"https://api.github.com/users/mwoehlke-kitware","html_url":"https://github.com/mwoehlke-kitware","followers_url":"https://api.github.com/users/mwoehlke-kitware/followers","following_url":"https://api.github.com/users/mwoehlke-kitware/following{/other_user}","gists_url":"https://api.github.com/users/mwoehlke-kitware/gists{/gist_id}","starred_url":"https://api.github.com/users/mwoehlke-kitware/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mwoehlke-kitware/subscriptions","organizations_url":"https://api.github.com/users/mwoehlke-kitware/orgs","repos_url":"https://api.github.com/users/mwoehlke-kitware/repos","events_url":"https://api.github.com/users/mwoehlke-kitware/events{/privacy}","received_events_url":"https://api.github.com/users/mwoehlke-kitware/received_events","type":"User","site_admin":false}],"milestone":null,"comments":11,"created_at":"2016-07-27T23:24:45Z","updated_at":"2016-07-28T19:55:04Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RobotLocomotion/drake/pulls/2961","html_url":"https://github.com/RobotLocomotion/drake/pull/2961","diff_url":"https://github.com/RobotLocomotion/drake/pull/2961.diff","patch_url":"https://github.com/RobotLocomotion/drake/pull/2961.patch"},"body":"Provide `bindings/swig` if and only if we are using `swig_matlab`, though in an admittedly somewhat brittle way.\r\n\r\nFixes #2957.\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/robotlocomotion/drake/2961)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/comments/236006469","html_url":"https://github.com/RobotLocomotion/drake/pull/2961#issuecomment-236006469","issue_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2961","id":236006469,"user":{"login":"mwoehlke-kitware","id":6474409,"avatar_url":"https://avatars.githubusercontent.com/u/6474409?v=3","gravatar_id":"","url":"https://api.github.com/users/mwoehlke-kitware","html_url":"https://github.com/mwoehlke-kitware","followers_url":"https://api.github.com/users/mwoehlke-kitware/followers","following_url":"https://api.github.com/users/mwoehlke-kitware/following{/other_user}","gists_url":"https://api.github.com/users/mwoehlke-kitware/gists{/gist_id}","starred_url":"https://api.github.com/users/mwoehlke-kitware/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mwoehlke-kitware/subscriptions","organizations_url":"https://api.github.com/users/mwoehlke-kitware/orgs","repos_url":"https://api.github.com/users/mwoehlke-kitware/repos","events_url":"https://api.github.com/users/mwoehlke-kitware/events{/privacy}","received_events_url":"https://api.github.com/users/mwoehlke-kitware/received_events","type":"User","site_admin":false},"created_at":"2016-07-28T19:55:04Z","updated_at":"2016-07-28T19:55:04Z","body":"\n\n\n\nReview status: 0 of 2 files reviewed at latest revision, 2 unresolved discussions.\n\n---\n\n*[drake/bindings/CMakeLists.txt, line 5 \\[r2\\]](https://reviewable.io:443/reviews/robotlocomotion/drake/2961#-KNm2E_yt-7E8KrUIb3h:-KNn3DzmjoXgSmm0GxU3:49713427) ([raw file](https://github.com/robotlocomotion/drake/blob/d367048c5737976d8dc428c4c9291380c3402ae6/drake/bindings/CMakeLists.txt#L5)):*\n> Apparently the cache is sticky and preserves old, wrong values? Is there a way to tell CMake to be less unsound?\n\nIn general, the `find_` set of functions *need* to work this way; otherwise the user wouldn't be able to tell CMake where to find something it can't find in the default paths.\n\nIn this case, if you're sure you want to force the One True Value and not ever let the user override it, you could `unset(SWIG_EXECUTABLE CACHE)` before calling `find_program`.\n\n---\n\n\n*Comments from [Reviewable](https://reviewable.io:443/reviews/robotlocomotion/drake/2961)*\n<!-- Sent from Reviewable.io -->\n"}}
{"actor":{"display_login":"mwoehlke-kitware"}}