Skip to content

Commit

Permalink
Update to latest javac and Error Prone
Browse files Browse the repository at this point in the history
Fixes #2591

Change-Id: I7038b71811c54d5ece1c0993052abdffc9d0de96
  • Loading branch information
cushon authored and damienmg committed Mar 2, 2017
1 parent 105db84 commit 3049051
Show file tree
Hide file tree
Showing 15 changed files with 22 additions and 22 deletions.
2 changes: 1 addition & 1 deletion scripts/bootstrap/compile.sh
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
# Script for building bazel from scratch without bazel

PROTO_FILES=$(ls src/main/protobuf/*.proto src/main/java/com/google/devtools/build/lib/buildeventstream/proto/*.proto)
LIBRARY_JARS=$(find third_party -name '*.jar' | grep -Fv /javac-9-dev-r3297-3.jar | grep -Fv /javac7.jar | grep -Fv JavaBuilder | grep -ve third_party/grpc/grpc.*jar | tr "\n" " ")
LIBRARY_JARS=$(find third_party -name '*.jar' | grep -Fv /javac-9-dev-r3297-4.jar | grep -Fv /javac7.jar | grep -Fv JavaBuilder | grep -ve third_party/grpc/grpc.*jar | tr "\n" " ")
GRPC_JAVA_VERSION=0.15.0
GRPC_LIBRARY_JARS=$(find third_party/grpc -name '*.jar' | grep -e .*${GRPC_JAVA_VERSION}.*jar | tr "\n" " ")
LIBRARY_JARS="${LIBRARY_JARS} ${GRPC_LIBRARY_JARS}"
Expand Down
6 changes: 3 additions & 3 deletions src/create_embedded_tools.sh
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ for i in $*; do
*JavaBuilder*_deploy.jar) OUTPUT_PATH=tools/jdk/JavaBuilder_deploy.jar ;;
*JacocoCoverage*_deploy.jar) OUTPUT_PATH=tools/jdk/JacocoCoverage_deploy.jar ;;
*turbine_deploy.jar) OUTPUT_PATH=tools/jdk/turbine_deploy.jar ;;
*javac-9-dev-r3297-3.jar) OUTPUT_PATH=third_party/java/jdk/langtools/javac-9-dev-r3297-3.jar ;;
*javac-9-dev-r3297-4.jar) OUTPUT_PATH=third_party/java/jdk/langtools/javac-9-dev-r3297-4.jar ;;
*javac7.jar) OUTPUT_PATH=third_party/java/jdk/langtools/javac7.jar ;;
*SingleJar_deploy.jar) OUTPUT_PATH=tools/jdk/SingleJar_deploy.jar ;;
*GenClass_deploy.jar) OUTPUT_PATH=tools/jdk/GenClass_deploy.jar ;;
Expand All @@ -68,9 +68,9 @@ for i in $*; do
chmod u+w "${PACKAGE_DIR}/${OUTPUT_PATH}"
done

if [ ! -f ${PACKAGE_DIR}/third_party/java/jdk/langtools/javac-9-dev-r3297-3.jar ]; then
if [ ! -f ${PACKAGE_DIR}/third_party/java/jdk/langtools/javac-9-dev-r3297-4.jar ]; then
cp ${PACKAGE_DIR}/third_party/java/jdk/langtools/javac7.jar \
${PACKAGE_DIR}/third_party/java/jdk/langtools/javac-9-dev-r3297-3.jar
${PACKAGE_DIR}/third_party/java/jdk/langtools/javac-9-dev-r3297-4.jar
fi

cat > "${PACKAGE_DIR}/WORKSPACE" <<EOF
Expand Down
4 changes: 2 additions & 2 deletions src/test/shell/testenv.sh
Original file line number Diff line number Diff line change
Expand Up @@ -364,8 +364,8 @@ function create_new_workspace() {

copy_tools_directory

[ -e third_party/java/jdk/langtools/javac-9-dev-r3297-3.jar ] \
|| ln -s "${langtools_path}" third_party/java/jdk/langtools/javac-9-dev-r3297-3.jar
[ -e third_party/java/jdk/langtools/javac-9-dev-r3297-4.jar ] \
|| ln -s "${langtools_path}" third_party/java/jdk/langtools/javac-9-dev-r3297-4.jar

touch WORKSPACE
}
Expand Down
18 changes: 9 additions & 9 deletions third_party/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -310,10 +310,10 @@ java_import(
java_import(
name = "error_prone",
jars = [
"error_prone/error_prone_check_api-2.0.18-20170224.jar",
"error_prone/error_prone_core-2.0.18-20170224.jar",
"error_prone/error_prone_annotations-2.0.18-20170224.jar",
"error_prone/error_prone_annotation-2.0.18-20170224.jar",
"error_prone/error_prone_check_api-2.0.18.jar",
"error_prone/error_prone_core-2.0.18.jar",
"error_prone/error_prone_annotations-2.0.18.jar",
"error_prone/error_prone_annotation-2.0.18.jar",
],
exports = [
":auto_common",
Expand Down Expand Up @@ -345,10 +345,10 @@ filegroup(
name = "bootstrap_guava_and_error_prone-jars",
srcs = [
"auto/auto-common-0.3.jar",
"error_prone/error_prone_annotation-2.0.18-20170224.jar",
"error_prone/error_prone_annotations-2.0.18-20170224.jar",
"error_prone/error_prone_check_api-2.0.18-20170224.jar",
"error_prone/error_prone_core-2.0.18-20170224.jar",
"error_prone/error_prone_annotation-2.0.18.jar",
"error_prone/error_prone_annotations-2.0.18.jar",
"error_prone/error_prone_check_api-2.0.18.jar",
"error_prone/error_prone_core-2.0.18.jar",
"guava/guava-21.0.jar",
"jcip_annotations/jcip-annotations-1.0-1.jar",
"jsr305/jsr-305.jar",
Expand All @@ -367,7 +367,7 @@ filegroup(
java_import(
name = "guava",
jars = [
"error_prone/error_prone_annotations-2.0.18-20170224.jar",
"error_prone/error_prone_annotations-2.0.18.jar",
"jcip_annotations/jcip-annotations-1.0-1.jar",
"jsr305/jsr-305.jar",
] + select({
Expand Down
4 changes: 2 additions & 2 deletions third_party/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ a minimal set of extra dependencies.

## [Error Prone](https://github.com/google/error-prone)

* Version: 2.0.18-20170224
* Version: 2.0.18
* License: Apache License 2.0


Expand Down Expand Up @@ -166,7 +166,7 @@ a minimal set of extra dependencies.

## [javac](https://github.com/google/error-prone-javac)

* Version: 9-dev-r3297-3 (javac-9-dev-r3297-3.jar)
* Version: 9-dev-r3297-4 (javac-9-dev-r3297-4.jar)
* License: GNU GPL v2 with Classpath exception (plus other licenses, see third_party/java/jdk/langtools/LICENSE file).

## [jarjar](https://code.google.com/p/jarjar/)
Expand Down
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
2 changes: 1 addition & 1 deletion third_party/java/jdk/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ version of the JRE. Those builds are vendored in
//third_party/java/jdk/langtools.

Currently Bazel supports running on a JRE 8 only because the default Java
compiler used (//third_party/java/jdk/langtools/javac-9-dev-r3297-3.jar) is the
compiler used (//third_party/java/jdk/langtools/javac-9-dev-r3297-4.jar) is the
Java compiler of OpenJDK 9 compiled to run on a JRE 8. This cannot
be built to run on a JRE 7 because of code incompatibility. Bazel's
JavaBuilder at HEAD cannot be linked with earlier version of the
Expand Down
8 changes: 4 additions & 4 deletions third_party/java/jdk/langtools/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -19,22 +19,22 @@ filegroup(
"//conditions:default": [
"BUILD",
"LICENSE",
"javac-9-dev-r3297-3.jar",
"javac-9-dev-r3297-3.srcjar",
"javac-9-dev-r3297-4.jar",
"javac-9-dev-r3297-4.srcjar",
],
}),
)

java_import(
name = "javac",
jars = ["javac-9-dev-r3297-3.jar"],
jars = ["javac-9-dev-r3297-4.jar"],
)

filegroup(
name = "javac_jar",
srcs = select({
"//tools/jdk:jdk7": ["javac7.jar"],
"//conditions:default": ["javac-9-dev-r3297-3.jar"],
"//conditions:default": ["javac-9-dev-r3297-4.jar"],
}),
)

Expand Down
Binary file not shown.
Binary file not shown.

0 comments on commit 3049051

Please sign in to comment.