From: matthew@debian.org   
      
   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?r   
   f_type=heads#L2519   
   [1]   
   https://salsa.debian.org/debian/rdesktop/-/blob/master/xwin.c?re   
   _type=heads#L3591   
      
      
   > Kind regards and thank you for your patience over so many years   
   >     Andreas.   
   >   
      
   --- SoupGate-Win32 v1.05   
    * Origin: you cannot sedate... all the things you hate (1:229/2)