diff --git a/rules/android_local_test/impl.bzl b/rules/android_local_test/impl.bzl index 3a6221fd..7d4c1e9f 100644 --- a/rules/android_local_test/impl.bzl +++ b/rules/android_local_test/impl.bzl @@ -255,21 +255,11 @@ def _process_deploy_jar(ctx, java_package, jvm_ctx, proto_ctx, resources_ctx, ** def _preprocess_stub(ctx, **_unused_sub_ctxs): javabase = ctx.attr._current_java_runtime[java_common.JavaRuntimeInfo] - java_executable = str(javabase.java_executable_runfiles_path) + java_executable = javabase.java_executable_exec_path java_executable_files = javabase.files - # Absolute java_executable does not require any munging - if java_executable.startswith("/"): - java_executable = "JAVABIN=" + java_executable - - prefix = ctx.attr._runfiles_root_prefix[BuildSettingInfo].value - if not java_executable.startswith(prefix): - java_executable = prefix + java_executable - - java_executable = "JAVABIN=${JAVABIN:-${JAVA_RUNFILES}/" + java_executable + "}" - substitutes = { - "%javabin%": java_executable, + "%javabin%": "JAVABIN=" + java_executable, "%load_lib%": "", "%set_ASAN_OPTIONS%": "", } diff --git a/test/rules/android_local_test/BUILD b/test/rules/android_local_test/BUILD index 05850157..9020285d 100644 --- a/test/rules/android_local_test/BUILD +++ b/test/rules/android_local_test/BUILD @@ -88,9 +88,9 @@ android_local_test_launcher_test_suite( android_local_test_launcher_integration_test_suite( name = "android_local_test_launcher_integration_tests", expected_executable = select({ - ":jdk17_linux": "rules_android/../remotejdk17_linux/bin/java", - ":jdk17_macos": "rules_android/../remotejdk17_macos/bin/java", - ":jdk17_macos_aarch64": "rules_android/../remotejdk17_macos_aarch64/bin/java", + ":jdk17_linux": "external/remotejdk17_linux/bin/java", + ":jdk17_macos": "external/remotejdk17_macos/bin/java", + ":jdk17_macos_aarch64": "external/remotejdk17_macos_aarch64/bin/java", "//conditions:default": "rules_android/third_party/java/jdk/jdk-sts-k8/bin/java", }), ) diff --git a/test/rules/android_local_test/integration_test_stub_script.sh b/test/rules/android_local_test/integration_test_stub_script.sh index 9f836f22..0881c9fe 100644 --- a/test/rules/android_local_test/integration_test_stub_script.sh +++ b/test/rules/android_local_test/integration_test_stub_script.sh @@ -14,11 +14,11 @@ # limitations under the License. exit_with_error() { - echo "Expected to find JAVABIN=\${JAVABIN:-\${JAVA_RUNFILES}/%expected_executable% but was" + echo "Expected to find JAVABIN=%expected_executable% but was" echo $reference exit 1 } executable="%executable%" -reference=`grep -o 'JAVABIN=\\${JAVABIN:-\\${JAVA_RUNFILES}/.*}' $executable` -grep -c "JAVABIN:-\${JAVA_RUNFILES}/%expected_executable%" $executable >> /dev/null || exit_with_error +reference=`grep -o '^JAVABIN=.*' $executable` +grep -c "^JAVABIN=%expected_executable%$" $executable >> /dev/null || exit_with_error \ No newline at end of file