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

downgrade grpc to 1.26.0 #871

Merged
merged 1 commit into from
Jun 24, 2020
Merged

downgrade grpc to 1.26.0 #871

merged 1 commit into from
Jun 24, 2020

Conversation

C0rby
Copy link
Contributor

@C0rby C0rby commented Jun 23, 2020

In ocis we currently face the problem that some of our dependencies rely on
grpc 1.26.0. With the upgraded grpc dependency in reva we were unable to
build ocis. And if we upgrade grpc in ocis the build failed because of
the other dependency.

I hoped we could downgrade grpc in reva until we have found a better
long term solution.

Here are some examples of our attempts to resolve this without this
downgrade. owncloud/ocis#328 (comment)

@C0rby C0rby requested a review from labkode as a code owner June 23, 2020 12:44
In ocis we currently face the problem that some of our dependencies rely on
grpc 1.26.0. With the upgraded grpc dependency in reva we were unable to
build ocis. And if we upgrade grpc in ocis the build failed because of
the other dependency.

I hoped we could downgrade grpc in reva until we have found a better
long term solution.

Here are some examples of our attempts to resolve this without this
downgrade. owncloud/ocis#328 (comment)
@labkode labkode merged commit 793914d into cs3org:master Jun 24, 2020
@labkode
Copy link
Member

labkode commented Jun 24, 2020

@C0rby temporary downgrading, please tackle this issue also on your repo.

C0rby added a commit to owncloud/reva that referenced this pull request Jun 30, 2020
@ishank011
Copy link
Contributor

@C0rby any update on when we can revert this downgrade?

@C0rby
Copy link
Contributor Author

C0rby commented Oct 8, 2020

@ishank011, sadly one library we are using is still depending on that downgrade.
I'll try and see how we can maybe change that.

@C0rby C0rby deleted the downgrade-grpc branch October 8, 2020 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants