multiple display support broken

[Brian May]

Binary package hint: ratpoison

If I select a window on my second display, that window gets keyboard focus (good), however any new programs I start appear on the first screen. This was fine before I upgraded from Maverick to Natty.

To create a window on the second display, I have to do it manually, e.g.:

export DISPLAY=:0.1
xterm

Will try downgrading ratpoision and see if this helps.

[Brian May (brian-microcomaustralia)]

Ummm... ok, the version of ratpoison in maverick is the same version. Looks like some incompatible change with the X server???

[Nicolas FAURE (n-faure)]

I have the same issue.
A workaround is to start ratpoison with "-d :0.0" (can be changed in /usr/share/xsessions/ratpoison.desktop)

[Brian May (brian-microcomaustralia)]

Hmmm.... Looks like DISPLAY, by default, is set to :0 not :0.0 now.

ratpoison doesn't add the screen number if there isn't already one there.

I am guessing the exact code is screen.c, from line 277:

=== cut ===
  /* Build the display string for each screen */
  s->display_string = xmalloc (strlen(DisplayString (dpy)) + 21);
  sprintf (s->display_string, "DISPLAY=%s", DisplayString (dpy));
  if (strrchr (DisplayString (dpy), ':'))
    {
      char *dot;

      dot = strrchr(s->display_string, '.');
      if (dot)
        sprintf(dot, ".%i", screen_num);
    }

  PRINT_DEBUG (("%s\n", s->display_string));
=== cut ===

If the "if (dot)" condition fails, it needs to append the string at the end I think. So something like (not tested):

char * dot;
dot = strrchr(s->display_string, '.');
if (!dot) dot = s->display_string + strlen(s->display_string);
sprintf(dot, ".%i", screen_num);

I think the buffer should already be big enough for this - strlen(DisplayString (dpy)) + 21 bytes.