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 dependencies for Arch Linux #8998

Merged
merged 3 commits into from
Oct 1, 2019
Merged

Conversation

ValkyrieStella
Copy link
Contributor

This pull request adds Arch Linux dependencies (including base-devel). Arch Linux is quite popular, especially among Minetest players, as are derivatives such as Manjaro.

To do

This PR is Ready for Review.

Added Arch Linux dependency list (including base-devel which covers many of them)
README.md Outdated Show resolved Hide resolved
@SmallJoker SmallJoker added the @ Documentation Improvements or additions to documentation label Sep 29, 2019
@sfan5
Copy link
Member

sfan5 commented Sep 29, 2019

gnucurl

this does not exist

vorbis-tools

not needed, libvorbis + libogg is enough

png++ glu mesa libspatialite

we don't use any of these

doxygen

most users won't need this so I'd leave it off the list

@ValkyrieStella
Copy link
Contributor Author

I meant gnurl instead of gnucurl, my bad

@ValkyrieStella
Copy link
Contributor Author

updated deps

@sfan5
Copy link
Member

sfan5 commented Sep 29, 2019

Minetest does not use gnurl either.

@ValkyrieStella
Copy link
Contributor Author

I believe it replaces the standard libcurl

@sfan5
Copy link
Member

sfan5 commented Sep 29, 2019

It does not, but even if it did: Why is the standard libcurl not okay?

@ValkyrieStella
Copy link
Contributor Author

there's no libcurl package. maybe libcurl-gnutls is enough?

@sfan5
Copy link
Member

sfan5 commented Sep 30, 2019

libcurl-gnutls provides libcurl and is what CMake will pick.

@ValkyrieStella
Copy link
Contributor Author

ok

@ValkyrieStella
Copy link
Contributor Author

done

@SmallJoker SmallJoker merged commit 251038e into minetest:master Oct 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@ Documentation Improvements or additions to documentation One approval ✅ ◻️
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants