Add embedded jdk support for s390x#11965
Add embedded jdk support for s390x#11965ruixin-bao wants to merge 1 commit intobazelbuild:masterfrom
Conversation
This patch follows a similar approach used in PR bazelbuild#11436 to add support for embedding OpenJDK in bazel on s390x. Similar to ppc, the openjdk used will be downloadded from adoptopenjdk website.
|
Thank you! Merging this. |
|
@ruixin-bao Just super curious - can you confirm that with this change, Bazel can be built out of the box on s390x and works fine? 😀 |
|
Hi @philwo, thank you for the review :D.
nvm about the first half : ), see this being included in e14a6f2. Thanks for doing that.
Sure, for bazel master, last time I checked, it can be built. I will be testing again on s390x with bazel that contains this commit and reply back in this PR once I have the results. This patch allows embedded jdk to be built on s390x and this helps us running bazel tests a bit easier. |
|
Hi @philwo, I have got the results. Bazel can be built out of the box on s390x and works fine : ). I haven't run the bazel test suite yet though. Also, I am a bit curious about the binary release steps for bazel with another architecture (s390x), wondering if you mind giving some pointers to look at? I have seen #8833 and wondering if s390x can do a similar thing. Let me know and happy to help on that. Thanks. Bootstrap 3.4.1 Bazel Build bazel with this commit I can then use this bazel to build bazel master successfully following the similar steps (without --host_javabase=@local_jdk//:jdk anymore) as above |
|
@philwo friendly ping on the previous message, quoting here just in case.
Do let me know if I can provide any help. I am more than happy to help with any PRs, testing etc. that is required. Thanks! |
|
@ruixin-bao As we currently cannot cross-compile Bazel releases, we would need an s390x machine (or a way to emulate one) so that we can build Bazel releases directly on that platform. 😊 |
|
@philwo Thank you for your reply! Yes, we are able to provide VM access, wondering if you mind providing an email, then I can send you the relevant information regarding VM access? : ) |
|
@ruixin-bao Thank you, that sounds interesting! Please contact me at philwo@google.com. |
This patch follows a similar approach used in PR
#11436 to add support
for embedding OpenJDK in bazel on s390x.
Similar to ppc, the openjdk used will be downloadded from adoptopenjdk website.
/cc @philwo