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 for newbies #64

Closed
wants to merge 2 commits into from
Closed

README for newbies #64

wants to merge 2 commits into from

Conversation

tjkierzkowski
Copy link

Hello! I learned about this project through a reddit post and couldn't help but try it out. I know it's like super alpha right now but I wanted to contribute a better readme to help others interested in building/running the project. I only have the prerequisites done and still have to work on the build section. I didn't get to fix the build section just yet because I'm still hacking away at getting it to run via cargo run.

Thanks again for your awesome contribution!

@eddyb
Copy link
Contributor

eddyb commented Aug 19, 2014

This is neat, but we're using SDL2.
It may be a good idea to switch to GLFW. Thoughts, @bvssvni?

About getting cargo run to work, all you need is to make the argument to AssetStore::from_folder relative to target (../bin/assets in this case).
I already did this in my minecraft branch, and I'm no longer using the Makefile.
Sorry for taking so long to merge that, btw.

@bvssvni
Copy link
Member

bvssvni commented Aug 19, 2014

It does not matter whether it's SDL2 or GLFW. Perhaps we should have a flag and set it to SDL2 by default?

Here is a link of how to deal with SDL2 problems: PistonDevelopers/rust-empty#175

Here is a link of how to deal with GLFW problems: PistonDevelopers/rust-empty#48

@bvssvni
Copy link
Member

bvssvni commented Aug 28, 2014

@tjkierzkowski If you want to update the README, open up a new PR.

@bvssvni bvssvni closed this Aug 28, 2014
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