xpra icon
Bug tracker and wiki

This bug tracker and wiki are being discontinued
please use https://github.com/Xpra-org/xpra instead.

Opened 5 years ago

Closed 5 years ago

Last modified 16 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:


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 5 years ago.
work in progress - doesn't like daemon=yes

Download all attachments as: .zip

Change History (5)

comment:1 Changed 5 years 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 5 years ago by Antoine Martin

Attachment: displayfd-v6.patch added

work in progress - doesn't like daemon=yes

comment:2 Changed 5 years 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 5 years 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!

comment:4 Changed 16 months ago by migration script

this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/1535

Note: See TracTickets for help on using tickets.