Skip to content
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

Minor changes for node installation on Linux #417

Merged
merged 2 commits into from
Nov 24, 2021
Merged

Minor changes for node installation on Linux #417

merged 2 commits into from
Nov 24, 2021

Conversation

os11k
Copy link
Collaborator

@os11k os11k commented Nov 22, 2021

Minor changes for node installation on Linux. Updated link to latest binaries, added apt-get upgrade and most important added details what to enter when installing ghcup and some other minor cosmetic changes

Updating documentation

Issue

Description of the change

Minor changes for Installing cardano-node and cardano-cli from source on Linux. Updated link to latest binaries, added apt-get upgrade and most important added details what to enter when installing ghcup and some other minor cosmetic changes

Minor changes for node installation on Linux. Updated link to latest binaries, added apt-get upgrade and most important added details what to enter when installing ghcup and some other minor cosmetic changes
@@ -83,7 +83,13 @@ Use the following command to install `ghcup`
```bash
curl --proto '=https' --tlsv1.2 -sSf https://get-ghcup.haskell.org | sh
```
Please follow the instructions and provide the necessary input to the installer. Once complete, you should have `ghc` and `cabal` installed to your system.
Please follow the instructions and provide the necessary input to the installer.\
`Do you want ghcup to automatically add the required PATH variable to "/root/.bashrc"?` - (P or enter)\
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd never recommend use of root

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, I will update it, in my defense, I was just testing this on my test VM, what is already destroyed. :)

Updated home directory of our user as requested by @rdlrt
Copy link
Member

@katomm katomm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @os11k

@katomm katomm merged commit 4e99eca into cardano-foundation:staging Nov 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants