Merge lp:~jelmer/brz/sphinx-only into lp:brz

Proposed by Jelmer Vernooij
Status: Merged
Approved by: Jelmer Vernooij
Approved revision: no longer in the source branch.
Merge reported by: The Breezy Bot
Merged at revision: not available
Proposed branch: lp:~jelmer/brz/sphinx-only
Merge into: lp:brz
Diff against target: 207 lines (+14/-141)
4 files modified
.travis.yml (+1/-1)
Makefile (+5/-132)
breezy/doc_generate/conf.py (+7/-7)
byov.conf (+1/-1)
To merge this branch: bzr merge lp:~jelmer/brz/sphinx-only
Reviewer Review Type Date Requested Status
Martin Packman Approve
Review via email: mp+358957@code.launchpad.net

Commit message

Require sphinx for docs building, remove docutils.

Description of the change

Require sphinx for docs building, remove docutils.

This was originally proposed by Ian in https://code.launchpad.net/~ian-clatworthy/bzr/sphinx-only/+merge/26011, but put on hold then because sphinx wasn't widely available.

To post a comment you must log in.
Revision history for this message
Martin Packman (gz) wrote :

Thanks, and retrospective thanks to Ian.

review: Approve
Revision history for this message
The Breezy Bot (the-breezy-bot) wrote :
Revision history for this message
The Breezy Bot (the-breezy-bot) wrote :
Revision history for this message
The Breezy Bot (the-breezy-bot) wrote :
Revision history for this message
The Breezy Bot (the-breezy-bot) wrote :

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file '.travis.yml'
2--- .travis.yml 2018-11-16 15:32:10 +0000
3+++ .travis.yml 2018-11-18 19:07:41 +0000
4@@ -22,7 +22,7 @@
5 install:
6 - sudo apt install python-all-dev python3-all-dev subunit
7 - travis_retry pip install -U setuptools
8- - travis_retry pip install -U pip coverage codecov flake8 testtools paramiko fastimport configobj cython testscenarios six docutils python-subunit dulwich $TEST_REQUIRE
9+ - travis_retry pip install -U pip coverage codecov flake8 testtools paramiko fastimport configobj cython testscenarios six docutils python-subunit dulwich $TEST_REQUIRE sphinx sphinx_epytext
10
11 after_success:
12 - codecov
13
14=== modified file 'Makefile'
15--- Makefile 2018-11-18 16:26:08 +0000
16+++ Makefile 2018-11-18 19:07:41 +0000
17@@ -118,14 +118,11 @@
18
19 ### Documentation ###
20
21-# Default to plain documentation for maximum backwards compatibility.
22-# (Post 2.0, the defaults will most likely be Sphinx-style instead.)
23-
24-docs: docs-plain
25-
26-clean-docs: clean-plain
27-
28-html-docs: html-plain
29+docs: docs-sphinx
30+
31+clean-docs: clean-sphinx
32+
33+html-docs: html-sphinx
34
35
36 ### Man-page Documentation ###
37@@ -216,130 +213,6 @@
38 $(PYTHON) tools/package_docs.py doc/developers $(DOC_WEBSITE_BUILD)
39
40
41-### Plain Documentation ###
42-
43-# While Sphinx is the preferred tool for building documentation, we still
44-# support our "plain" html documentation so that Sphinx is not a hard
45-# dependency for packagers on older platforms.
46-
47-rst2html = $(PYTHON) tools/rst2html.py --link-stylesheet --footnote-references=superscript --halt=warning
48-
49-# translate txt docs to html
50-derived_txt_files = \
51- doc/en/release-notes/NEWS.txt
52-txt_all = \
53- doc/en/tutorials/tutorial.txt \
54- doc/en/tutorials/using_breezy_with_launchpad.txt \
55- doc/en/tutorials/centralized_workflow.txt \
56- $(call sw,doc/*/mini-tutorial/index.txt) \
57- $(call sw,doc/*/user-guide/index-plain.txt) \
58- doc/en/admin-guide/index-plain.txt \
59- $(call sw,doc/es/guia-usario/*.txt) \
60- $(derived_txt_files) \
61- doc/en/upgrade-guide/index.txt \
62- doc/index.txt \
63- $(call sw,doc/index.*.txt)
64-txt_nohtml = \
65- doc/en/user-guide/index.txt \
66- doc/en/admin-guide/index.txt
67-txt_files = $(filter-out $(txt_nohtml), $(txt_all))
68-htm_files = $(patsubst %.txt, %.html, $(txt_files))
69-
70-non_txt_files = \
71- doc/default.css \
72- $(call sw,doc/*/brz-en-quick-reference.svg) \
73- $(call sw,doc/*/brz-en-quick-reference.png) \
74- $(call sw,doc/*/brz-en-quick-reference.pdf) \
75- $(call sw,doc/*/bzr-es-quick-reference.svg) \
76- $(call sw,doc/*/bzr-es-quick-reference.png) \
77- $(call sw,doc/*/bzr-es-quick-reference.pdf) \
78- $(call sw,doc/*/bzr-ru-quick-reference.svg) \
79- $(call sw,doc/*/bzr-ru-quick-reference.png) \
80- $(call sw,doc/*/bzr-ru-quick-reference.pdf) \
81- $(call sw,doc/*/user-guide/images/*.png)
82-
83-# doc/developers/*.txt files that should *not* be individually
84-# converted to HTML
85-dev_txt_nohtml = \
86- doc/developers/add.txt \
87- doc/developers/annotate.txt \
88- doc/developers/bundle-creation.txt \
89- doc/developers/commit.txt \
90- doc/developers/diff.txt \
91- doc/developers/directory-fingerprints.txt \
92- doc/developers/gc.txt \
93- doc/developers/implementation-notes.txt \
94- doc/developers/incremental-push-pull.txt \
95- doc/developers/index.txt \
96- doc/developers/initial-push-pull.txt \
97- doc/developers/merge-scaling.txt \
98- doc/developers/miscellaneous-notes.txt \
99- doc/developers/missing.txt \
100- doc/developers/performance-roadmap-rationale.txt \
101- doc/developers/performance-use-case-analysis.txt \
102- doc/developers/planned-change-integration.txt \
103- doc/developers/planned-performance-changes.txt \
104- doc/developers/plans.txt \
105- doc/developers/process.txt \
106- doc/developers/revert.txt \
107- doc/developers/specifications.txt \
108- doc/developers/status.txt \
109- doc/developers/uncommit.txt
110-
111-dev_txt_all = $(call sw,$(addsuffix /*.txt, doc/developers))
112-dev_txt_files = $(filter-out $(dev_txt_nohtml), $(dev_txt_all))
113-dev_htm_files = $(patsubst %.txt, %.html, $(dev_txt_files))
114-
115-doc/en/user-guide/index-plain.html: $(call sw,$(addsuffix /*.txt, doc/en/user-guide))
116- $(rst2html) --stylesheet=../../default.css $(dir $@)index-plain.txt $@
117-
118-#doc/es/user-guide/index.html: $(call sw,$(addsuffix /*.txt, doc/es/user-guide))
119-# $(rst2html) --stylesheet=../../default.css $(dir $@)index.txt $@
120-#
121-#doc/ru/user-guide/index.html: $(call sw,$(addsuffix /*.txt, doc/ru/user-guide))
122-# $(rst2html) --stylesheet=../../default.css $(dir $@)index.txt $@
123-#
124-doc/en/admin-guide/index-plain.html: $(call sw,$(addsuffix /*.txt, doc/en/admin-guide))
125- $(rst2html) --stylesheet=../../default.css $(dir $@)index-plain.txt $@
126-
127-doc/developers/%.html: doc/developers/%.txt
128- $(rst2html) --stylesheet=../default.css $< $@
129-
130-doc/index.html: doc/index.txt
131- $(rst2html) --stylesheet=default.css $< $@
132-
133-doc/index.%.html: doc/index.%.txt
134- $(rst2html) --stylesheet=default.css $< $@
135-
136-%.html: %.txt
137- $(rst2html) --stylesheet=../../default.css $< "$@"
138-
139-doc/en/release-notes/NEWS.txt: $(NEWS_FILES) tools/generate_release_notes.py
140- $(PYTHON) tools/generate_release_notes.py "$@" $(NEWS_FILES)
141-
142-upgrade_guide_dependencies = $(call sw,$(addsuffix /*.txt, doc/en/upgrade-guide))
143-
144-doc/en/upgrade-guide/index.html: $(upgrade_guide_dependencies)
145- $(rst2html) --stylesheet=../../default.css $(dir $@)index.txt $@
146-
147-derived_web_docs = $(htm_files) $(dev_htm_files)
148-WEB_DOCS = $(derived_web_docs) $(non_txt_files)
149-ALL_DOCS = $(derived_web_docs) $(MAN_PAGES)
150-
151-# the main target to build all the docs
152-docs-plain: $(ALL_DOCS)
153-
154-# produce a tree containing just the final docs, ready for uploading to the web
155-HTMLDIR = html_docs
156-html-plain: docs-plain
157- $(PYTHON) tools/win32/ostools.py copytree $(WEB_DOCS) $(HTMLDIR)
158-
159-# clean produced docs
160-clean-plain:
161- $(PYTHON) tools/win32/ostools.py remove $(ALL_DOCS) \
162- $(HTMLDIR) $(derived_txt_files)
163-
164-
165 ### Miscellaneous Documentation Targets ###
166
167 # build a png of our performance task list
168
169=== modified file 'breezy/doc_generate/conf.py'
170--- breezy/doc_generate/conf.py 2018-11-18 14:23:08 +0000
171+++ breezy/doc_generate/conf.py 2018-11-18 19:07:41 +0000
172@@ -113,15 +113,15 @@
173 # * blue: darkblue, mediumblue, darkslateblue, cornflowerblue, royalblue,
174 # midnightblue
175 # * gray: dimgray, slategray, lightslategray
176- 'sidebarbgcolor': "cornflowerblue",
177- 'sidebarlinkcolor': "midnightblue",
178- 'relbarbgcolor': "darkblue",
179- 'footerbgcolor': "lightslategray",
180+ #'sidebarbgcolor': "cornflowerblue",
181+ #'sidebarlinkcolor': "midnightblue",
182+ #'relbarbgcolor': "darkblue",
183+ #'footerbgcolor': "lightslategray",
184
185 # Text, heading and code colouring
186- 'codebgcolor': "lightyellow",
187- 'codetextcolor': "firebrick",
188- 'linkcolor': "mediumblue",
189+ #'codebgcolor': "lightyellow",
190+ #'codetextcolor': "firebrick",
191+ #'linkcolor': "mediumblue",
192 }
193
194 # Add any paths that contain custom themes here, relative to this directory.
195
196=== modified file 'byov.conf'
197--- byov.conf 2018-11-18 14:27:52 +0000
198+++ byov.conf 2018-11-18 19:07:41 +0000
199@@ -25,7 +25,7 @@
200
201 # FIXME: Arguably this should be vm.build_deps=brz but it requires either an
202 # available package or at least a debian/ dir ? -- vila 2018-02-23
203-brz.build_deps = gcc, debhelper, python, python-all-dev, python3-all-dev, python-configobj, python3-configobj, python-docutils, python3-docutils, python-paramiko, python3-paramiko, python-subunit, python3-subunit, python-testtools, python3-testtools, subunit, cython, cython3, python-pip, python3-pip, python-setuptools, python3-setuptools
204+brz.build_deps = gcc, debhelper, python, python-all-dev, python3-all-dev, python-configobj, python3-configobj, python-docutils, python3-docutils, python-paramiko, python3-paramiko, python-subunit, python3-subunit, python-testtools, python3-testtools, subunit, cython, cython3, python-pip, python3-pip, python-setuptools, python3-setuptools, python-flake8, python3-flake8, python-sphinx, python3-sphinx
205 subunit.build_deps = python-testscenarios, python3-testscenarios, python-testtools, python3-testtools
206 vm.packages = {brz.build_deps}, {subunit.build_deps}, bzr, git, python-junitxml
207 [brz-xenial]

Subscribers

People subscribed via source and target branches