-
-
Notifications
You must be signed in to change notification settings - Fork 41
Jenkins 35096 altran cppcheck plugin pipeline support - #37
Jenkins 35096 altran cppcheck plugin pipeline support #37AltranManatree wants to merge 3 commits into
Conversation
kerrhome
commented
Sep 25, 2017
Any chance you could share your build? I'd like to test it.
AltranManatree
commented
Sep 26, 2017
Hi Kerrhome,
Yes, we have a build branch that you can check. see :
https://github.com/AltranManatree/cppcheck-plugin/tree/jenkins-35096_Altran_cppcheck_plugin_pipeline_support
I believe this looks good. From #36 (comment):
"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.
https://wiki.jenkins.io/display/JENKINS/Adopt+a+Plugin"
Any chance you're prepared to do that?
yitshakyarom
commented
Oct 19, 2017
Any way to get the build artifact for this?
kumbasar
commented
Oct 20, 2017
@yitshakyarom, yes simple git clone https://github.com/AltranManatree/cppcheck-plugin/tree/jenkins-35096_Altran_cppcheck_plugin_pipeline_support and run "mvn install"
yitshakyarom
commented
Oct 22, 2017
via email
kerrhome
commented
Jan 30, 2018
I'm using this with Jenkins 2.89.2 and while it does give us the summary block on the individual build pages, if we click on the link to drill down into the cppcheck data, I get the following screen (pipeline and non-pipeline builds result in the same):
Stack trace
org.apache.commons.jelly.JellyTagException: jar:file:/var/lib/jenkins/plugins/cppcheck/WEB-INF/lib/cppcheck.jar!/org/jenkinsci/plugins/cppcheck/CppcheckResult/index.jelly:3:62: <st:include> No page found 'sidepanel.jelly' for class org.jenkinsci.plugins.cppcheck.CppcheckResult
Does anyone else see this?
kerrhome
commented
Feb 1, 2018
Looks like this is the bug: https://issues.jenkins-ci.org/browse/JENKINS-41704
michael-doubez
commented
May 2, 2018
I solved the conflicts and made fixes in pull request #39
Uh oh!
There was an error while loading. Please reload this page.
Add support for ccpcheck pipeline jenkins 2.0
Note:
Another feature branch, jenkins-35096_cppcheck_plugin_pipeline_support already exists, but due to the fact it is still open in Jenkins jira. We are pushing our finished working cppCheck plugin for review.
Our plugin has been tested in our working environment, and is working correctly in jenkins V2 pipeline.
As an addition, the step can be called as follows:
instead of
step([$class: 'CppcheckPublisher', allowNoReport: false, displayAllErrors: true, displayErrorSeverity: false, displayInformationSeverity: false, displayNoCategorySeverity: false, displayPerformanceSeverity: false, displayPortabilitySeverity: false, displayStyleSeverity: false, displayWarningSeverity: false, failureThreshold: '', healthy: '', ignoreBlankFiles: false, newFailureThreshold: '', newThreshold: '', numBuildsInGraph: 0, pattern: 'cppcheck.xml', severityError: true, severityInformation: true, severityNoCategory: true, severityPerformance: true, severityPortability: true, severityStyle: true, severityWarning: true, threshold: '', unHealthy: '', xSize: 500, ySize: 200])