Skip to content

Conversation

@metacritical
Copy link
Contributor

Adds additional installation instructions for Linux Debian/Ubuntu/Redhat as wells as macOS. and resolving mac path issues.

Adds additional installation instructions for Linux Debian/Ubuntu/Redhat as wells as macOS.  and resolving mac path issues.
Comment on lines +22 to +23
sudo apt-get update
sudo apt-get install libgdbm-dev
Copy link
Member

@junaruga junaruga Oct 4, 2023

Choose a reason for hiding this comment

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

Suggested change
sudo apt-get update
sudo apt-get install libgdbm-dev
sudo apt update
sudo apt install libgdbm-dev

I think the apt is better than apt-get. You can see https://itsfoss.com/apt-vs-apt-get-difference/ for the context.

Comment on lines +26 to +31
### On Redhat/Fedora Yum or DNF
```
sudo yum install gdbm-devel
sudo dnf install gdbm-devel
```
Copy link
Member

@junaruga junaruga Oct 4, 2023

Choose a reason for hiding this comment

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

Aligning with the "On Debian/Ubuntu" (upstream/downstream distro) above, I would suggest "On Fedora/Red Hat Enterprise Linux DNF or YUM" or "On Fedora/RHEL DNF or YUM" (upstream/downstream distro).

The OS name is RHEL. It's different from old outdated OS, Red Hat Linux.

Nowadays the yum is just alias for the dnf. The dnf is available in RHEL 8 and later versions.

$ cat /etc/fedora-release
Fedora release 38 (Thirty Eight)

$ ls -l /bin/yum
lrwxrwxrwx. 1 root root 5 Jul 27 02:00 /bin/yum -> dnf-3*

So, I would suggest the dnf as first, and yum as second.

sudo dnf install gdbm-devel

or

sudo yum install gdbm-devel


## Installation


Copy link
Member

Choose a reason for hiding this comment

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

Can you remove changes of L40-L43?

@hsbt hsbt merged commit 07b2a2a into ruby:master Jul 9, 2025
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.

3 participants