giraph icon indicating copy to clipboard operation
giraph copied to clipboard

GIRAPH-1172 Removed dependency com.google.code.findbugs:annotations

Open jdasch opened this issue 7 years ago • 2 comments

Replaced com.google.code.findbugs:annotations:2.0.2 with com.github.stephenc.findbugs:findbugs-annotations:1.3.9-1 and com.google.code.findbugs:jsr305:2.0.2.

Had to remove a couple annotations that don't exist in the older version of the drop-in replacement. There is a newer version of com.github.stephenc.findbugs:findbugs-annotations, but it hasn't been uploaded to maven central for some reason. Moved usage of those annotations to the findbugs-exclude.xml file.

jdasch avatar Feb 22 '18 19:02 jdasch

FindBugs is BSD, there's been some threads in other places indicating this is true based on the LICENSE file, but the maven package was built incorrectly.

adamkennedy avatar Mar 20 '18 03:03 adamkennedy

I did come across this thread [1], which discusses that com.google.code.findbugs:jsr305 is BSD, and documents that the JSR-305's BSD license is included in the findbugs repo [2]. But the license for FindBugs source is LGPL [3].

Unfortunately, Giraph is using com.google.code.findbugs:annotations which includes both JSR-305 and some FindBugs annotations [4], [5] which do have LGPL headers in the source listings.

[1] https://github.com/findbugsproject/findbugs/issues/128 [2] https://github.com/findbugsproject/findbugs/blob/2.0.2/findbugs/licenses/LICENSE-jsr305.txt [3] https://github.com/findbugsproject/findbugs/blob/2.0.2/findbugs/licenses/LICENSE.txt [4] https://github.com/findbugsproject/findbugs/blob/2.0.2/findbugs/src/java/edu/umd/cs/findbugs/annotations/SuppressFBWarnings.java [5] https://github.com/findbugsproject/findbugs/blob/2.0.2/findbugs/src/java/edu/umd/cs/findbugs/annotations/SuppressWarnings.java

jdasch avatar Mar 20 '18 19:03 jdasch