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

correction README.md #1140

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ArimaBatista
Copy link

Correcting the dependency name from libsecret-1.0 to libsecret-1-dev, located in the "'Setting up & running the development version >>> Dependencies >>> You can get most of those from your distribution packages: >>> # On Debian/Ubuntu:' section."

Correcting the dependency name from libsecret-1.0 to libsecret-1-dev, located in the "'Setting up & running the development version >>> Dependencies >>> You can get most of those from your distribution packages: >>> # On Debian/Ubuntu:' section."
@Neui
Copy link
Contributor

Neui commented Sep 21, 2024

Why? From what I can see, it adds nothing for us (unless mesen wants the pkg-config files?). In fact, we could remove libsecret-1.0 because gir1.2-secret-1 already depends on it (according to Ubuntu packages).

(Also the PR title and commit message are super generic, something like "README: fix debian/ubuntu libsecret dependency" is way more descriptive, along with the why in the message body.)

@diegogangl
Copy link
Contributor

Definitely, if gir1.2-secret-1 is already going to drag libsecret-1 then we should get rid of that package in the list

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