From b39a51cf9d4b90902379f9668e3e2ed6a4a057c9 Mon Sep 17 00:00:00 2001 From: Thi Doan Date: Mon, 7 Dec 2020 11:27:56 -0800 Subject: [PATCH] Explicitly state that embedding macOS OpenJDK is for x86_64 This allows adding additional macOS OpenJDK that targets arm64 arch later. Closes #12620. PiperOrigin-RevId: 346138763 --- WORKSPACE | 6 +++--- src/BUILD | 14 +++++++------- src/test/shell/testenv.sh | 2 +- 3 files changed, 11 insertions(+), 11 deletions(-) diff --git a/WORKSPACE b/WORKSPACE index c7f04306d5da2c..e37e806d6f6962 100644 --- a/WORKSPACE +++ b/WORKSPACE @@ -394,21 +394,21 @@ http_file( ) http_file( - name = "openjdk_macos", + name = "openjdk_macos_x86_64", downloaded_file_path = "zulu-macos.tar.gz", sha256 = "8e283cfd23c7555be8e17295ed76eb8f00324c88ab904b8de37bbe08f90e569b", urls = ["https://mirror.bazel.build/openjdk/azul-zulu11.37.17-ca-jdk11.0.6/zulu11.37.17-ca-jdk11.0.6-macosx_x64-allmodules-b23d4e05466f2aa1fdcd72d3d3a8e962206b64bf-1581689066.tar.gz"], ) http_file( - name = "openjdk_macos_vanilla", + name = "openjdk_macos_x86_64_vanilla", downloaded_file_path = "zulu-macos-vanilla.tar.gz", sha256 = "e1fe56769f32e2aaac95e0a8f86b5a323da5af3a3b4bba73f3086391a6cc056f", urls = ["https://mirror.bazel.build/openjdk/azul-zulu11.37.17-ca-jdk11.0.6/zulu11.37.17-ca-jdk11.0.6-macosx_x64.tar.gz"], ) http_file( - name = "openjdk_macos_minimal", + name = "openjdk_macos_x86_64_minimal", downloaded_file_path = "zulu-macos-minimal.tar.gz", sha256 = "1bacb1c07035d4066d79f0b65b4ea0ebd1954f3662bdfe3618da382ac8fd23a6", urls = ["https://mirror.bazel.build/openjdk/azul-zulu11.37.17-ca-jdk11.0.6/zulu11.37.17-ca-jdk11.0.6-macosx_x64-minimal-b23d4e05466f2aa1fdcd72d3d3a8e962206b64bf-1581689063.tar.gz"], diff --git a/src/BUILD b/src/BUILD index 32b45405887a4f..a655dbd335ba92 100644 --- a/src/BUILD +++ b/src/BUILD @@ -175,8 +175,8 @@ rule_size_test( filegroup( name = "embedded_jdk_allmodules_cached", srcs = select({ - "//src/conditions:darwin": [ - "@openjdk_macos//file", + "//src/conditions:darwin_x86_64": [ + "@openjdk_macos_x86_64//file", ], "//src/conditions:windows": [ "@openjdk_win//file", @@ -194,8 +194,8 @@ filegroup( filegroup( name = "embedded_jdk_minimal_cached", srcs = select({ - "//src/conditions:darwin": [ - "@openjdk_macos_minimal//file", + "//src/conditions:darwin_x86_64": [ + "@openjdk_macos_x86_64_minimal//file", ], "//src/conditions:windows": [ "@openjdk_win_minimal//file", @@ -213,8 +213,8 @@ filegroup( filegroup( name = "embedded_jdk_vanilla", srcs = select({ - "//src/conditions:darwin": [ - "@openjdk_macos_vanilla//file", + "//src/conditions:darwin_x86_64": [ + "@openjdk_macos_x86_64_vanilla//file", ], "//src/conditions:windows": [ "@openjdk_win_vanilla//file", @@ -709,7 +709,7 @@ filegroup( "@openjdk15_windows_archive//:WORKSPACE", "@openjdk_linux_aarch64_minimal//file", "@openjdk_linux_minimal//file", - "@openjdk_macos_minimal//file", + "@openjdk_macos_x86_64_minimal//file", "@openjdk_win_minimal//file", "@remote_coverage_tools_for_testing//:WORKSPACE", "@remote_java_tools_darwin_for_testing//:WORKSPACE", diff --git a/src/test/shell/testenv.sh b/src/test/shell/testenv.sh index 6f23519be62a25..e827826df26c7f 100755 --- a/src/test/shell/testenv.sh +++ b/src/test/shell/testenv.sh @@ -294,7 +294,7 @@ EOF "openjdk15_windows_archive" "openjdk_linux_aarch64_minimal" "openjdk_linux_minimal" - "openjdk_macos_minimal" + "openjdk_macos_x86_64_minimal" "openjdk_win_minimal" "remote_coverage_tools_for_testing" "remote_java_tools_for_testing"