Skip to content

Commit a1f75bb

Browse files
committed
chore: remove the installation of doxygen + graphviz from setup.sh
`doxygen` and `graphviz` are only required to generate the docs. Compiling PythonMonkey doesn't actually need them. The CI job to generate the docs (for https://docs.pythonmonkey.io/) will install doxygen on its own in https://github.com/Distributive-Network/PythonMonkey/blob/2633b5a/.github/workflows/test-and-publish.yaml#L168-L177. There's no point to install them automatically in `setup.sh`, if the user doesn't care about the docs.
1 parent 6631291 commit a1f75bb

File tree

1 file changed

+2
-9
lines changed

1 file changed

+2
-9
lines changed

setup.sh

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -13,18 +13,11 @@ if [[ "$OSTYPE" == "linux-gnu"* ]]; then # Linux
1313
SUDO='sudo'
1414
fi
1515
echo "Installing apt packages"
16-
$SUDO apt-get install --yes cmake graphviz llvm clang pkg-config m4 unzip \
16+
$SUDO apt-get install --yes cmake llvm clang pkg-config m4 unzip \
1717
wget curl python3-dev
18-
# Install Doxygen
19-
# the newest version in Ubuntu 20.04 repository is 1.8.17, but we need Doxygen 1.9 series
20-
echo "Installing doxygen"
21-
wget -c -q https://www.doxygen.nl/files/doxygen-1.9.7.linux.bin.tar.gz
22-
tar xf doxygen-1.9.7.linux.bin.tar.gz
23-
cd doxygen-1.9.7 && $SUDO make install && cd -
24-
rm -rf doxygen-1.9.7 doxygen-1.9.7.linux.bin.tar.gz
2518
elif [[ "$OSTYPE" == "darwin"* ]]; then # macOS
2619
brew update || true # allow failure
27-
brew install cmake doxygen pkg-config wget unzip coreutils # `coreutils` installs the `realpath` command
20+
brew install cmake pkg-config wget unzip coreutils # `coreutils` installs the `realpath` command
2821
elif [[ "$OSTYPE" == "msys"* ]]; then # Windows
2922
echo "Dependencies are not going to be installed automatically on Windows."
3023
else

0 commit comments

Comments
 (0)