Hello Willy,

2015-11-01 20:20 GMT+01:00 Willy Tarreau <w...@1wt.eu>:
> On Sun, Nov 01, 2015 at 01:18:00PM +0100, Jérôme Duval wrote:
>> * don't install the systemd-wrapper for all platforms, replace it with
>> $(EXTRA) like the "all" target already does.
>
> This case is not specific to your operating system, in fact only a
> minority of platforms need the systemd-wrapper. I personally build
> with "make ... EXTRA=" to avoid building it and installing it. I
> think we took the wrong approach by enabling it by default, since
> we're bothering all platforms. We should probably change this to
> have a "linux-systemd" target which enables it.
>
> If nobody objects, I'd rather do that so that you don't have to add
> a specific if/elif/endif just for this. What do you think ?

EXTRA is already correctly set to haproxy-systemd-wrapper for
platforms which might care (linux26 and linux2628). Please note that
the systemd-wrapper isn't built by default, only installed by default,
which is the problem I met, and tried to fix in this patch (I could
have better worded my commit message). I hope it helps.

Bye,
Jérôme

Reply via email to