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

Add installation instructions for more Linux distributions in README #1492

Merged
merged 3 commits into from Apr 29, 2024

Conversation

AlbydST
Copy link
Contributor

@AlbydST AlbydST commented Feb 11, 2024

Added installation instructions for Arch AUR, ALT Linux and Solus, added a disclaimer to replace sudo with doas in commands depending on what you use and made a few minor changes.

Added installation instructions for Arch AUR, ALT Linux and Solus and did a few minor changes
@tavianator
Copy link
Collaborator

I don't think adding sudo to the front of most commands is worth it. Users typically know whether their package manager needs to run as root. There is a convention to use the prompt ($/#) to indicate regular user/root, e.g.

# apt install fd

README.md Outdated
@@ -668,6 +684,8 @@ Note that rust version *1.70.0* or later is required.

`make` is also needed for the build.

Make sure to replace **sudo** with **doas** in commands depending on what you use.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is also unnecessary. If you use linux and wen to the effort to install and set up doas, and use that instead of sudo, then you probably already know that you should change the commands to use doas instead of sudo.

Removed unnecessary "sudo" in front of all commands, also removed the "replace sudo with doas..." instruction at the end.
@tmccombs tmccombs merged commit c4094c7 into sharkdp:master Apr 29, 2024
17 checks passed
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