publish docs the new way

Review Request #1292 - Created Nov. 6, 2014 and submitted

Information
Larry Hosken
pants
a8b1495...
Reviewers
pants-reviews
jinfeng, jsirois, stuhood

as part of cleaning up Sphinx-things, I touched things I understood and things I halfway-understood.

yay, understood:
mv publish_docs_new.sh publish_docs.sh ; rm src/python/pants/docs/*.rst , other Sphinx things

the parts I only halfway-understood, listed in order of rising scariness:

rm'd sphinx things from 3rdparty/python/dev_only_requirements.txt . which was all of it so...
rm'd 3rdparty/python/dev_only_requirements.txt
rm'd DEV_REQUIREMENTS from pants_venv
rm'd reference to DEV_REQUIREMENTS from ./pants , use REQUIREMENTS instead.

Silver lining: reviewboard lists the scary changes first.

I hoped to get this in front of JSirois before he went away for a couple of weeks, but alas. If reviewer[s] want me to revert the pants_venv and ./pants changes until he gets back, I won't be offended; I think he's the only one who's touched that code.

(Something I didn't do in this change: With this change, the only thing that mentions build-support/pants_venv is ./pants. It ?might? make sense to fold pants_venv into pants? Or maybe folks use pants_venv ad-hoc?)

https://travis-ci.org/pantsbuild/pants/builds/40938680

./build-support/bin/publish_docs.sh -opd sitegen-test

for the ./pants change:
export WRAPPER_REQUIREMENTS="tmp_requirements.txt" # file with one line: scrapy
in some temporarily-added pants code, from scrapy import Spider, Item, Field
worked OK. (and correctly errored out in another shell that didn't have WRAPPER_REQUIREMENTS set)

Issues

  • 1
  • 0
  • 0
  • 1
Description From Last Updated
you probabaly wanna merge, those two are now in common.sh Jin Feng Jin Feng
Larry Hosken
Larry Hosken
Larry Hosken
Larry Hosken
Larry Hosken
Larry Hosken
Jin Feng
Larry Hosken
Larry Hosken
Jin Feng
Larry Hosken
Review request changed

Status: Closed (submitted)

Loading...