Use cc implementation of singlejar except on Windows.

Change-Id: Ie4a2db2a6fa1ea86f4bdd42cd806f6804a0c56d4
PiperOrigin-RevId: 157473007

2 files changed, 14 insertions, 19 deletions

diff --git a/tools/android/BUILD.tools b/tools/android/BUILD.tools
index 2c2e10f18b..32d6eb9a6e 100644
--- a/tools/android/BUILD.tools
+++ b/tools/android/BUILD.tools
@@ -61,13 +61,13 @@ genrule(
         "@local_jdk//:bootclasspath"
     ],
     tools = [
-        "@bazel_tools//tools/jdk:SingleJar_binary",
+        "@bazel_tools//tools/jdk:singlejar",
     ],
     outs = ["java_lang_extras.jar"],
     cmd = """
           for jar in $(locations @local_jdk//:bootclasspath); do
             if [[ "$${jar}" == *"/rt.jar" ]]; then
-              $(location @bazel_tools//tools/jdk:SingleJar_binary) \
+              $(location @bazel_tools//tools/jdk:singlejar) \
                   --exclude_build_data \
                   --dont_change_compression \
                   --sources $${jar} \
diff --git a/tools/jdk/BUILD b/tools/jdk/BUILD
index 005ae91602..d52681033d 100644
--- a/tools/jdk/BUILD
+++ b/tools/jdk/BUILD
@@ -63,6 +63,17 @@ filegroup(
     srcs = glob(["ijar/*"]),
 )
 
+# On Windows, Java implementation of singlejar is used. We create a little
+# filegroup that contains the appropriate platform-dependent file.
+# Once https://github.com/bazelbuild/bazel/issues/2241 is fixed (that is,
+# the native singlejar is used on windows), this file group can be reused since
+# on Windows, executables end in ".exe", but the label we reach it through
+# must be platform-independent.
+filegroup(
+    name = "singlejar",
+    srcs = glob(["singlejar/*"]),
+)
+
 filegroup(
     name = "genclass",
     srcs = ["//tools/jdk:GenClass_deploy.jar"],
@@ -74,11 +85,6 @@ filegroup(
 )
 
 filegroup(
-    name = "singlejar",
-    srcs = ["//tools/jdk:SingleJar_deploy.jar"],
-)
-
-filegroup(
     name = "javabuilder",
     srcs = ["//tools/jdk:JavaBuilder_deploy.jar"],
 )
@@ -152,7 +158,7 @@ java_toolchain(
     misc = [
         "-XDskipDuplicateBridges=true",
     ],
-    singlejar = ["@bazel_tools//tools/jdk:SingleJar_deploy.jar"],
+    singlejar = ["@bazel_tools//tools/jdk:singlejar"],
     source_version = "8",
     target_version = "8",
 )
@@ -201,14 +207,3 @@ java_import(
     name = "JacocoCoverage",
     jars = [":JacocoCoverage_deploy.jar"],
 )
-
-java_import(
-    name = "SingleJar",
-    jars = ["SingleJar_deploy.jar"],
-)
-
-java_binary(
-    name = "SingleJar_binary",
-    main_class = "com.google.devtools.build.singlejar.SingleJar",
-    runtime_deps = [":SingleJar"],
-)