[PATCH] XOpenDisplay: save the correct display_name value

Aaron Plattner aplattner at nvidia.com
Wed Aug 18 07:48:47 PDT 2010


On Tue, Aug 17, 2010 at 09:33:59AM -0700, Jeremy Huddleston wrote:
> On Aug 16, 2010, at 09:34, Aaron Plattner wrote:
> > Jeremy, you seem to the be the most familiar with launchd.  Does this
> > change seem okay to you?  Since this change stops appending the screen
> > number all the time, it seems like it should be a no-op for the launchd
> > case.
>
> Yeah, we were getting bitten by assumptions elsewhere about the screen
> number not being tacked on in ssh's support for the launchd socket.  I
> just assumed it was an ambiguity and removed it for that case.  Another
> fix is being made to ssh (maybe upstream now, maybe not) which will
> support a screen number and pass it on to the remote tunneled DISPLAY.
>
> The configure.ac changes are actually still needed for the xtrans path.
> Aside from that, it looks good.

Which xtrans path are you referring to?  There are no users of HAVE_LAUNCHD
left, and the xtrans path was removed with commit 15e5eaf628 right after
libX11-1.3.4.

-- Aaron


More information about the xorg-devel mailing list