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

Migrate to protobuf javalite #8

Closed
wants to merge 2 commits into from
Closed

Conversation

FZambia
Copy link
Member

@FZambia FZambia commented Mar 11, 2020

No description provided.

@startoftext
Copy link

Any update on getting this merged/released?

The reason I ask is because I am trying to use Centrifugo on android and firebase is using protobuf-javalite. This causes a build error because multiple classes have the same path and name.

@FZambia
Copy link
Member Author

FZambia commented Apr 14, 2021

@startoftext hello!

I think it's possible to make everything work by excluding protobuf-javalite dependency from your project (see this comment). This way firebase and centrifuge-java should use full protobuf implementation without conflict.

I'll try to find a time to merge this and make new release very soon. Maybe in a couple of weeks but can't really promise any concrete deadlines - a bit busy these days at new job and concentrating on Centrifuge/Centrifugo server improvements at my spare time.

@FZambia
Copy link
Member Author

FZambia commented May 2, 2021

Done in #22

@FZambia FZambia closed this May 2, 2021
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.

2 participants