-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
Portability #29
Comments
Related with #17 . Also, I saw a branch with cmake support, was it deleted or merged @enriquesomolinos ? |
Yes, this is one of the main golas of the project. I try to create a stable window version in this months to start this task later. |
@enriquesomolinos it seems you are trying to fix travis to build the portability branch. Do you need some help with that? |
yes, help will be appreciated. In my locla computer dreerally compiles fine, but in travis libraries are not well recognized |
I'm taking a look to this issue again.. |
Thanks for the contribution neuromancer, the code compiles fine 👍 . Now i need to solve some problems in order to have a functional executable under linux. |
It fails to run here:
|
@enriquesomolinos I tested the latest revision of the
Just to clarify, I'm using the official Death Rally data files. |
you right, try to remove the video files (*.haf files) In linux there is a lot of errors. |
It works! 🎉 (however there is no keyboard input). Keep up the great work! |
Yes, it has a lot of error, but it is a good start. |
It seems that the linux version won't compile unless you create the |
Yes, the makefile.linux file tries to create the executable under this directory. In the next release we can change it without problem |
In the branch 0.3.x the problem is resolved. |
I cannot compile under linux: first problem: the was no -lSDLMain, could be safely removed, I think But now many other errors: make -f Makefile.linux |
I tried to compile the latest 0.3 ver today (bb07127) on Linux and I got this error: https://gist.github.com/ToughGuyKunio/606701cbb473293b37b9207b613cf7d6 Is there another branch that is working? |
I managed to get it to compile on Arch Linux. I had to do some minor fixes, see this branch: https://github.com/karjonas/DreeRally/commits/linuxtest Sound is disabled, keyboard input is still borked and it crashes randomly so don't expect too much. |
Is your feature request related to a problem? Please describe.
According to the website, one of the stated goals of the project is the possibilty to port Death Rally to other platforms. However, current codebase is strongly coupled with Windows/Visual Studio/MSVC environment.
Describe the solution you'd like
There are several issues which prevent us from being able to compile DreeRally for e.g. Linux. The most obvious one is Windows-specific C functions and headers. They should be identified and replaced with SDL equivalents (if graphics/windowing/GUI related) or with ISO C equivalents (if related to things like e.g. file access). If all else fails, we can use
#ifdef
macros.Less obvious issues are non-standard language elements (e.g. I'm pretty sure that some of the
#pragma
s are MSVC specific and won't work with gcc or clang) and build process. I think that pragmas are mostly used to suppress warnings, which I think is something we could live without. As for build process, it seems simple enough - shouldn't be too hard to make an equivalent Makefile.Describe alternatives you've considered
Running in Wine, I guess? But I think source portability would be preferable.
Additional context
N/A
The text was updated successfully, but these errors were encountered: