-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Jenkins 35096 cppcheck plugin pipeline support #36
Conversation
… displayed properly (returned RAW html)
Is there anything that blocks merging the PR? |
@loganek not as far as I know the build should build on java 8. |
@James-Dengel awesome! @mixalturek, would it be possible to merge it? |
Hi, I'm no longer interested in this plugin and mainly I don't have any time for testing of the changes and releasing of a new version. Please become a maintainer and ask for the permissions. |
Hi all, I have been working on the plugin to make it compatible with jenkins pipeline. I think it is finished and has been tested by our company. It seems to be working to our expectations. The plugin has been developed in Java8 environment. Can I do a pull request to test the plugin, since the JENKINS-35096 is still open ? |
@James-Dengel thanks to your work and following the guidelines from https://jenkins.io/doc/developer/plugin-development/pipeline-integration/#basic-update on making plugins pipeline compatible. If you're fine with that I would merge the branch to master. |
I have no issue with that at all @marcosteffan :) |
Any news when this will be available? |
I would like to know when you are planning to release the next version of cppCheck. It contains bug fixes that our team needs. Thank you for your work on this. |
Same question as above: Any plans to release a new version? We'd like to use this feature.. |
Also looking forward to a new version - please cut a release. |
+1 Any news on a new release date. |
We need cppCheck for pipeline and waiting for new version with this fix. Any plan for new release? |
Any plans for releasing this? We need it aswell |
We couldn't wait for it to be released, so I made a custom parser using the Warnings plugin. It works perfectly and you get all of the built-in power of the Warnings plugin, which most have installed already. If anyone is interested in my setup, let me know and I'll post it. Since the Warnings plugin plans to add cppcheck as a built-in parser and we already have our customer parser, we actually have no plans to return to this plugin. |
@kerrhome I would be interested in learning more about your setup. |
You'll find that under Jenkins->Configure in the section "Compiler Warnings". Look for the "Compiler Warnings" section and add a new parser. Give it a name like "Cppcheck-" so you can differentiate it in the pick list. The rest should be exactly what I have there. Then you can implement it in a pipeline like this:
To run cppcheck, use this option (at least):
and capture stderr in cppcheck.txt. (adjust things how you like of course). Hope that helps. |
This includes changes for the syntax generator as well as the other changes from Marco Steffan.