Perform 'blurb split' for master.#2719
Conversation
brettcannon
left a comment
There was a problem hiding this comment.
A quick perusal of the output LGTM.
Sorry, something went wrong.
|
It looks like Sphinx snuck in a new deprecation warning in Sphinx 1.6.3 which this is triggering. If you can easily fix it by replacing any use of |
Sorry, something went wrong.
|
And I think the culprit might be , but that's just from a quick search for [Directive]. |
Sorry, something went wrong.
|
LIke Brett, I didn't thoroughly review the changes but, in general, LGTM. One concern: as is, making this change will immediately break all doc builds of |
Sorry, something went wrong.
|
I tried to restart the Travis doc build but it is now getting a |
Sorry, something went wrong.
|
So... make suspicious is itself suspicious? GREAT SUCCESS I experimented with having This does make the Doc build process dependent on blurb, which I hadn't anticipated. But that's a natural enough outcome of the process. And we can easily update the |
Sorry, something went wrong.
|
@brettcannon: It looks to me like Travis built against an old version. Did I hit some sort of workflow bug? I changed The build didn't work because I had to fix What's strange is that on Travis's "requests" dashboard: https://travis-ci.org/python/cpython/requests What's going on? I may try to fool Travis into running another build by uploading a no-op revision. |
Sorry, something went wrong.
|
The docs job fails on Travis CI because this job only clones the last 50 commits: whereas blurb looks for a specific commit: I don't think that we can change how Travis CI clones a Git repository, so we should change how blurb checks if if's running in a Git repository. Maybe it should catch subprocess.SubprocessError on: To check if it's running in Git or not? |
Sorry, something went wrong.
vstinner
left a comment
There was a problem hiding this comment.
LGTM.
Sorry, something went wrong.
|
Looks like @Haypo spotted why the revision check is failing. A bit annoying but understandable since Travis doesn't want to pay the CPU/network costs of downloading the entire CPython repo. As for having the build process depend on blurb, that's just life and I have no issue with it since we already depend on sphinx. If it becomes a really serious problem we can talk about just putting blurb into the cpython repo to essentially vendor blurb in Python itself. |
Sorry, something went wrong.
ned-deily
left a comment
There was a problem hiding this comment.
Not surprisingly, trying to add blurb into the doc build has twisty little passages. I ran into two major problems reviewing and testing this:
- At the moment, the most recent version of blurb that is published on PyPI and the first one that works as expected was published as
1.0.2.dev1. Because it's adevversion, by default pip won't try to install it and falls back to the last "production" release1.0.1.post1which has bugs. I think the best solution is to specify a version >=1.0.2dev1 with Travis (not tested) and pip. - Installing blurb into the venv produced by
make venvis good except that the rest of the Doc Makefile does not use thevenvby default. The (undocumented, other than in the original checkin message d5ea39d) intended use was to then explicitly pass the venv Python into Doc makefile targets by setting the PYTHON variable, in part, because trying to use venvactivatein Makefile recipes can be problematic. But even then, theblurbcommand isn't found because using PYTHON= doesn't put the venv bin directory (where theblurbscript is installed) on the processPATH. I've suggested one way around it which is to invokeblurbvia$(PYTHON) -m blurb; that way, the lack ofblurbsort of fails the same way the lack ofsphinx-buildwill. It might be nice to fail with a more informative message. I think it mandatory to document thevenvtarget inDoc/README.rstand something about the need to usePYTHON=venv/bin/pythonwhen usingvenv. BTW, it appears that the 2.7 Doc/Makefile doesn't have avenvtarget so that will have to be backported for the 2.7 PR.
Sorry, something went wrong.
|
I suspect @larryhastings did a release on Sunday of blurb and simply forgot to bump the version number to 1.0.2 (i.e. drop the |
Sorry, something went wrong.
|
Booyah! We now pass CI testing in a post-blurb world. @ned-deily, care to review again? |
Sorry, something went wrong.
ned-deily
left a comment
There was a problem hiding this comment.
Thanks for making the changes. Other than the spurious change noted, I still think we need to document how to use make venv and the need to add PYTHON=venv/bin/python to the doc make steps. And I need to update the mac installer build accordingly.
Sorry, something went wrong.
|
@ned-deily where are you suggesting to document the use of |
Sorry, something went wrong.
|
@brettcannon The primary documentation for Doc builds is |
Sorry, something went wrong.
|
I created GH-2953 to document |
Sorry, something went wrong.
No description provided.