From f52fb410a504b1a0fd69d26d7f86ba21d86b0cf3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Antoine=20Beaupr=C3=A9?= Date: Fri, 13 Nov 2015 10:31:56 -0500 Subject: [PATCH] Revert "use build_py to fix build on RTD", disables docs build This reverts commit 86487d192a9a5ab7ff4eedb92d793485b4c30268. We will instead commit the generated `.rst` usage and API files directly into git. The setup commands remain to generate them when the usage or API changes, but as things are the hoops required to generate those RST files are way too complicated to justify simply build docs. See #384. Conflicts: setup.py --- setup.py | 31 ------------------------------- 1 file changed, 31 deletions(-) diff --git a/setup.py b/setup.py index 067959389..5c6e98a2f 100644 --- a/setup.py +++ b/setup.py @@ -6,8 +6,6 @@ from glob import glob from distutils.command.build import build from distutils.core import Command -from distutils import log -from setuptools.command.build_py import build_py min_python = (3, 2) my_python = sys.version_info @@ -203,39 +201,10 @@ API Documentation """ % mod) -class build_py_custom(build_py): - """override build_py to also build our stuff - - it is unclear why this is necessary, but in some environments - (Readthedocs.org, specifically), the above - ``build.sub_commands.append()`` doesn't seem to have an effect: - our custom build commands seem to be ignored when running - ``setup.py install``. - - This class overrides the ``build_py`` target by forcing it to run - our custom steps as well. - - See also the `bug report on RTD - `_. - """ - def run(self): - super().run() - self.announce('calling custom build steps', level=log.INFO) - self.run_command('build_ext') - if on_rtd: - # only build these files if running on readthedocs, but not - # for a normal production install. It requires "mock" and we - # do not have that as a build dependency. Also, for really - # building the docs, it would also require sphinx, etc. - self.run_command('build_api') - self.run_command('build_usage') - - cmdclass = { 'build_ext': build_ext, 'build_api': build_api, 'build_usage': build_usage, - 'build_py': build_py_custom, 'sdist': Sdist }