This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/reef.git
The following commit(s) were added to refs/heads/master by this push:
new 66baaad [REEF-2062] Migrate from FindBugs to SpotBugs
66baaad is described below
commit 66baaad3161d2ad9cf296893cac2d1d0d05f9f93
Author: Sergiy Matusevych <[email protected]>
AuthorDate: Thu May 21 18:58:38 2020 -0700
[REEF-2062] Migrate from FindBugs to SpotBugs
FindBugs is not supported anymore, so we're migrating to the latest version
of SpotBugs.
Summary of changes:
* Update Maven pom.xml files to reference to the latest verison of SpotBugs
* Rename teh related config files and intermediate files and make sure the
SpotBugs report is correct
* No changes in code were required.
JIRA:
[REEF-2062](https://issues.apache.org/jira/projects/REEF/issues/REEF-2062)
---
lang/java/reef-bridge-proto-java/pom.xml | 4 +--
.../{findbugs-exclude.xml => spotbugs-exclude.xml} | 0
lang/scala/reef-examples-scala/pom.xml | 4 +--
pom.xml | 29 +++++++++++-----------
4 files changed, 19 insertions(+), 18 deletions(-)
diff --git a/lang/java/reef-bridge-proto-java/pom.xml
b/lang/java/reef-bridge-proto-java/pom.xml
index 8fec306..56c17b6 100644
--- a/lang/java/reef-bridge-proto-java/pom.xml
+++ b/lang/java/reef-bridge-proto-java/pom.xml
@@ -283,8 +283,8 @@ under the License.
<artifactId>maven-compiler-plugin</artifactId>
</plugin>
<plugin>
- <groupId>org.codehaus.mojo</groupId>
- <artifactId>findbugs-maven-plugin</artifactId>
+ <groupId>com.github.spotbugs</groupId>
+ <artifactId>spotbugs-maven-plugin</artifactId>
<configuration>
<skip>true</skip>
</configuration>
diff --git a/lang/java/reef-common/src/main/resources/findbugs-exclude.xml
b/lang/java/reef-common/src/main/resources/spotbugs-exclude.xml
similarity index 100%
rename from lang/java/reef-common/src/main/resources/findbugs-exclude.xml
rename to lang/java/reef-common/src/main/resources/spotbugs-exclude.xml
diff --git a/lang/scala/reef-examples-scala/pom.xml
b/lang/scala/reef-examples-scala/pom.xml
index 85e0258..57a1dbb 100644
--- a/lang/scala/reef-examples-scala/pom.xml
+++ b/lang/scala/reef-examples-scala/pom.xml
@@ -33,8 +33,8 @@ under the License.
<properties>
<rootPath>${basedir}/../../..</rootPath>
- <!-- Findbugs does not support Scala code -->
- <findbugs.skip>true</findbugs.skip>
+ <!-- SpotBugs does not support Scala code -->
+ <spotbugs.skip>true</spotbugs.skip>
</properties>
<dependencies>
diff --git a/pom.xml b/pom.xml
index 8316903..17e9151 100644
--- a/pom.xml
+++ b/pom.xml
@@ -62,7 +62,7 @@ under the License.
<maven-checkstyle-plugin.version>3.1.0</maven-checkstyle-plugin.version>
<sevntu.checkstyle.plugin.version>1.35.0</sevntu.checkstyle.plugin.version>
<checkstyle.version>8.29</checkstyle.version>
- <findbugs.version>3.0.4</findbugs.version>
+ <spotbugs.version>3.1.12.2</spotbugs.version>
<reflections.version>0.9.9-RC1</reflections.version>
<jsr305.version>3.0.1</jsr305.version>
<kryo.version>3.0.3</kryo.version>
@@ -361,16 +361,17 @@ under the License.
<includeTestSourceDirectory>true</includeTestSourceDirectory>
</configuration>
</plugin>
+
<plugin>
- <groupId>org.codehaus.mojo</groupId>
- <artifactId>findbugs-maven-plugin</artifactId>
- <version>${findbugs.version}</version>
+ <groupId>com.github.spotbugs</groupId>
+ <artifactId>spotbugs-maven-plugin</artifactId>
+ <version>${spotbugs.version}</version>
<configuration>
<effort>Max</effort>
<threshold>Default</threshold>
<failOnError>true</failOnError>
-
<findbugsXmlOutputDirectory>${project.build.directory}/findbugs</findbugsXmlOutputDirectory>
-
<excludeFilterFile>${rootPath}/lang/java/reef-common/src/main/resources/findbugs-exclude.xml</excludeFilterFile>
+
<spotbugsXmlOutputDirectory>${project.build.directory}/spotbugs</spotbugsXmlOutputDirectory>
+
<excludeFilterFile>${rootPath}/lang/java/reef-common/src/main/resources/spotbugs-exclude.xml</excludeFilterFile>
</configuration>
</plugin>
<plugin>
@@ -379,19 +380,19 @@ under the License.
<version>1.0</version>
<dependencies>
<dependency>
- <groupId>org.codehaus.mojo</groupId>
- <artifactId>findbugs-maven-plugin</artifactId>
- <version>${findbugs.version}</version>
+ <groupId>com.github.spotbugs</groupId>
+ <artifactId>spotbugs-maven-plugin</artifactId>
+ <version>${spotbugs.version}</version>
</dependency>
</dependencies>
<configuration>
<transformationSets>
<transformationSet>
- <dir>${project.build.directory}/findbugs</dir>
+ <dir>${project.build.directory}/spotbugs</dir>
<includes>
- <include>findbugsXml.xml</include>
+ <include>spotbugsXml.xml</include>
</includes>
-
<outputDir>${project.build.directory}/findbugs</outputDir>
+
<outputDir>${project.build.directory}/spotbugs</outputDir>
<stylesheet>default.xsl</stylesheet>
<fileMappers>
<!-- Configures the file extension of the
output files. -->
@@ -903,8 +904,8 @@ under the License.
</executions>
</plugin>
<plugin>
- <groupId>org.codehaus.mojo</groupId>
- <artifactId>findbugs-maven-plugin</artifactId>
+ <groupId>com.github.spotbugs</groupId>
+ <artifactId>spotbugs-maven-plugin</artifactId>
<executions>
<execution>
<id>analyze-compile</id>
