{"action":"created","issue":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/3199","repository_url":"https://api.github.com/repos/RobotLocomotion/drake","labels_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/3199/labels{/name}","comments_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/3199/comments","events_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/3199/events","html_url":"https://github.com/RobotLocomotion/drake/issues/3199","id":172180629,"number":3199,"title":"cpplint_wrapper_test fails on Windows","user":{"login":"sherm1","id":4088016,"avatar_url":"https://avatars.githubusercontent.com/u/4088016?v=3","gravatar_id":"","url":"https://api.github.com/users/sherm1","html_url":"https://github.com/sherm1","followers_url":"https://api.github.com/users/sherm1/followers","following_url":"https://api.github.com/users/sherm1/following{/other_user}","gists_url":"https://api.github.com/users/sherm1/gists{/gist_id}","starred_url":"https://api.github.com/users/sherm1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sherm1/subscriptions","organizations_url":"https://api.github.com/users/sherm1/orgs","repos_url":"https://api.github.com/users/sherm1/repos","events_url":"https://api.github.com/users/sherm1/events{/privacy}","received_events_url":"https://api.github.com/users/sherm1/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RobotLocomotion/drake/labels/configuration:%20windows","name":"configuration: windows","color":"fbca04"},{"url":"https://api.github.com/repos/RobotLocomotion/drake/labels/priority:%20low","name":"priority: low","color":"e99695"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":10,"created_at":"2016-08-19T17:28:42Z","updated_at":"2016-08-19T22:04:19Z","closed_at":null,"body":"Starting with a fresh clone of drake I did a virgin Debug build on Windows (64 bit). Superbuild was clean in Visual Studio. I switched to build/drake/drake.sln; build there was clean (and fast since everything was already built). I ran the test suite and everything worked except `cpplint_wrapper_test`. \r\n\r\nI do have cpplint in my path and it appears that the test ran but failed. I didn't see it in the list of tests or in the bin directory so I'm not sure how to get more information about the failure.\r\n\r\nSome not-very-helpful output:\r\n```\r\n1>        Start 14: cpplint_wrapper_test\r\n1>  14/87 Test #14: cpplint_wrapper_test .....................***Failed    4.02 sec\r\nThe following tests FAILED:\r\n1>  \t 14 - cpplint_wrapper_test (Failed)\r\n1>  Errors while running CTest\r\n1>C:\\Program Files (x86)\\MSBuild\\Microsoft.Cpp\\v4.0\\V140\\Microsoft.CppCommon.targets(133,5):\r\nerror MSB3073: The command \"setlocal ... VCEnd\" exited with code 8.\r\n```"},"comment":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/comments/241147066","html_url":"https://github.com/RobotLocomotion/drake/issues/3199#issuecomment-241147066","issue_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/3199","id":241147066,"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},"created_at":"2016-08-19T22:04:19Z","updated_at":"2016-08-19T22:04:19Z","body":"I just mean that the line separators emitted by cpplint when printing warnings are using `\\r\\n`, but my unit test (where cpplint itself is the DUT) doesn't allow for `\\r` at all.  If you want to edit the `cpplint_wrapper_test.py` to admit an optional `\\r` and PR that, I will review it for you."}}
{"actor":{"display_login":"jwnimmer-tri"}}