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

Book update #3

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Book update #3

wants to merge 4 commits into from

Conversation

aanno
Copy link

@aanno aanno commented Sep 4, 2015

Hallo,

I've bought your book to teach my son a bit of programming. We are working on linux where gosu has no mp3 support. Hence I converted the affected files to ogg.

Pull request also includes some gem updates and eclipse files for the dynamic language extension.

Cheers,

aanno

@spajus
Copy link
Owner

spajus commented Sep 8, 2015

Hi, thank you for buying the book, I hope you enjoy it. I will have to consider merging this PR though, because code samples would get diverged in the book and in this repo - I need to keep them synchronized, so book update should come together. Thank you for contributing, however, I will look into the issue.

@shawn42
Copy link

shawn42 commented Sep 8, 2015

@spajus maybe you could have a current_book_version tag on the examples going forward to make these sorts of things easier?

@spajus
Copy link
Owner

spajus commented Sep 8, 2015

@shawn42 good idea, I will do that, so this PR can be merged then. But I have second thoughts about including Eclipse configuration in the repo. I believe that code should be editor / IDE agnostic.

@aanno
Copy link
Author

aanno commented Sep 12, 2015

Well, I'm ok with just proposing my changes. It is a bit strange that gosu got mp3 support for Win and Mac but not with Linux. And I'm aware that my change is not as focused as it should be.

However, if somebody encounter the same problem, he probably will find this discussion and could fix it...

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.

4 participants