Skip to content
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

Removed references to public APIs from codegen/ #7559

Merged
merged 21 commits into from
Sep 14, 2016

Conversation

dgquintas
Copy link
Contributor

@dgquintas dgquintas commented Jul 28, 2016

Fixes #6023

This PR looks scary but there's (almost) no new code, simply lines moving around. The exception to the no-new-code is the addition of some methods to the CoreInterface API (see core_codegen_interface.h and core_codegen.{h,cc}).

@ctiller
Copy link
Member

ctiller commented Aug 31, 2016

LGTM. objc failure seems scary though - @jcanizales PTAL.

@dgquintas
Copy link
Contributor Author

It seems to have been failing for a while
https://grpc-testing.appspot.com/job/gRPC_pull_requests/12543/config=dbg,language=objc,platform=macos/testReport/(root)/tests/objc_examples_build/history/

On 1 Sep 2016 12:30 am, "Craig Tiller" notifications@github.com wrote:

LGTM. objc failure seems scary though - @jcanizales
https://github.com/jcanizales PTAL.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#7559 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAHVmSB3qTFSXjSqlQTjBm-AFGrzlDblks5qlgBqgaJpZM4JW5Wk
.

@jcanizales
Copy link
Contributor

@makdharma and @muxi for ObjC Jenkins. I see a timeout - Craig, is that the one you mean, or am I looking at the wrong place?

@muxi
Copy link
Member

muxi commented Aug 31, 2016

The timeout of objc-examples-build happens when the macos machines get busy. However, logs of other previous PRs showed that macos node 1's cocoapods repo is not up to date. Expected to happen since node 1 was offline for a while last few weeks, due to which we stopped automatic update of cocoapods repo. I am updating it right now and should be no problem.

Can we try a retest?

@dgquintas
Copy link
Contributor Author

Test this please

@dgquintas
Copy link
Contributor Author

Yes, the objc failure is gone, thanks!

@dgquintas
Copy link
Contributor Author

@kpayson64 can this be merged forcefully? The interop tests are still failing across the board and for unrelated reasons it seems.

@ctiller ctiller merged commit a51f541 into grpc:master Sep 14, 2016
crclark added a commit to crclark/gRPC-haskell that referenced this pull request Mar 1, 2017
crclark added a commit to awakesecurity/gRPC-haskell that referenced this pull request Apr 15, 2017
* update for reorganized headers in gRPC v1.1: grpc/grpc#7559

* update nix files

* rename all gpr_slice to grpc_slice since we can't be backwards compatible anyway

* use gRPC 1.1.4

* now compatible with gRPC 1.2.0

* use nixpkgs 17.03 to get protobuf3_2, newer proto3-* deps, fix some warnings

* another warning
@lock lock bot locked as resolved and limited conversation to collaborators Jan 26, 2019
@lock lock bot unassigned sreecha Jan 26, 2019
@thelinuxfoundation
Copy link

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.

After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

Regards,
CLA GitHub bot

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants