Print

Please login or register.
1 Hour 1 Day 1 Week 1 Month Forever
Login with username, password and session length

[slarti]

I've just tried to compile the current 1.2 CVS version on a remote machine (debian I think, but quite specialised). The build failed because linux/input.h didn't define the FF_EFFECT_MIN and FF_WAVEFORM_MIN macros. Of course I cannot confirm that the header file was original, but our IT guys have no reason to delete a few macros. It might just be an older version. Anyway I propose the following fix:

Code: [Select]

--- linux/EventHelpers.cpp (revision 5668)
+++ linux/EventHelpers.cpp (working copy)
@@ -35,6 +35,20 @@
 # include <iostream>
 #endif
 
+// Fixes for missing macros in input.h
+#ifndef FF_EFFECT_MIN
+#define FF_EFFECT_MIN FF_RUMBLE
+#endif
+#ifndef FF_EFFECT_MAX
+#define FF_EFFECT_MAX FF_RAMP
+#endif
+#ifndef FF_WAVEFORM_MIN
+#define FF_WAVEFORM_MIN FF_SQUARE
+#endif
+#ifndef FF_WAVEFORM_MAX
+#define FF_WAVEFORM_MAX FF_CUSTOM
+#endif
+
 using namespace std;
 using namespace OIS;
 


[pjcast]

Please submit as a patch so it doesn't get lost on the forum. I haven't tried the latest distro's out there, but I don't know why that header include would be missing some defines - I don't think it would be missing in an old version, but who knows - Linux API's are not always known for remaining the same.