Skip to content

Conversation

@liusheng
Copy link
Contributor

@liusheng liusheng commented Nov 19, 2019

This change modified the building scripts to support cross compile
grpc-java for aarch64 platform

For #6364

This change modified the building scripts to support cross compile
grpc-java for aarch64 platform

For #grpc/grpc-java/issues#6364
@liusheng
Copy link
Contributor Author

liusheng commented Dec 6, 2019

Hi @ejona86,

I have updated this PR and now the buildscripts/kokoro/linux_artifacts.sh can be used to build the compiler for aarch64 platform. cloud you please take a look ?

Thanks a lot!

@ejona86 ejona86 added the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Dec 6, 2019
@grpc-kokoro grpc-kokoro removed the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Dec 6, 2019
Copy link
Member

@ejona86 ejona86 left a comment

Choose a reason for hiding this comment

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

Looks pretty close. Just some things that look unnecessary. I've started a Kokoro run, so you can see what the output looks like.

@ejona86 ejona86 added the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Dec 6, 2019
@grpc-kokoro grpc-kokoro removed the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Dec 6, 2019
@ejona86 ejona86 added the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Dec 6, 2019
@grpc-kokoro grpc-kokoro removed the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Dec 6, 2019
Copy link
Member

@ejona86 ejona86 left a comment

Choose a reason for hiding this comment

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

Need to still wait for this last build to complete, but it looks like this will work.

@ejona86 ejona86 requested a review from creamsoup December 6, 2019 18:57
@ejona86 ejona86 added the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Dec 6, 2019
@grpc-kokoro grpc-kokoro removed the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Dec 6, 2019
@ejona86 ejona86 added the TODO:backport PR needs to be backported. Removed after backport complete label Dec 6, 2019
Copy link
Contributor

@creamsoup creamsoup left a comment

Choose a reason for hiding this comment

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

LGTM

@ejona86 ejona86 merged commit 470a4ab into grpc:master Dec 6, 2019
@ejona86
Copy link
Member

ejona86 commented Dec 6, 2019

@liusheng, thank you!

@ejona86 ejona86 removed the TODO:backport PR needs to be backported. Removed after backport complete label Dec 6, 2019
@liusheng liusheng deleted the 6364 branch December 7, 2019 01:05
@liusheng
Copy link
Contributor Author

liusheng commented Dec 7, 2019

@ejona86 thanks a lot for changing and proposing this change :)
Is there plan to release and publish the artifacts including aarch64 support to maven central ? I will be very appreciated if you can help to do that.

@ejona86
Copy link
Member

ejona86 commented Dec 9, 2019

@liusheng, with #6496 it will go out in the 1.26 release, which is scheduled for Dec 17th.

@liusheng
Copy link
Contributor Author

@ejona86 got it, thank you!

@lock lock bot locked as resolved and limited conversation to collaborators Mar 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants