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

master #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

master #1

wants to merge 1 commit into from

Conversation

nexero
Copy link

@nexero nexero commented Jul 17, 2016

Had to change some files for successful building with linux platform. not sure, if it breaks something for win32 or macos, though..

@antrad
Copy link
Owner

antrad commented Jul 21, 2016

Hello. I browsed through your changes and I guess most of it should be fine (though I have very limited/no knowledge of Cmake and cross platform programming), but I currently don't have the PC set up to test it, I'm having a break from coding and all.

@tsudoko
Copy link

tsudoko commented Aug 25, 2016

Had to change a bit more to get it to compile on Gentoo:

diff --git a/src/common.h b/src/common.h
index ee45256..a798dfb 100644
--- a/src/common.h
+++ b/src/common.h
@@ -16,6 +16,7 @@
 //
 #include <SDL2/SDL_config.h>//AR (#include <SDL_config.h>)
 #include <stdio.h>
+#include <stdint.h>

 #ifdef _MSC_VER
 // For simplicity sake, just make snprintf sprintf_s even though they aren't quite the same
diff --git a/src/sdlport/video.cpp b/src/sdlport/video.cpp
index b126044..9a0247a 100644
--- a/src/sdlport/video.cpp
+++ b/src/sdlport/video.cpp
@@ -27,7 +27,7 @@
 #ifdef WIN32
 #include <windows.h>
 #endif
-#include "GLee.h"
+#include <GL/GLee.h>
 //Abuse linker->input:
 // - opengl32.lib
 // - GLee.lib
diff --git a/src/tool/AR_SPEC.h b/src/tool/AR_SPEC.h
index 70e1d60..edc1359 100644
--- a/src/tool/AR_SPEC.h
+++ b/src/tool/AR_SPEC.h
@@ -25,7 +25,6 @@
 // - linker
 //     cv210.lib
 //     cxcore210.lib
-//     highgui210.lib
 // - libraries
 //     OpenCV2.1\lib
 // - include
@@ -60,7 +59,6 @@
 #include <sstream>

 #include <opencv2/opencv.hpp>
-#include <opencv2/highgui.hpp>
 using namespace cv;

 #include "common.h"

@monreal
Copy link

monreal commented Sep 11, 2017

I just compiled this on Fedora 26. Needed to apply both patchsets.

@tuomosipola
Copy link

tuomosipola commented Jun 7, 2019

Just compiled on Ubuntu 19.04. Applied both patches.

In addition, GLee seems to be unmaintaided (and not included in Ubuntu), so the more modern glew should be used in src/sdlport/video.cpp: change #include "GLee.h" to #include <GL/glew.h>

@ghost
Copy link

ghost commented Oct 24, 2019

@tuomosipola Could you try my fork? It uses Glad instead of GLee or glew, as well as some other changes.

@tuomosipola
Copy link

@fluffrabbit Sorry, I just noticed this message. Tried it. It compiles and I could play a bit. No sound, but I don't think even this fork had them.

@ghost
Copy link

ghost commented Mar 2, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants