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

README.md updated with prerequisites for building from source code #393

Conversation

idelac3
Copy link

@idelac3 idelac3 commented Dec 24, 2024

Please include the following oneliner

apt install autoconf autoconf-archive pkg-config libimlib2 libimlib2-dev libxcomposite1 libxcomposite-dev libxext6 libxext-dev libxfixes3 libxfixes-dev libxinerama1 libxinerama-dev

under dependencies section, in the README.md file.

@N-R-K
Copy link
Collaborator

N-R-K commented Dec 24, 2024

If you're an end user you should just be able to do apt install scrot (i think). For developers, we have some instruction in CONTRIBUTING.md already:

scrot/CONTRIBUTING.md

Lines 10 to 13 in 5ed4a99

You will want to install the dependencies listed in the `Building` section of
[README.md](README.md). Look for the text `install_script` in the
[.cirrus.yml](.cirrus.yml) CI definitions for example package manager
invocations on many operating systems.

And the mentioned .cirrus.yml file contains dependency list for more than just debian.

Personally, I don't see any reason to special case debian here. Especially since different distros have different naming/packing conventions and so it's not necessarily transferable across distros.

@N-R-K N-R-K closed this Dec 28, 2024
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.

2 participants