diff options
author | 2017-01-16 13:11:05 +0000 | |
---|---|---|
committer | 2017-01-16 13:48:55 +0000 | |
commit | 56328986232dfe4d92f738ab3bbd26d89236a428 (patch) | |
tree | e1a72698ef0a508b90eb303e5cb2b7602d43f283 /src/java_tools/junitrunner/java/com/google | |
parent | 3a7f5626976a3452427832dde0e0d1a09c57917d (diff) |
Jarjar the third-party dependencies of the Jacoco test runner.
This is the second encore of commit 823091f7516abf7d854021edc765daf1467f1647. The first one failed because jarjar tried to write to C:\WINDOWS , the second one failed because cygpath doesn't work for some reason on our CI machines and I hope that the "jarjar_bin cannot found" error message was only a fluke.
Otherwise, this'll be fixed online.
--
PiperOrigin-RevId: 144625508
MOS_MIGRATED_REVID=144625508
Diffstat (limited to 'src/java_tools/junitrunner/java/com/google')
-rw-r--r-- | src/java_tools/junitrunner/java/com/google/testing/coverage/BUILD | 33 | ||||
-rw-r--r-- | src/java_tools/junitrunner/java/com/google/testing/coverage/JacocoCoverage.jarjar | 2 |
2 files changed, 34 insertions, 1 deletions
diff --git a/src/java_tools/junitrunner/java/com/google/testing/coverage/BUILD b/src/java_tools/junitrunner/java/com/google/testing/coverage/BUILD index 319f13b6d2..89fc2fc5c3 100644 --- a/src/java_tools/junitrunner/java/com/google/testing/coverage/BUILD +++ b/src/java_tools/junitrunner/java/com/google/testing/coverage/BUILD @@ -22,7 +22,7 @@ filegroup( filegroup( name = "embedded_tools", srcs = [ - "JacocoCoverage_deploy.jar", + "JacocoCoverage_jarjar_deploy.jar", ], ) @@ -68,3 +68,34 @@ java_library( "//third_party:apache_commons_lang", ], ) + +genrule( + name = "Jacoco_jarjar", + srcs = [ + ":JacocoCoverage_deploy.jar", + ":JacocoCoverage.jarjar", + ], + outs = ["JacocoCoverage_jarjar_deploy.jar"], + cmd = "\n".join([ + # File#createTempFile() apparently tries to create the temp file under + # C:\WINDOWS by default on Windows, which is not writable. Help Java a bit. + # https://github.com/bazelbuild/bazel/issues/2349 will be a general + # solution for creating temp files in Java binaries in actions. Once + # that's fixed, we won't need to set TEMP here anymore. + "set +e", + "if [[ $$(uname -a) =~ MSYS ]]; then", + " DOS_TEMP=\"$$(cygpath -m --mydocs)/temp\";", + " PATH_DOS=$$?;", + " UNIX_TEMP=\"$$(cygpath -u --mydocs)/temp\";", + " if [[ $$? != 0 || $$PATH_DOS != 0 ]]; then", + " echo Cannot find temp directory, defaulting to /c/temp;", + " export DOS_TEMP=C:\\\\TEMP;", + " UNIX_TEMP=/c/temp;", + " fi", + " export TEMP=\"$$DOS_TEMP\";", + " mkdir -p \"$$UNIX_TEMP\";", + "fi", + "\"$(location //third_party/java/jarjar:jarjar_bin)\" process \"$(location :JacocoCoverage.jarjar)\" \"$(location :JacocoCoverage_deploy.jar)\" \"$@\"", + ]), + tools = ["//third_party/java/jarjar:jarjar_bin"], +) diff --git a/src/java_tools/junitrunner/java/com/google/testing/coverage/JacocoCoverage.jarjar b/src/java_tools/junitrunner/java/com/google/testing/coverage/JacocoCoverage.jarjar new file mode 100644 index 0000000000..56190d874c --- /dev/null +++ b/src/java_tools/junitrunner/java/com/google/testing/coverage/JacocoCoverage.jarjar @@ -0,0 +1,2 @@ +rule org.apache.commons.** com.google.testing.coverage.jarjar.@0 +rule org.objectweb.asm.** com.google.testing.coverage.jarjar.@0 |