Re: [PATCH xinit 1/2] startx: Fix startx picking an already used display number when -nolock is used

2015-03-25 Thread Hans de Goede
Hi, On 25-03-15 01:54, Peter Hutterer wrote: On Fri, Mar 20, 2015 at 03:02:35PM +0100, Hans de Goede wrote: Currently startx relies on /tmp/.X?-lock being present for automatically picking a free display number. This does not work if -nolock is used when starting the server, or if the server

Re: [PATCH xinit 1/2] startx: Fix startx picking an already used display number when -nolock is used

2015-03-24 Thread Peter Hutterer
On Fri, Mar 20, 2015 at 03:02:35PM +0100, Hans de Goede wrote: Currently startx relies on /tmp/.X?-lock being present for automatically picking a free display number. This does not work if -nolock is used when starting the server, or if the server is started with -displayfd as -displayfd

[PATCH xinit 1/2] startx: Fix startx picking an already used display number when -nolock is used

2015-03-20 Thread Hans de Goede
Currently startx relies on /tmp/.X?-lock being present for automatically picking a free display number. This does not work if -nolock is used when starting the server, or if the server is started with -displayfd as -displayfd implies -nolock. This is becoming a problem now that -displayfd is

Re: [PATCH xinit 1/2] startx: Fix startx picking an already used display number when -nolock is used

2015-03-20 Thread Jasper St. Pierre
... Why does displayfd imply nolock? That seems strange to me. I consider the .X0-lock files an API that we shouldn't break. On Fri, Mar 20, 2015 at 7:02 AM, Hans de Goede hdego...@redhat.com wrote: Currently startx relies on /tmp/.X?-lock being present for automatically picking a free display

Re: [PATCH xinit 1/2] startx: Fix startx picking an already used display number when -nolock is used

2015-03-20 Thread Ray Strode
Hi, On Fri, Mar 20, 2015 at 1:01 PM, Jasper St. Pierre jstpie...@mecheye.net wrote: ... Why does displayfd imply nolock? It doesn't really matter why it implies, nolock, I guess. The issue at hand is that it does (and always has) implied nolock. You can't fix that without breaking things that