apeforest commented on a change in pull request #14534: [DOC] Update ubuntu
install instructions from source
URL: https://github.com/apache/incubator-mxnet/pull/14534#discussion_r273337035
##########
File path: docs/install/ubuntu_setup.md
##########
@@ -137,12 +137,33 @@ It is recommended that you review the general [build
from source](build_from_sou
On Ubuntu versions 16.04 or later, you need the following dependencies:
-**Step 1:** Install build tools and git.
+**Step 1:** Install prerequisite packages.
```bash
sudo apt-get update
- sudo apt-get install -y build-essential git
+ sudo apt-get install -y build-essential git ninja-build ccache
Review comment:
Again, even though ccache is good, I am not in favor of adding something
optional as a prerequisite for installation. But I am open to other opinions.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services