{"action":"created","issue":{"url":"https://api.github.com/repos/chapel-lang/chapel/issues/1628","labels_url":"https://api.github.com/repos/chapel-lang/chapel/issues/1628/labels{/name}","comments_url":"https://api.github.com/repos/chapel-lang/chapel/issues/1628/comments","events_url":"https://api.github.com/repos/chapel-lang/chapel/issues/1628/events","html_url":"https://github.com/chapel-lang/chapel/pull/1628","id":62430838,"number":1628,"title":"Support valgrind testing on chpldoc tests.","user":{"login":"thomasvandoren","id":530981,"avatar_url":"https://avatars.githubusercontent.com/u/530981?v=3","gravatar_id":"","url":"https://api.github.com/users/thomasvandoren","html_url":"https://github.com/thomasvandoren","followers_url":"https://api.github.com/users/thomasvandoren/followers","following_url":"https://api.github.com/users/thomasvandoren/following{/other_user}","gists_url":"https://api.github.com/users/thomasvandoren/gists{/gist_id}","starred_url":"https://api.github.com/users/thomasvandoren/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thomasvandoren/subscriptions","organizations_url":"https://api.github.com/users/thomasvandoren/orgs","repos_url":"https://api.github.com/users/thomasvandoren/repos","events_url":"https://api.github.com/users/thomasvandoren/events{/privacy}","received_events_url":"https://api.github.com/users/thomasvandoren/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-03-17T15:32:52Z","updated_at":"2015-03-17T15:36:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/chapel-lang/chapel/pulls/1628","html_url":"https://github.com/chapel-lang/chapel/pull/1628","diff_url":"https://github.com/chapel-lang/chapel/pull/1628.diff","patch_url":"https://github.com/chapel-lang/chapel/pull/1628.patch"},"body":"Update sub_test to correctly wrap chpldoc compiler commands with valgrind.\r\nPreviously, the chpldoc tests (i.e. those with .doc.chpl suffix) were using the\r\nchpl compiler when running under valgrind due to an incorrect if/else block.\r\nThis led to invalid options error, because chpl was called with chpldoc options\r\n(e.g. --text-only).\r\n\r\n### Verification:\r\n\r\n* [x] Run `start_test -valgrind test/release/examples/primers/chpldoc.doc.chpl`\r\n  and confirm it passes."},"comment":{"url":"https://api.github.com/repos/chapel-lang/chapel/issues/comments/82419275","html_url":"https://github.com/chapel-lang/chapel/pull/1628#issuecomment-82419275","issue_url":"https://api.github.com/repos/chapel-lang/chapel/issues/1628","id":82419275,"user":{"login":"thomasvandoren","id":530981,"avatar_url":"https://avatars.githubusercontent.com/u/530981?v=3","gravatar_id":"","url":"https://api.github.com/users/thomasvandoren","html_url":"https://github.com/thomasvandoren","followers_url":"https://api.github.com/users/thomasvandoren/followers","following_url":"https://api.github.com/users/thomasvandoren/following{/other_user}","gists_url":"https://api.github.com/users/thomasvandoren/gists{/gist_id}","starred_url":"https://api.github.com/users/thomasvandoren/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thomasvandoren/subscriptions","organizations_url":"https://api.github.com/users/thomasvandoren/orgs","repos_url":"https://api.github.com/users/thomasvandoren/repos","events_url":"https://api.github.com/users/thomasvandoren/events{/privacy}","received_events_url":"https://api.github.com/users/thomasvandoren/received_events","type":"User","site_admin":false},"created_at":"2015-03-17T15:36:50Z","updated_at":"2015-03-17T15:36:50Z","body":"@ronawho -- will you please review?"}}