OSX: inconsistence docs, incomplete cmakelist.txt build file

[nagyd]

OSX: inconsistence docs, incomplete cmakelist.txt build file

[mfn]

I don't have time right to now to verify the issues but IMHO it's fine to close it and if I find something at a later point I would open another issue!

[NagyD]

Hello and welcome!

First, I need to mention that OSX-related things were done by StaticReturn and @ecalot.
And, CMake-related things are done by @Falcury.

The Readme.txt mentions to use make -f Makefile.osx. However this file does not exist and Makefile seems to handle both cases already

You're right, the Makefiles were merged in bc3c1c1 without changing the Readme.
I updated it now: f99c423

The CMakeLists.txt actually contains a comment that the OSX configuration is missing; I was able to get i working with this change:

Okay. If possible, you should add an if so that Windows sees the original line with mingw.

Btw, what do you think about moving /docs/Readme.txt to /readme.md, format it a bit to markdown on the way?
I think it would be nicer for first time visitors to have this information here readily presented.

Markdown is a good idea, but I'm not sure whether the whole Readme belongs to the main page.
Anyway, it's a good thing if people see that this project is alive.

Thanks for your comments!
Feel free to make a pull request for anything you mentioned.

[ecalot]

My mistake, I forgot to remove the -f argument in the documentation!

[mfn]

Ok, thanks for the feedback! :)

Btw, being a complete n00b to the whole OSX/brew thing is already shown why I don't had ogg support: it's a matter of running brew install sdl2_mixer --with-libvorbis actually.

[ecalot]

I've pull requested your suggestions to the readme file

[Falcury]

I added a pull request for the CMake configuration, with an if so that on Windows mingw32 and SDL2.dll will still be linked.

[NagyD]

Can we close this now, or is there still something left?

[NagyD]

Okay.