Skip to content

Commit

Permalink
Remove third_party/checker_* from the binary.
Browse files Browse the repository at this point in the history
The binaries were there because there Java tooling needed it (until it
was moved to its own separate repository) and the sources were there for
GPL compliance (which is not the case anymore since we don't distribute
the binaries, either)

To be extra safe on the GPL compliance side, I checked with
```
bazel query "rdeps(//...,//third_party/checker_framework_{javacutil,dataflow})"'
```

whether we have any dependencies on it that might end up being
distributed; turns out, the only dependencies are JavaBuilder (which is
not distributed with Bazel anymore, as proven by the above query) and some tests.

Closes #8163.

PiperOrigin-RevId: 245416925
  • Loading branch information
lberki authored and copybara-github committed Apr 26, 2019
1 parent 4b092cd commit 6362c2d
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -605,6 +605,8 @@ JAVA_VERSIONS = ("9", "10")
"//src/java_tools/junitrunner:srcs",
"//src/java_tools/singlejar:srcs",
"//src/tools/singlejar:embedded_java_tools",
"//third_party/checker_framework_dataflow:srcs",
"//third_party/checker_framework_javacutil:srcs",
"//third_party/jarjar:srcs",
"//third_party/ijar:transitive_sources",
"//third_party/java/jacoco:jacoco_source_jars_zip",
Expand Down

0 comments on commit 6362c2d

Please sign in to comment.