Skip to content

Commit

Permalink
[JENKINS-23575] Post-Build-Step "Publish Cppcheck results" not availa…
Browse files Browse the repository at this point in the history
…ble in job generator project

- isApplicable() returns true, to make Cppcheck publisher available in a whatever job type.
- Dependencies to maven and ivy plugins removed.
  • Loading branch information
mixalturek committed Jul 19, 2014
1 parent 4f53138 commit b166231
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 26 deletions.
17 changes: 2 additions & 15 deletions pom.xml
Expand Up @@ -36,8 +36,8 @@
<scm>
<connection>scm:git:git://github.com/jenkinsci/cppcheck-plugin.git</connection>
<developerConnection>scm:git:git@github.com:jenkinsci/cppcheck-plugin.git</developerConnection>
<tag>HEAD</tag>
</scm>
<tag>HEAD</tag>
</scm>

<properties>
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
Expand All @@ -49,17 +49,11 @@
<java2html.version>5.0</java2html.version>
<junit.version>4.8.2</junit.version>
<mockito.version>1.8.5</mockito.version>
<ivy.plugin.version>1.19</ivy.plugin.version>
<dashboard.view.version>2.0</dashboard.view.version>
</properties>

<dependencies>

<dependency>
<groupId>org.jenkins-ci.main</groupId>
<artifactId>maven-plugin</artifactId>
</dependency>

<dependency>
<groupId>javax.xml.bind</groupId>
<artifactId>jaxb-api</artifactId>
Expand Down Expand Up @@ -99,13 +93,6 @@
<scope>test</scope>
</dependency>

<dependency>
<groupId>org.jenkins-ci.plugins</groupId>
<artifactId>ivy</artifactId>
<version>${ivy.plugin.version}</version>
<optional>true</optional>
</dependency>

<dependency>
<groupId>org.jenkins-ci.plugins</groupId>
<artifactId>dashboard-view</artifactId>
Expand Down
Expand Up @@ -6,8 +6,6 @@
import hudson.FilePath;
import hudson.Launcher;
import hudson.XmlFile;
import hudson.matrix.MatrixProject;
import hudson.maven.MavenModuleSet;
import hudson.model.*;
import hudson.remoting.VirtualChannel;
import hudson.tasks.BuildStepDescriptor;
Expand Down Expand Up @@ -217,15 +215,7 @@ public CppcheckDescriptor() {
}

public boolean isApplicable(Class<? extends AbstractProject> jobType) {
boolean isIvyProject = false;
if (Hudson.getInstance().getPlugin("ivy") != null) {
isIvyProject = hudson.ivy.AbstractIvyProject.class.isAssignableFrom(jobType);
}

return FreeStyleProject.class.isAssignableFrom(jobType)
|| MavenModuleSet.class.isAssignableFrom(jobType)
|| MatrixProject.class.isAssignableFrom(jobType)
|| isIvyProject;
return true;
}

@Override
Expand Down

0 comments on commit b166231

Please sign in to comment.