Skip to content

Run ruby aarch64 tests on the CI (with an emulator)#8495

Merged
jtattermusch merged 3 commits into
protocolbuffers:masterfrom
jtattermusch:run_ruby_aarch64_test
Apr 26, 2021
Merged

Run ruby aarch64 tests on the CI (with an emulator)#8495
jtattermusch merged 3 commits into
protocolbuffers:masterfrom
jtattermusch:run_ruby_aarch64_test

Conversation

@jtattermusch
Copy link
Copy Markdown
Contributor

Based on #8485 (only the few last commits are ruby related).

Ruby's native build is very simple (it only compiles a few .c files), so running ruby's build directly under an emulator is the simplest solution and it's quick enough. Only the protoc binaries get crosscompiled since building protoc under an emulator would be slow.

@jtattermusch
Copy link
Copy Markdown
Contributor Author

@dlj-NaN please review or reassign to protobuf ruby owner.

Copy link
Copy Markdown
Contributor

@fowles fowles left a comment

Choose a reason for hiding this comment

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

this appears to include both java and python as part of this. Please update these to be separate or commit earlier ones before pushing follow ons

@fowles fowles requested review from haberman and removed request for dlj-NaN April 22, 2021 15:10
@fowles
Copy link
Copy Markdown
Contributor

fowles commented Apr 22, 2021

Josh, you can wait for jtattermusch to separate the change a bit before reviewing

@jtattermusch
Copy link
Copy Markdown
Contributor Author

@fowles I rebased the PR. The original reason for depending on the other changes was that is it reuses kokoro/linux/aarch64/protoc_crosscompile_aarch64.sh introduced by the java PR.
To make the review simpler, I simply added an identical file into this PR.

@haberman @fowles please review.

@jtattermusch
Copy link
Copy Markdown
Contributor Author

@haberman is this ok to merge?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants