Re: [PATCH 0/3] libxtrans + xserver: Add support for systemd socket activation (v5)

2013-12-04 Thread Hans de Goede
Hi, On 12/03/2013 08:20 PM, Kristian Høgsberg wrote: On Tue, Dec 3, 2013 at 1:48 AM, Hans de Goede hdego...@redhat.com wrote: Hi, On 12/02/2013 09:54 PM, Kristian Høgsberg wrote: On Mon, Dec 2, 2013 at 11:44 AM, Hans de Goede hdego...@redhat.com wrote: Hi All, Hi, I didn't follow all

Re: [PATCH 0/3] libxtrans + xserver: Add support for systemd socket activation (v5)

2013-12-04 Thread Łukasz Stelmach
It was 2013-12-04 śro 10:56, when Hans de Goede wrote: Hi, On 12/03/2013 08:20 PM, Kristian Høgsberg wrote: On Tue, Dec 3, 2013 at 1:48 AM, Hans de Goede hdego...@redhat.com wrote: Hi, On 12/02/2013 09:54 PM, Kristian Høgsberg wrote: On Mon, Dec 2, 2013 at 11:44 AM, Hans de Goede

Re: [PATCH 0/3] libxtrans + xserver: Add support for systemd socket activation (v5)

2013-12-03 Thread Hans de Goede
Hi, On 12/02/2013 09:54 PM, Kristian Høgsberg wrote: On Mon, Dec 2, 2013 at 11:44 AM, Hans de Goede hdego...@redhat.com wrote: Hi All, Hi, I didn't follow all this in detail, but I'm not sure that we need Xtrans patches at all. The server already has ListenOnOpenFD() in os/connection.c.

Re: [PATCH 0/3] libxtrans + xserver: Add support for systemd socket activation (v5)

2013-12-03 Thread Kristian Høgsberg
On Tue, Dec 3, 2013 at 1:48 AM, Hans de Goede hdego...@redhat.com wrote: Hi, On 12/02/2013 09:54 PM, Kristian Høgsberg wrote: On Mon, Dec 2, 2013 at 11:44 AM, Hans de Goede hdego...@redhat.com wrote: Hi All, Hi, I didn't follow all this in detail, but I'm not sure that we need

[PATCH 0/3] libxtrans + xserver: Add support for systemd socket activation (v5)

2013-12-02 Thread Hans de Goede
Hi All, Here is v5 (I'm starting the numbering where Łukasz stopped) of the systemd socket activation patch-set. This version addresses all review comments made in response to my previous posting of this set. Testing can still be done like this: 1) Create a file named

Re: [PATCH 0/3] libxtrans + xserver: Add support for systemd socket activation (v5)

2013-12-02 Thread Kristian Høgsberg
On Mon, Dec 2, 2013 at 11:44 AM, Hans de Goede hdego...@redhat.com wrote: Hi All, Hi, I didn't follow all this in detail, but I'm not sure that we need Xtrans patches at all. The server already has ListenOnOpenFD() in os/connection.c. It was added for launchd under OSX, but we use it for