-
Notifications
You must be signed in to change notification settings - Fork 15.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Build extensions for Ruby 2.7 #7027
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
8369552
to
d13d160
Compare
CLAs look good, thanks! ℹ️ Googlers: Go here for more info. |
77edea4
to
165bdc3
Compare
@haberman sorry for the ping, but any change for this to be merged ? |
No worries about the ping, please take a look at the test failures. Feel free to ping me again. |
We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google. ℹ️ Googlers: Go here for more info. |
c2a88ad
to
5782756
Compare
CLAs look good, thanks! ℹ️ Googlers: Go here for more info. |
Ah OK, the CI doesn't run automatically. |
I guess there's a mismatch in bundler versions? one is from |
Yeah that's a classic error with Ruby 2.7 |
We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google. ℹ️ Googlers: Go here for more info. |
@qnighy Looks like we're still getting an error. :(
|
grpc/grpc@4c66747 do we perhaps need to get rid of |
@haberman I'd like to see what happens when I remove the custom RCD image. Could you kick the CI again? |
Okay, this change must be important: grpc/grpc@4c66747#diff-52c976fc38ed2b4e3b1192f8a8e24cffL131-R150 @haberman one more kick, please? Thanks in advance! |
Looks like we're still getting an error:
Do you think it's worth trying
In my experience, |
I'll instead follow the grpc/grpc's approach: grpc/grpc@09bc1ee |
I guess when I update dockerfiles in Then, in this PR, I'd like to focus on building extensions for Ruby 2.7. I'll prepare another PR for testing protobuf in Ruby 2.7. |
@haberman can it be merged? I think the remaining CI failures are unrelated to this PR. |
Thanks a lot! I'm really glad to see this merged. |
I've made two groundworks for building protobuf extensions for Ruby 2.7:
Now that 2.7.0-rc2 has been released, can we start building extensions for Ruby 2.7?