{"action":"created","issue":{"url":"https://api.github.com/repos/oclint/oclint/issues/398","repository_url":"https://api.github.com/repos/oclint/oclint","labels_url":"https://api.github.com/repos/oclint/oclint/issues/398/labels{/name}","comments_url":"https://api.github.com/repos/oclint/oclint/issues/398/comments","events_url":"https://api.github.com/repos/oclint/oclint/issues/398/events","html_url":"https://github.com/oclint/oclint/issues/398","id":204653243,"number":398,"title":"OCLint does not report compilation failures using PMD Reporter","user":{"login":"Uranium-92","id":1793932,"avatar_url":"https://avatars.githubusercontent.com/u/1793932?v=3","gravatar_id":"","url":"https://api.github.com/users/Uranium-92","html_url":"https://github.com/Uranium-92","followers_url":"https://api.github.com/users/Uranium-92/followers","following_url":"https://api.github.com/users/Uranium-92/following{/other_user}","gists_url":"https://api.github.com/users/Uranium-92/gists{/gist_id}","starred_url":"https://api.github.com/users/Uranium-92/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Uranium-92/subscriptions","organizations_url":"https://api.github.com/users/Uranium-92/orgs","repos_url":"https://api.github.com/users/Uranium-92/repos","events_url":"https://api.github.com/users/Uranium-92/events{/privacy}","received_events_url":"https://api.github.com/users/Uranium-92/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-02-01T17:40:14Z","updated_at":"2017-02-01T17:45:41Z","closed_at":null,"body":"### Issue Summary\r\nIf OCLint fails to compile source file during analysis using PMD Reporter, the resulting analysis report does not have any signs of that, nor analysis is failed.\r\nHowever OCLint reports compilation errors when is run with Plain Text Report.\r\n\r\n### Environment\r\n- Operation system name and version: Mac OS X 10.11.6\r\n- OCLint version: 0.11.1\r\n- How OCLint is installed: binary downloaded from GitHubvim\r\n\r\n### Reproduction Steps\r\nRun analysis using PMD Reporter against source file with compilation error; or simply pass non-existent warning-flag as compilation argument along with `-Werror`.\r\n\r\n#### Sample code\r\n\r\nFile main.cpp:\r\n```\r\n#include <iostream>\r\n\r\nint main(int args, char* argv[])\r\n{\r\n\tstd::cout << \"Hello, World!\" << std::endl;\r\n}\r\n```\r\n\r\nCommand to run OCLint with the code above:\r\n`oclint --report-type pmd -o result.xml main.cpp -- -Wnon-exitent-warning -Werror`\r\n\r\n### Expected Behavior\r\nThe resulting analysis report file result.xml has entries about compilation errors, or analysis is failed with descriptive message printed to stderr.\r\n\r\n### Actual Behavior\r\nThe resulting analysis report file result.xml does not have any signs of compilation errors, nor analysis is failed.\r\n"},"comment":{"url":"https://api.github.com/repos/oclint/oclint/issues/comments/276727613","html_url":"https://github.com/oclint/oclint/issues/398#issuecomment-276727613","issue_url":"https://api.github.com/repos/oclint/oclint/issues/398","id":276727613,"user":{"login":"Uranium-92","id":1793932,"avatar_url":"https://avatars.githubusercontent.com/u/1793932?v=3","gravatar_id":"","url":"https://api.github.com/users/Uranium-92","html_url":"https://github.com/Uranium-92","followers_url":"https://api.github.com/users/Uranium-92/followers","following_url":"https://api.github.com/users/Uranium-92/following{/other_user}","gists_url":"https://api.github.com/users/Uranium-92/gists{/gist_id}","starred_url":"https://api.github.com/users/Uranium-92/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Uranium-92/subscriptions","organizations_url":"https://api.github.com/users/Uranium-92/orgs","repos_url":"https://api.github.com/users/Uranium-92/repos","events_url":"https://api.github.com/users/Uranium-92/events{/privacy}","received_events_url":"https://api.github.com/users/Uranium-92/received_events","type":"User","site_admin":false},"created_at":"2017-02-01T17:45:41Z","updated_at":"2017-02-01T17:45:41Z","body":"It seems that for consistency OCLint must add entries to resulting report file but the whole compilation should not be failed.\r\n@ryuichis, what do you think? Can make pull request if you approve this behavior."}}
{"actor":{"display_login":"Uranium-92"}}