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

ReadTheDocs broken by moving subpackages #2405

Closed
dhermes opened this issue Sep 23, 2016 · 10 comments
Closed

ReadTheDocs broken by moving subpackages #2405

dhermes opened this issue Sep 23, 2016 · 10 comments
Assignees

Comments

@dhermes
Copy link
Contributor

dhermes commented Sep 23, 2016

https://readthedocs.org/projects/google-cloud-python/builds/4436621/

@dhermes
Copy link
Contributor Author

dhermes commented Sep 26, 2016

  1. Succeeds
python -mvirtualenv \
  --no-site-packages \
  --no-download \
  .../google-cloud-python/envs/latest
  1. Succeeds
python .../pip install \
  --use-wheel \
  -U \
  --cache-dir .../google-cloud-python/.cache/pip \
  sphinx==1.3.5 Pygments==2.1.3 setuptools==20.1.1 docutils==0.12
  mkdocs==0.15.0 mock==1.0.1 pillow==2.6.1
  git+https://github.com/rtfd/readthedocs-sphinx-ext.git@0.6-alpha#egg=readthedocs-sphinx-ext
  sphinx-rtd-theme==0.1.9 alabaster>=0.7,<0.8,!=0.7.5
  commonmark==0.5.4 recommonmark==0.1.1
  1. Fails
python setup.py install --force

Maybe we can get away specifying dependencies with a git+... or with a relative directory in requirements.txt

@dhermes
Copy link
Contributor Author

dhermes commented Sep 27, 2016

Mostly fixed by #2444 and #2445. The remaining issue is that we depend on google-gax 0.14.1 in logging and pubsub but google-gax<0.14.0,>=0.13.0 is required by gapic-google-logging-v2 and gapic-google-pubsub-v1 (both 0.9.x).

@bjwatson hopes to have 0.10.0 of each released soon.

If the release doesn't happen by this afternoon, I'll just revert #2388 for the purposes of this release.

@dhermes
Copy link
Contributor Author

dhermes commented Sep 28, 2016

Blerg! https://readthedocs.org/projects/google-cloud-python/builds/4449511/ b0rken because

error: oauth2client 2.2.0 is installed but oauth2client<4.0.0dev,>=3.0.0
is required by
set(['gapic-google-logging-v2', 'grpc-google-logging-v2',
     'gapic-google-pubsub-v1', 'grpc-google-pubsub-v1'])

@dhermes
Copy link
Contributor Author

dhermes commented Sep 28, 2016

Starting to see some really strange RTD failures that seem to be different than the referenced HEAD commit (94a596e):

@dhermes
Copy link
Contributor Author

dhermes commented Sep 28, 2016

This at least shows that grpc-google-iam-v1 comes from gapic-google-pubsub-v1, so that is comforting. But then the oauth2client upper bound on that package contradicts the one from gapic-google-pubsub-v1.

@dhermes
Copy link
Contributor Author

dhermes commented Sep 28, 2016

@bjwatson
Copy link

@landrito Can you push an updated version of Python grpc-google-iam-v1 with the bounds currently in packman? (e.g. oauth2client >=3.0.0, <4.0.0)

@landrito
Copy link
Contributor

Will do!

@landrito
Copy link
Contributor

Pushed

@dhermes
Copy link
Contributor Author

dhermes commented Sep 29, 2016

Thanks!

@dhermes dhermes closed this as completed Sep 29, 2016
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

No branches or pull requests

3 participants