[Vm-dev] Small diff for the new Joystick updates on old enough Linux

[Bruce O'Neel]

Hi,

I am sending this just to save the next person who stumbles over the
new Joystick stuff not building on old enough linuxes some time.  In
my case old enough is RedHat 7.

We should not add complexity to make this automagic.  But when
someone says "does not build on Linux kernels 3.x or so", I know,
ancient, we can say "here, this patch."

cheers

bruce

git diff
../../../../platforms/unix/plugins/JoystickTabletPlugin/sqUnixJoystickTablet.c

diff --git
a/platforms/unix/plugins/JoystickTabletPlugin/sqUnixJoystickTablet.c
b/platforms/unix/plugins/JoystickTabletPlugin/sqUnixJoys

index b2e69c7..a3baf8d 100644

---
a/platforms/unix/plugins/JoystickTabletPlugin/sqUnixJoystickTablet.c

+++
b/platforms/unix/plugins/JoystickTabletPlugin/sqUnixJoystickTablet.c

@@ -26,7 +26,7 @@

  */

/* Author: Ian.Piumarta at INRIA.Fr

-/* Author: davidf at afeka.ac.il

+ Author: davidf at afeka.ac.il

  */

#include <assert.h>

@@ -42,7 +42,7 @@

#include <dirent.h>

#include <errno.h>

#include <poll.h>

-#include <linux/input-event-codes.h>

+// #include <linux/input-event-codes.h>

#include <linux/input.h>

#include "JoystickTabletPlugin.h"

@@ -161,10 +161,10 @@ detectButtons(joystick_state_t* pj)

        else {

                // it's a modern gamepad device

-               pj->button_index[0] = BTN_NORTH;

+/*             pj->button_index[0] = BTN_NORTH;

                pj->button_index[1] = BTN_EAST;

                pj->button_index[2] = BTN_SOUTH;

-               pj->button_index[3] = BTN_WEST;

+               pj->button_index[3] = BTN_WEST; */

        }

        return 1;


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/vm-dev/attachments/20220613/ca1fd77f/attachment.html>