-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Initial Haiku support #6568
Initial Haiku support #6568
Conversation
src/CMakeLists.txt
Outdated
@@ -329,9 +337,11 @@ else() | |||
endif(APPLE) | |||
|
|||
# This way Xxf86vm is found on OpenBSD too |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this comment should be moved inside the if()
too
src/CMakeLists.txt
Outdated
if(NOT HAIKU) | ||
# This way Xxf86vm is found on OpenBSD too |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
now you just need to indent it ;)
src/client/renderingengine.cpp
Outdated
@@ -34,7 +34,8 @@ with this program; if not, write to the Free Software Foundation, Inc., | |||
#include "inputhandler.h" | |||
#include "gettext.h" | |||
|
|||
#if !defined(_WIN32) && !defined(__APPLE__) && !defined(__ANDROID__) && !defined(SERVER) && !defined(__HAIKU__) | |||
#if !defined(_WIN32) && !defined(__APPLE__) && !defined(__ANDROID__) \ | |||
&& !defined(SERVER) && !defined(__HAIKU__) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the tab in the middle should be a space
should we support a such OS ? |
@nerzhul should not, but why not to try. |
the linter still has a complaint about your syntax: https://travis-ci.org/minetest/minetest/jobs/294459402#L504 |
Not officially support, ie: no builds or ci tests maintained by us, but this pr is fine |
src/client/renderingengine.cpp
Outdated
@@ -34,8 +34,8 @@ with this program; if not, write to the Free Software Foundation, Inc., | |||
#include "inputhandler.h" | |||
#include "gettext.h" | |||
|
|||
#if !defined(_WIN32) && !defined(__APPLE__) && !defined(__ANDROID__) \ | |||
&& !defined(SERVER) && !defined(__HAIKU__) | |||
#if !defined(_WIN32) && !defined(__APPLE__) && !defined(__ANDROID__) && |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still need the \
on this line
* Iitial Haiku support
* Iitial Haiku support
* Iitial Haiku support
No description provided.