This pipeline builds the driver against the latest Xserver stable
release as well as current master.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
The module directory has changed to a per ABI folder in the xlibre-xserver.
Now the default value of `xorg-module-dir` will be detected from the `moduledir` variable in xorg-server.pc.
Signed-off-by: b-aaz <b-aazbsd.proton.me>
Add the following forms for issue creation:
* Bug report
* Feature request
* Code change
* Documentation update
* Organizational task
* add issue type selection page on "New Issue" call
* mention Github Discussions and the mailing list where appropriate
Part-of: X11Libre/misc#156
Signed-off-by: callmetango <callmetango@users.noreply.github.com>
mandoc: mousedrv.4:3:13: WARNING: cannot parse date, using it verbatim: TH xf86-input-mouse 1.9.5
an.tmac:mousedrv.4:51: style: blank line in input
an.tmac:mousedrv.4:187: style: .BR expects at least 2 arguments, got 1
an.tmac:mousedrv.4:190: style: .BR expects at least 2 arguments, got 1
an.tmac:mousedrv.4:215: style: .BI expects at least 2 arguments, got 1
an.tmac:mousedrv.4:217: style: .BI expects at least 2 arguments, got 1
an.tmac:mousedrv.4:258: style: .BR expects at least 2 arguments, got 1
an.tmac:mousedrv.4:344: style: blank line in input
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-input-mouse/-/merge_requests/24>
Clears warnings from clang 19.1.7:
mouse.c:2877:31: warning: empty expression statement has no effect;
remove unnecessary ';' to silence this warning [-Wextra-semi-stmt]
2877 | do_ps2Reset(pInfo);
| ^
mouse.c:3441:29: warning: empty expression statement has no effect;
remove unnecessary ';' to silence this warning [-Wextra-semi-stmt]
3441 | RESET_VALIDATION;
| ^
[repeated for each call]
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-input-mouse/-/merge_requests/23>
Should never happen, but the compiler would need to understand the flow
of our state machine to realize that, and this protects us if that ever
changes.
Clears warning from gcc 14.2.0:
mouse.c: In function ‘createProtoList’:
mouse.c:3226:30: warning: dereference of NULL ‘protoList’ [CWE-476]
[-Wanalyzer-null-dereference]
3226 | for (i = 0; protoList[i] != PROT_UNKNOWN; i++) {
| ~~~~~~~~~^~~
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-input-mouse/-/merge_requests/23>
Since the standard API on Linux is evdev (and we've got evdev and
libinput drivers for that), there's probably no need for using this
driver on Linux anymore. So, we can drop Linux support here.
Making the build fail on Linux, similar to xf86-input-keyboard.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-input-mouse/-/merge_requests/19>
Clears -Wcalloc-transposed-args warnings from gcc 14.1, such as:
mouse.c: In function ‘MousePreInit’:
mouse.c:900:32: warning: ‘calloc’ sizes specified with ‘sizeof’ in the
earlier argument and not in the later argument [-Wcalloc-transposed-args]
900 | if (!(pMse = calloc(sizeof(MouseDevRec), 1)))
| ^~~~~~~~~~~
mouse.c:900:32: note: earlier argument should specify number of elements,
later size of each element
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-input-mouse/-/merge_requests/17>
xf86_OSlib.h used to include sys/stat.h but that was recently removed by
xorg/xserver@5057c716eb so we need to do
it here now.
Fixes compiler errors:
bsd_mouse.c: In function 'FindDevice':
bsd_mouse.c:282:17: error: storage size of 'devMouseStat' isn't known
282 | struct stat devMouseStat;
| ^~~~~~~~~~~~
bsd_mouse.c:283:17: error: storage size of 'sb' isn't known
283 | struct stat sb;
| ^~
bsd_mouse.c:298:21: error: implicit declaration of function 'fstat' [-Werror=implicit-function-declaration]
298 | if (fstat(fd, &devMouseStat) == 0)
| ^~~~~
bsd_mouse.c:298:21: warning: nested extern declaration of 'fstat' [-Wnested-externs]
bsd_mouse.c:283:17: warning: unused variable 'sb' [-Wunused-variable]
283 | struct stat sb;
| ^~
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-input-mouse/-/merge_requests/12>
Two source files currently relying on xorg headers including errno.h for them.
Even though it practically works, it's not a very robust programming style
to rely on indirect includes. It's better when everybody includes what he need
on its own.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
AC_PROG_LIBTOOL was replaced by LT_INIT in libtool 2 in 2008,
so it's time to rely on it.
Clears autoconf warnings:
configure.ac:39: warning: The macro 'AC_PROG_LIBTOOL' is obsolete.
configure.ac:39: You should run autoupdate.
aclocal.m4:3465: AC_PROG_LIBTOOL is expanded from...
configure.ac:39: the top level
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Needed to avoid "Failed to unpack image" errors in the container-prep
job since FreeBSD 13.0 images are no longer available for download
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Originally written by fei.feng@oracle.com to fix Oracle Bug 17429216:
"global vuidMouseList should not keep info for removed mouse device"
"During the testing for bug#17251473, I often see the mouse hang in X.
By debugging, I get that there seems something wrong in mouse_drv.so -
when a mouse is disconnected, the global vuidMouseList does not cleanup
the mouse's recorded info. So if a newly inserted mouse allocates a
input info pInfo which happens to have the same memory address as
previous pInfo, the driver would go wrong."
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Enabled automatically for the Sun ILOM device, to avoid acceleration
that causes the Remote KVMS mouse cursor to become unaligned with the
X mouse cursor.
Code originally written by david.m.marx@oracle.com in 2013 to fix
Oracle bug 15798251 - SUNBT7177072 "actual mouse pointer is off from
the cursor on remote window"
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Not needed by mouse driver, but quiets warnings from xorg server headers:
In file included from /usr/include/xorg/misc.h:117:0,
from /usr/include/xorg/xf86str.h:37,
from /usr/include/xorg/xf86.h:44,
from mouse.c:57:
/usr/include/xorg/os.h:595:1: warning: redundant redeclaration of ‘timingsafe_memcmp’ [-Wredundant-decls]
timingsafe_memcmp(const void *b1, const void *b2, size_t len);
^~~~~~~~~~~~~~~~~
In file included from mouse.c:52:0:
/usr/include/string.h:235:12: note: previous declaration of ‘timingsafe_memcmp’ was here
extern int timingsafe_memcmp(const void *s1, const void *s2, size_t n);
^~~~~~~~~~~~~~~~~
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
mouse.c: In function ‘autoGood’:
mouse.c:3724:12: warning: this statement may fall through [-Wimplicit-fallthrough=]
if (mPriv->goodCount < PROBE_UNCERTAINTY/2)
^
mouse.c:3726:5: note: here
default:
^~~~~~~
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
mouse.c:185:1: warning: missing initializer for field ‘default_options’ of ‘InputDriverRec {aka struct _InputDriverRec}’ [-Wmissing-field-initializers]
};
^
In file included from mouse.c:63:0:
/usr/include/xorg/xf86Xinput.h:83:18: note: ‘default_options’ declared here
const char **default_options;
^~~~~~~~~~~~~~~
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
mouse.c: In function ‘SetupMouse’:
mouse.c:2620:27: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (i = 0; i < sizeof(pMse->protoPara); i++)
^
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
mouse.c: In function ‘checkForErraticMovements’:
mouse.c:3759:52: warning: suggest braces around empty body in an ‘else’ statement [-Wempty-body]
AP_DBG(("accDx=%i\n",mPriv->accDx));
^
mouse.c:3772:52: warning: suggest braces around empty body in an ‘else’ statement [-Wempty-body]
AP_DBG(("accDy=%i\n",mPriv->accDy));
^
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
sun_mouse.c: In function ‘vuidReadInput’:
sun_mouse.c:299:22: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
} else if (n == -1) {
^~
sun_mouse.c: In function ‘vuidMouseProc’:
sun_mouse.c:507:33: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
if (vuidMouseGeneration != serverGeneration) {
^~
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
