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

Simplify the installation instructions. #1312

Open
H4CKY54CK opened this issue Feb 16, 2024 · 1 comment
Open

Simplify the installation instructions. #1312

H4CKY54CK opened this issue Feb 16, 2024 · 1 comment
Labels
area: deployment Issues in setting up qpixel, containers, etc. priority: medium type: change request New feature or request

Comments

@H4CKY54CK
Copy link

In the Arch-based section of the installation guide, the current instructions for pacman may work, but they could be improved. Namely, one should not be issuing pacman -Sy <package>, as it currently says to do. As currently written, it can lead to partial upgrades. The forced refresh at the top of the instructions is also questionable, but I can understand why it was listed.

I'm not that familiar with this project, so I can't say whether the multiple pacman commands is absolutely necessary or not. I propose two alternatives:

Install all the packages in one command, avoiding the possibility of dependency issues:

pacman -Syu gcc make ruby autoconf bison base-devel unixodbc openssl mariadb mysqld nodejs

Similarly, the Debian-based instructions could also be condensed down to a single command. Admittedly, avoiding multiple commands like this is just a preference to me, but it would simplify things.

Or, if it is necessary to install them separately and in the order they are listed, update the package database first, then use pacman -S to install what is needed:

sudo pacman -Syu
sudo pacman -S gcc
sudo pacman -S make
sudo pacman -S ruby autoconf bison base-devel unixodbc
sudo pacman -S openssl
sudo pacman -S mariadb mysqld nodejs

However, I am fairly sure this is not necessary.

I apologize if this issue is not formatted properly. This wasn't exactly a bug/feature request.

@cellio cellio added type: change request New feature or request priority: medium area: deployment Issues in setting up qpixel, containers, etc. labels Feb 16, 2024
@cellio
Copy link
Member

cellio commented Feb 16, 2024

Thanks for filing this, and the formatting is just fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: deployment Issues in setting up qpixel, containers, etc. priority: medium type: change request New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants