xpra icon
Bug tracker and wiki

Opened 5 months ago

Closed 3 months ago

Last modified 2 months ago

#1535 closed enhancement (fixed)

displayfd switch for xpra

Reported by: Antoine Martin Owned by: Antoine Martin
Priority: critical Milestone: 2.2
Component: server Version: trunk
Keywords: x11 Cc:

Description

We can now start servers via the proxy (#1105) and this may well become the default.
Since we may not know the actual X11 display in advance (we rely on Xorg -displayfd, #172), we check for new sockets and probe them.
A better way would be to provide a "--displayfd" switch and write the display to this file descriptor.

Attachments (1)

displayfd-v6.patch (14.3 KB) - added by Antoine Martin 3 months ago.
work in progress - doesn't like daemon=yes

Download all attachments as: .zip

Change History (4)

comment:1 Changed 4 months ago by Antoine Martin

Milestone: 3.02.2
Priority: majorcritical
Status: newassigned

Would really help speedup the start-via-proxy feature (#1105 aka logind multiseat), so raising.

Changed 3 months ago by Antoine Martin

Attachment: displayfd-v6.patch added

work in progress - doesn't like daemon=yes

comment:2 Changed 3 months ago by Antoine Martin

Resolution: fixed
Status: assignedclosed

Done in r16612.

Not as useful as I thought it was going to be: we still have to go through the identify_new_socket code because the display may not be available when we get the display value back (sockets take time, may not be accessible, etc).

But at least things should be more reliable than before.
This feature may also be used for other purposes.

comment:3 Changed 2 months ago by Antoine Martin

Minor related fixes:

  • r16636: honour displayfd with "request-start" mode (start via system wide proxy)
  • r16631: ignore displayfd=0 (the default), don't try to write there!
Note: See TracTickets for help on using tickets.