Skip to content

Commit 3a921dc

Browse files
committed
Make Sphinx to share environment between commands
Remove the `-E` from Sphinx command so it can shares the saved environment between commands from the same build. This will speed up a little those builds that are building multiple formats. Closes #10701
1 parent dcd4655 commit 3a921dc

File tree

2 files changed

+0
-11
lines changed

2 files changed

+0
-11
lines changed

readthedocs/doc_builder/backends/sphinx.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,6 @@ def build(self):
295295
build_command = [
296296
*self.get_sphinx_cmd(),
297297
"-T",
298-
"-E",
299298
]
300299
if self.config.sphinx.fail_on_warning:
301300
build_command.extend(["-W", "--keep-going"])
@@ -483,7 +482,6 @@ def build(self):
483482
self.run(
484483
*self.get_sphinx_cmd(),
485484
"-T",
486-
"-E",
487485
"-b",
488486
self.sphinx_builder,
489487
"-d",

readthedocs/projects/tests/test_build_tasks.py

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,6 @@ def test_build_sphinx_formats(self, load_yaml_config, formats, builders):
191191
"-m",
192192
"sphinx",
193193
"-T",
194-
"-E",
195194
"-b",
196195
"html",
197196
"-d",
@@ -212,7 +211,6 @@ def test_build_sphinx_formats(self, load_yaml_config, formats, builders):
212211
"-m",
213212
"sphinx",
214213
"-T",
215-
"-E",
216214
"-b",
217215
builder,
218216
"-d",
@@ -829,7 +827,6 @@ def test_build_commands_executed(
829827
"-m",
830828
"sphinx",
831829
"-T",
832-
"-E",
833830
"-b",
834831
"html",
835832
"-d",
@@ -846,7 +843,6 @@ def test_build_commands_executed(
846843
"-m",
847844
"sphinx",
848845
"-T",
849-
"-E",
850846
"-b",
851847
"readthedocssinglehtmllocalmedia",
852848
"-d",
@@ -891,7 +887,6 @@ def test_build_commands_executed(
891887
"-m",
892888
"sphinx",
893889
"-T",
894-
"-E",
895890
"-b",
896891
"latex",
897892
"-d",
@@ -911,7 +906,6 @@ def test_build_commands_executed(
911906
"-m",
912907
"sphinx",
913908
"-T",
914-
"-E",
915909
"-b",
916910
"epub",
917911
"-d",
@@ -1467,7 +1461,6 @@ def test_sphinx_normalized_language(self, load_yaml_config):
14671461
"-m",
14681462
"sphinx",
14691463
"-T",
1470-
"-E",
14711464
"-W", # fail on warning flag
14721465
"--keep-going", # fail on warning flag
14731466
"-b",
@@ -1506,7 +1499,6 @@ def test_sphinx_fail_on_warning(self, load_yaml_config):
15061499
"-m",
15071500
"sphinx",
15081501
"-T",
1509-
"-E",
15101502
"-W", # fail on warning flag
15111503
"--keep-going", # fail on warning flag
15121504
"-b",
@@ -1843,7 +1835,6 @@ def test_sphinx_builder(self, load_yaml_config, value, command):
18431835
"-m",
18441836
"sphinx",
18451837
"-T",
1846-
"-E",
18471838
"-b",
18481839
command,
18491840
"-d",

0 commit comments

Comments
 (0)