-
Notifications
You must be signed in to change notification settings - Fork 5.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[docs] Update docs for building from source #26045
Conversation
@@ -143,11 +143,20 @@ To build Ray on Ubuntu, run the following commands: | |||
.. code-block:: bash | |||
|
|||
sudo apt-get update | |||
sudo apt-get install -y build-essential curl unzip psmisc | |||
sudo apt-get install -y build-essential curl gcc-9 g++-9 pkg-config psmisc unzip | |||
sudo update-alternatives --install /usr/bin/gcc gcc /usr/bin/gcc-9 90 \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems this doesn't work for ubuntu 18.04 because of missing gcc 9 package.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
on ubuntu 18.04, you need add
sudo add-apt-repository -y ppa:ubuntu-toolchain-r/test
before sudo apt-get update
.
https://askubuntu.com/questions/1140183/install-gcc-9-on-ubuntu-18-04
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need support 18.04
friendly ping! |
I've pushed your comment but haven't had time to spin up 18.04 vm for manual testing (i'm using 22.04 these days) |
Thanks for the fix! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
stamp
The existing docs didn't work for me and these updates did. π€·ββοΈ I selectively pulled this stuff out of the CI (which ideally would just be runnable locally). Signed-off-by: Edward Oakes <[email protected]>
The existing docs didn't work for me and these updates did. π€·ββοΈ I selectively pulled this stuff out of the CI (which ideally would just be runnable locally). Signed-off-by: Nikita Vemuri <[email protected]>
The existing docs didn't work for me and these updates did. π€·ββοΈ I selectively pulled this stuff out of the CI (which ideally would just be runnable locally). Signed-off-by: Catch-Bull <[email protected]>
The existing docs didn't work for me and these updates did. π€·ββοΈ I selectively pulled this stuff out of the CI (which ideally would just be runnable locally). Signed-off-by: Catch-Bull <[email protected]>
The existing docs didn't work for me and these updates did. π€·ββοΈ I selectively pulled this stuff out of the CI (which ideally would just be runnable locally). Signed-off-by: klwuibm <[email protected]>
The existing docs didn't work for me and these updates did. π€·ββοΈ I selectively pulled this stuff out of the CI (which ideally would just be runnable locally). Signed-off-by: Frank Luan <[email protected]>
The existing docs didn't work for me and these updates did. π€·ββοΈ I selectively pulled this stuff out of the CI (which ideally would just be runnable locally). Signed-off-by: Scott Graham <[email protected]>
The existing docs didn't work for me and these updates did. π€·ββοΈ I selectively pulled this stuff out of the CI (which ideally would just be runnable locally).
Why are these changes needed?
The existing docs didn't work for me and these updates did. π€·ββοΈ I selectively pulled this stuff out of the CI (which ideally would just be runnable locally).
Related issue number
Checks
scripts/format.sh
to lint the changes in this PR.