Skip to content

Commit

Permalink
Fixes build issue with JDK headers location on ppc64le.
Browse files Browse the repository at this point in the history
  • Loading branch information
@ceseo
ceseo committed Jun 29, 2020
1 parent 920b093 commit bc22ab4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tools/jdk/BUILD
View file
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ cc_library(
name = "jni",
hdrs = [":jni_header"] + select({
"//src/conditions:linux_aarch64": [":jni_md_header-linux"],
"//src/conditions:linux_ppc": [":jni_md_header-linux"],
"//src/conditions:linux_ppc64le": [":jni_md_header-linux"],
"//src/conditions:linux_s390x": [":jni_md_header-linux"],
"//src/conditions:linux_x86_64": [":jni_md_header-linux"],
"//src/conditions:darwin": [":jni_md_header-darwin"],
Expand All @@ -146,7 +146,7 @@ cc_library(
}),
includes = ["include"] + select({
"//src/conditions:linux_aarch64": ["include/linux"],
"//src/conditions:linux_ppc": ["include/linux"],
"//src/conditions:linux_ppc64le": ["include/linux"],
"//src/conditions:linux_s390x": ["include/linux"],
"//src/conditions:linux_x86_64": ["include/linux"],
"//src/conditions:darwin": ["include/darwin"],
Expand Down

1 comment on commit bc22ab4

@ceseo
Copy link
Contributor Author

@ceseo ceseo commented on bc22ab4 Jun 29, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I submitted the wrong patch before.

Question: does bazel support 32-bit PPC or big endian PPC64? If so, we need an alternative, because adding conditions for those simultaneously with ppc64le causes an ambiguity error.

Please sign in to comment.