Hi,
Crikey, this is a blast from the past!
On 10/02/2026 20:11, Andreas Tille wrote:
I'm checking the list of bugs in rdesktop since this became a candidate
for the Bug of the Day. Reading the bug log including the link to
ArchLinux I think this very patch was not applied upstream. However,
there is
$ grep -- '-M:' *.c
rdesktop.c: fprintf(stderr, " -M: use local mouse cursor\n");
so the suggested option is in use. Since I'm not fully sure whether
this might be the feature you want to see implemented I'm tagging this
bug moreinfo.
I've looked at the source code, and I am reasonably confident that it
isn't - if you look at handle_button_event in xwin.c, you'll see[0]:
xevent.xbutton.button = g_pointer_log_to_phys_map[xevent.xbutton.button
- 1];
without any sort of conditional handling (which is what the patch in
this bug report sets up). Upstream's -M instead makes ui_set_cursor a
no-op[1].
I no longer have access to a suitable Windows server to test against,
but I think this means that upstream's -M doesn't leave mouse button
mappings alone.
So I think this bug report is likely still germane (although you'd now
need to change the option for dont-revert-mouse-button-mappings). I
might be wrong, though!
I see upstream is absent since 2019.
Regards,
Matthew
[0]
https://salsa.debian.org/debian/rdesktop/-/blame/master/xwin.c?ref_type=heads#L2519
[1]
https://salsa.debian.org/debian/rdesktop/-/blob/master/xwin.c?ref_type=heads#L3591
Kind regards and thank you for your patience over so many years
Andreas.
