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: Extend Linux compiling section for Fedora #1680

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
3 participants
@stefanbeller
Copy link

commented Sep 21, 2014

No description provided.

@ShadowNinja

This comment has been minimized.

Copy link
Member

commented Sep 21, 2014

This seems to be missing a lot of the dependencies, even if they're usually already installed.

@ShadowNinja ShadowNinja force-pushed the minetest:master branch 2 times, most recently to 56195dc Sep 21, 2014

@Zeno-

This comment has been minimized.

Copy link
Contributor

commented Sep 22, 2014

In addition to dependencies can out-of-source builds be added? Or is that beyond the scope of that file? I only suggest it because of a personal problem I have with building in the src directory when using cmake.

@stefanbeller

This comment has been minimized.

Copy link
Author

commented Sep 22, 2014

@ShadowNinja

I am currently setting up a vanilla Fedora 20 virtual machine to test what exactly is missing. All I'll do is upgrading to latest packages (sudo yum upgrade) and then checking which packages will be missing.

@Zeno-
I just started contributing to minetest. (It's my 2nd patch) So I guess the developers more familiar with the projects are to decide, if the out of source builds should be described within the README file.
Anyway as it's a slightly different topic, I'd suggest to handle that issue in another pull request?

@stefanbeller

This comment has been minimized.

Copy link
Owner Author

commented on 440e7ab Sep 22, 2014

additionally we need at least: libvorbis-devel, libogg-devel, gcc-c++

@ShadowNinja

This comment has been minimized.

Copy link
Member

commented Sep 22, 2014

@stefanbeller: Won't you also need your package for libpng, libjpeg, libxxf86vm, libsqlite3, libcurl, libfreetype, and make?

@stefanbeller

This comment has been minimized.

Copy link
Author

commented Sep 22, 2014

Indeed, I'll figure it out tommorow. and redo the patch

@ShadowNinja ShadowNinja force-pushed the minetest:master branch from cba038e to b98e8d6 Oct 7, 2014

@Zeno- Zeno- force-pushed the minetest:master branch to a1db83e Nov 25, 2014

@ShadowNinja

This comment has been minimized.

Copy link
Member

commented Dec 12, 2014

@stefanbeller: Any updates?

@stefanbeller

This comment has been minimized.

Copy link
Author

commented Dec 14, 2014

Sorry, I lost track here. I'll close it for now. Anyone who is interested may pick up the patch to improve it or redo as they like.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.