Merge lp:~widelands-dev/widelands/fix-sphinx into lp:widelands

Proposed by GunChleoc
Status: Merged
Merged at revision: 8829
Proposed branch: lp:~widelands-dev/widelands/fix-sphinx
Merge into: lp:widelands
Diff against target: 33 lines (+12/-11)
1 file modified
.travis.sh (+12/-11)
To merge this branch: bzr merge lp:~widelands-dev/widelands/fix-sphinx
Reviewer Review Type Date Requested Status
hessenfarmer Approve
Review via email: mp+354827@code.launchpad.net

Commit message

Remove sphinx from debug builds on Travis.

Description of the change

We need our Travis back ASAP, so I have uncommented the Sphinx build for now. I can't get my system to cooperate and use the current version of Sphinx, so I can't reproduce the error.

To post a comment you must log in.
Revision history for this message
GunChleoc (gunchleoc) wrote :

@bunnybot merge force

Revision history for this message
hessenfarmer (stephan-lutz) wrote :

weird thing is that according to the Travis console it works with sphinx 1.8.0 in OSX but not in Linux. So we could have left the OSX part in but I think it doesn't hurt to have this deactivated.

Revision history for this message
hessenfarmer (stephan-lutz) :
review: Approve
Revision history for this message
bunnybot (widelandsofficial) wrote :

Continuous integration builds have changed state:

Travis build 3959. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/428007379.
Appveyor build 3757. State: failed. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_fix_sphinx-3757.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file '.travis.sh'
2--- .travis.sh 2017-11-27 20:16:11 +0000
3+++ .travis.sh 2018-09-13 05:06:42 +0000
4@@ -30,17 +30,18 @@
5 if [ "$BUILD_TYPE" == "Debug" ]; then
6
7 # Build the documentation. Any warning is an error.
8- if [[ "$TRAVIS_OS_NAME" == "linux" ]]; then
9- sudo pip install sphinx
10- fi
11- if [[ "$TRAVIS_OS_NAME" == "osx" ]]; then
12- pip2 install sphinx
13- fi
14- pushd ../doc/sphinx
15- mkdir source/_static
16- ./extract_rst.py
17- sphinx-build -W -b json -d build/doctrees source build/json
18- popd
19+ # TODO: GunChleoc: Currently broken
20+ #if [[ "$TRAVIS_OS_NAME" == "linux" ]]; then
21+ # sudo pip install sphinx
22+ #fi
23+ #if [[ "$TRAVIS_OS_NAME" == "osx" ]]; then
24+ # pip2 install sphinx
25+ #fi
26+ #pushd ../doc/sphinx
27+ #mkdir source/_static
28+ #./extract_rst.py
29+ #sphinx-build -W -b json -d build/doctrees source build/json
30+ #popd
31
32 # Run the codecheck test suite.
33 pushd ../cmake/codecheck

Subscribers

People subscribed via source and target branches

to status/vote changes: