Re: Three patches to the haproxy-systemd-wrapper

2013-12-10 Thread kristoffer grönlund
On 10 dec 2013, at 01:12, Lukas Tribus luky...@hotmail.com wrote: Can we silence the warning as per the diff attached? If you agree, I will send a proper patch to Willy. Looks good to me! // Kristoffer

RE: Three patches to the haproxy-systemd-wrapper

2013-12-09 Thread Lukas Tribus
Hi folks, On Sat, 23 Nov 2013 12:05:24 +0100 Willy Tarreau w...@1wt.eu wrote: Hi Mark-Antoine, On Sat, Nov 23, 2013 at 07:37:21PM +0900, Marc-Antoine Perennou wrote: I don't have access to a computer to actually test those, but: - the first one looks nice, never felt really confident

Re: Three patches to the haproxy-systemd-wrapper

2013-12-09 Thread Willy Tarreau
Hi Lukas, On Tue, Dec 10, 2013 at 01:12:59AM +0100, Lukas Tribus wrote: Hi folks, On Sat, 23 Nov 2013 12:05:24 +0100 Willy Tarreau w...@1wt.eu wrote: Hi Mark-Antoine, On Sat, Nov 23, 2013 at 07:37:21PM +0900, Marc-Antoine Perennou wrote: I don't have access to a computer to

Re: Three patches to the haproxy-systemd-wrapper

2013-11-28 Thread Kristoffer Grönlund
On Sat, 23 Nov 2013 12:05:24 +0100 Willy Tarreau w...@1wt.eu wrote: Hi Mark-Antoine, On Sat, Nov 23, 2013 at 07:37:21PM +0900, Marc-Antoine Perennou wrote: I don't have access to a computer to actually test those, but: - the first one looks nice, never felt really confident hard coding

Re: Three patches to the haproxy-systemd-wrapper

2013-11-23 Thread Willy Tarreau
Hello Kristoffer, On Fri, Nov 22, 2013 at 11:21:21AM +0100, Kristoffer Grönlund wrote: Hello, I have packaged haproxy 1.4.24 for openSUSE 13.1 [1], which uses systemd as the init system. At first I tried using a service script originally written for Fedora, but this script didn't handle

Re: Three patches to the haproxy-systemd-wrapper

2013-11-23 Thread Marc-Antoine Perennou
Hello, On Saturday, November 23, 2013, Willy Tarreau wrote: Hello Kristoffer, On Fri, Nov 22, 2013 at 11:21:21AM +0100, Kristoffer Grönlund wrote: Hello, I have packaged haproxy 1.4.24 for openSUSE 13.1 [1], which uses systemd as the init system. At first I tried using a service

Re: Three patches to the haproxy-systemd-wrapper

2013-11-23 Thread Willy Tarreau
Hi Mark-Antoine, On Sat, Nov 23, 2013 at 07:37:21PM +0900, Marc-Antoine Perennou wrote: I don't have access to a computer to actually test those, but: - the first one looks nice, never felt really confident hard coding SBINDIR and the solution makes sense - nice catch for the second one,