-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Adding grpcio to setup.py as an extra. #1553
Conversation
Would it make sense to make this an extra? |
I like the idea of the extra: users can make perfectly valid use of |
👍 I like the idea. Then we'd have
for people that want to use gRPC? @jgeewax Please weigh in |
Actually I think it's better for the extra to the the part of this library that requires extra dependencies. So IMO it should be |
But in the not-too-distant future |
Hmm. Fair point. I'm unsure. It seems that right now grpc doesn't have a good install story (unless that's recently changed). One part of me is leaning on having grpc remain an "extra" (in one way or another) until the install story improves or the tipping point where most clients in this library use grpc. @jgeewax what do you think? |
|
That's fair, I suppose. Are any os-level dependencies needed other than just |
No idea. @nathanielmanistaatgoogle do you know who might be able to chime in? |
I don't think even |
Cool, SGTM. |
I'm about to test it on bare GCE Debian and Ubuntu images. |
Seems to work on Ubuntu: $ sudo apt-get update
$ sudo apt-get -y upgrade
$ curl -O https://bootstrap.pypa.io/get-pip.py
$ sudo python get-pip.py
$ sudo -H pip install --upgrade pip
$ sudo -H pip install 'grpcio==0.13.0'
# FAILURE: https://gist.github.com/dhermes/f3cc291bb289ae69c86d
$ sudo -H pip install --upgrade six
$ sudo -H pip install 'grpcio==0.13.0'
$ # SUCCESS!
$ dpkg -s build-essential
dpkg-query: package 'build-essential' is not installed and no information is available
Use dpkg --info (= dpkg-deb --info) to examine archive files,
and dpkg --contents (= dpkg-deb --contents) to list their contents. |
Confirmed it works (with the same funky |
grpcio can only be installed with Python 2.7, so the py27 tox environment is the only one that doesn't need to run without mocks.
python2.7 | ||
deps = | ||
{[testenv]deps} | ||
grpcio >= 0.13.0 |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
@tseaver PTAL. Updated to make it an extra. |
LGTM |
Adding grpcio to setup.py as an extra.
grpc 0.13.0 is broken on UCS2 versions of python 2.7 (which you get when you build Python 2.x from source). We should drop testing with it in tox until they make a release with the fix. |
@tseaver can you file an issue? I'm AFK right now |
grpcio
can only be installed with Python 2.7, so thepy27
tox environment is the only one that doesn't need to run without mocks.FYI @jgeewax the
grpcio
install fails on Python 2.6 and 3.4 (didn't try 3.5).grpcio
but will at least install it. I tried to tackle but can't write faithful powershell without a test machineattempt_system_tests.py