pudwerkin / udpt

Automatically exported from code.google.com/p/udpt
0 stars 0 forks source link

Please add OSX support (trivial) #3

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Perhaps the way to do this would be to add an ifdef for osx at the top of 
multiplatform.h that also defines another macro for the platform string 
included in the version string.

Quick work around:

Add "#define linux" at the top of src/multiplatform.h.
It will compile with no issues.

Original issue reported on code.google.com by speedrac...@gmail.com on 11 Mar 2013 at 9:30

GoogleCodeExporter commented 9 years ago
I think this is all that is required.

diff --git a/src/multiplatform.h b/src/multiplatform.h
index 97b5d4c..11808d1 100644
--- a/src/multiplatform.h
+++ b/src/multiplatform.h
@@ -22,6 +22,10 @@

 #include <stdint.h>

+#if defined(__APPLE__)
+#define linux
+#endif
+
 #if defined (_WIN32) && !defined (WIN32)
 #define WIN32
 #endif
@@ -58,3 +62,6 @@ typedef pthread_t HANDLE;
 #define VERSION "1.0.0-alpha (Linux)"
 #endif

+#if defined(__APPLE__)
+#define VERSION "1.0.0-alpha (OS X)"
+#endif

Original comment by speedrac...@gmail.com on 11 Mar 2013 at 9:35

GoogleCodeExporter commented 9 years ago
diff --git a/README b/README
index 5110a8f..8128fe0 100644
--- a/README
+++ b/README
@@ -4,13 +4,10 @@ README for the UDPT project.
 Licensed under GNU GPLv3.
 The license file is attached under the name gpl.txt.

-Compiling under linux or linux environment (MinGW):
+Compiling requires Linux, Windows with a Linux environment (MinGW),
+or OS X:

-For Windows (with Linux environment):
-$ make win32
-
-For Linux:
-$ make linux
+$ make

 Running:
 $ ./udpt

Original comment by speedrac...@gmail.com on 11 Mar 2013 at 9:37

GoogleCodeExporter commented 9 years ago
Thank you very much for posting this issue, I can do that - But i won't provide 
OSX support since i don't own a Mac... I will publish this change... (OSX is 
Linux based...)

Original comment by naim...@gmail.com on 13 Mar 2013 at 2:56

GoogleCodeExporter commented 9 years ago
Code changed: Commit e924bbca24b7

Original comment by naim...@gmail.com on 13 Mar 2013 at 4:08

GoogleCodeExporter commented 9 years ago
Please comment if the project doesn't compile on OS X.

Original comment by naim...@gmail.com on 13 Mar 2013 at 4:15

GoogleCodeExporter commented 9 years ago
Works fine, thanks.

Original comment by speedrac...@gmail.com on 13 Mar 2013 at 4:33

GoogleCodeExporter commented 9 years ago
Thank you for verifing.

Original comment by naim...@gmail.com on 14 Mar 2013 at 8:21

GoogleCodeExporter commented 9 years ago

Original comment by naim...@gmail.com on 22 Mar 2013 at 1:53