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

Remove grpc sources from grpc++ #21662

Merged
merged 1 commit into from
Jan 15, 2020
Merged

Remove grpc sources from grpc++ #21662

merged 1 commit into from
Jan 15, 2020

Conversation

veblush
Copy link
Contributor

@veblush veblush commented Jan 15, 2020

Let's test this out. This is a rollback of commit in #20988, which introduced the ODR violation which causes an undefined behavior, in this case, segfault in trying to initialize the same global variable multiple times.

Related with #21213.

@veblush veblush added lang/core release notes: no Indicates if PR should not be in release notes labels Jan 15, 2020
@veblush veblush requested review from muxi and yashykt January 15, 2020 00:34
@veblush veblush marked this pull request as ready for review January 15, 2020 00:34
@veblush
Copy link
Contributor Author

veblush commented Jan 15, 2020

grpc_build_artifacts_multiplatform: passed.
All test failures below are irrelevant to this.

@veblush veblush merged commit 5a419c6 into grpc:master Jan 15, 2020
@veblush veblush deleted the fix-odr-bug branch January 15, 2020 16:49
veblush added a commit that referenced this pull request Jan 15, 2020
lidavidm added a commit to lidavidm/arrow that referenced this pull request Jan 17, 2020
gRPC 1.26.0 is broken due to multiple definitions of the same
symbol. For now, build with bundled gRPC.

grpc/grpc#21662
lidavidm added a commit to lidavidm/arrow that referenced this pull request Jan 17, 2020
gRPC 1.26.0 is broken due to multiple definitions of the same
symbol. For now, build with bundled gRPC.

grpc/grpc#21662
@lock lock bot locked as resolved and limited conversation to collaborators Apr 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lang/core release notes: no Indicates if PR should not be in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants