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

Add current alpha of protobuf 3 #9011

Merged
merged 1 commit into from
Jul 27, 2015
Merged

Add current alpha of protobuf 3 #9011

merged 1 commit into from
Jul 27, 2015

Conversation

eduarrrd
Copy link
Contributor

No description provided.

@edolstra
Copy link
Member

I'm generally not in favor of adding alpha versions of software. Do we need this somewhere?

@wkennington
Copy link
Contributor

I believe if you want to use grpc you need protobuf3. This would be a fantastic addition as I've been wanting to write some tools which leverage grpc for rpc.

@wkennington
Copy link
Contributor

I should also mention that it turns out bazel needs this version of protobuf if we want to keep that up to date. #9008

@eduarrrd
Copy link
Contributor Author

I'm doing grpc stuff with proto3 which is why I would like to use it.
(Also, arguable Google's "alpha" might be a bit more stable than alpha
might suggest.)

On Mon, Jul 27, 2015 at 12:15 PM William A. Kennington III <
notifications@github.com> wrote:

I should also mention that it turns out bazel needs this version of
protobuf if we want to keep that up to date. #9008
#9008


Reply to this email directly or view it on GitHub
#9011 (comment).

@edolstra
Copy link
Member

Okay. Can you mark it as lowPrio in all-packages.nix?

@eduarrrd
Copy link
Contributor Author

Done.

wkennington added a commit that referenced this pull request Jul 27, 2015
Add current alpha of protobuf 3
@wkennington wkennington merged commit e4956ba into NixOS:master Jul 27, 2015
@philandstuff philandstuff mentioned this pull request Jul 28, 2015
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