[PATCH] XOpenDisplay: save the correct display_name value

Jeremy Huddleston jeremyhu at apple.com
Tue Aug 17 09:33:59 PDT 2010


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.  

Assuming you revert the configure.ac changes:
Reviewed-by: Jeremy Huddleston <jeremyhu at apple.com>

--Jeremy



More information about the xorg-devel mailing list