Re: [PATCH xinit 2/2] startx: Make startx auto display select work with per user /tmp dirs

2015-03-26 Thread Hans de Goede
Hi, On 25-03-15 22:45, Julien Cristau wrote: On Wed, Mar 25, 2015 at 09:15:12 -0400, Ray Strode wrote: Hi, Now, I've tried to avoid anything xauth-related, but from the little I know: to support displayfd in startx you'd have to communicate back to startx about the $DISPLAY and do the xauth

Re: [PATCH xinit 2/2] startx: Make startx auto display select work with per user /tmp dirs

2015-03-25 Thread Hans de Goede
Hi, On 25-03-15 02:53, Peter Hutterer wrote: On Mon, Mar 23, 2015 at 02:16:13PM +, Jon TURNEY wrote: On 20/03/2015 17:16, Ray Strode wrote: On Fri, Mar 20, 2015 at 10:02 AM, Hans de Goede hdego...@redhat.com wrote: If a separate /tmp per user is used the existing auto display select

Re: [PATCH xinit 2/2] startx: Make startx auto display select work with per user /tmp dirs

2015-03-25 Thread Hans de Goede
Hi, On 25-03-15 14:20, Ray Strode wrote: Hi, You're right I had forgotten that the xauth file needs to contain the displaynr, so there is no easy fix here I'm afraid. That's not true actually. The xauth file can have a display wildcard, but the xauth command doesn't support generating xauth

Re: [PATCH xinit 2/2] startx: Make startx auto display select work with per user /tmp dirs

2015-03-25 Thread Ray Strode
Hi, Now, I've tried to avoid anything xauth-related, but from the little I know: to support displayfd in startx you'd have to communicate back to startx about the $DISPLAY and do the xauth dance before continuing with the xinit initial client connection. AFAICT, that's the tricky bit about

Re: [PATCH xinit 2/2] startx: Make startx auto display select work with per user /tmp dirs

2015-03-25 Thread Ray Strode
Hi, You're right I had forgotten that the xauth file needs to contain the displaynr, so there is no easy fix here I'm afraid. That's not true actually. The xauth file can have a display wildcard, but the xauth command doesn't support generating xauth files with display wildcards. I think that

Re: [PATCH xinit 2/2] startx: Make startx auto display select work with per user /tmp dirs

2015-03-25 Thread Julien Cristau
On Wed, Mar 25, 2015 at 09:15:12 -0400, Ray Strode wrote: Hi, Now, I've tried to avoid anything xauth-related, but from the little I know: to support displayfd in startx you'd have to communicate back to startx about the $DISPLAY and do the xauth dance before continuing with the xinit

Re: [PATCH xinit 2/2] startx: Make startx auto display select work with per user /tmp dirs

2015-03-24 Thread Peter Hutterer
On Mon, Mar 23, 2015 at 02:16:13PM +, Jon TURNEY wrote: On 20/03/2015 17:16, Ray Strode wrote: On Fri, Mar 20, 2015 at 10:02 AM, Hans de Goede hdego...@redhat.com wrote: If a separate /tmp per user is used the existing auto display select code does not work, add an extra check for the

Re: [PATCH xinit 2/2] startx: Make startx auto display select work with per user /tmp dirs

2015-03-23 Thread Jon TURNEY
On 20/03/2015 17:16, Ray Strode wrote: On Fri, Mar 20, 2015 at 10:02 AM, Hans de Goede hdego...@redhat.com wrote: If a separate /tmp per user is used the existing auto display select code does not work, add an extra check for the unix socket for the display number existing in /proc/net/unix

[PATCH xinit 2/2] startx: Make startx auto display select work with per user /tmp dirs

2015-03-20 Thread Hans de Goede
If a separate /tmp per user is used the existing auto display select code does not work, add an extra check for the unix socket for the display number existing in /proc/net/unix (linux only). Signed-off-by: Hans de Goede hdego...@redhat.com --- startx.cpp | 6 +- 1 file changed, 5

Re: [PATCH xinit 2/2] startx: Make startx auto display select work with per user /tmp dirs

2015-03-20 Thread Ray Strode
Hi, On Fri, Mar 20, 2015 at 10:02 AM, Hans de Goede hdego...@redhat.com wrote: If a separate /tmp per user is used the existing auto display select code does not work, add an extra check for the unix socket for the display number existing in /proc/net/unix (linux only). This patch and the