Skip to content

Commit 138ab77

Browse files
tossy310vmcj
authored andcommitted
Move venv activate line to make it clear what needs it
1 parent cc9ad12 commit 138ab77

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

docker/domserver/build.sh

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,5 @@
11
#!/bin/sh -eu
22

3-
# Use venv to use latest Sphinx. 6.1.0 or higher is required to build DOMjudge docs.
4-
# shellcheck source=/dev/null
5-
. /venv/bin/activate
6-
73
cd /domjudge-src/domjudge*
84
chown -R domjudge: .
95
# If we used a local source tarball, it might not have been built yet
@@ -30,4 +26,7 @@ then
3026
fi
3127

3228
sudo -u domjudge sh -c '. /venv/bin/activate && make docs'
29+
# Use Python venv to use the latest Sphinx to build DOMjudge docs.
30+
# shellcheck source=/dev/null
31+
. /venv/bin/activate
3332
make install-docs

0 commit comments

Comments
 (0)