Willy, > I really don't like this at all, because it means that we'll annoy all > the happy people who are not forced to suffer from systemd, ie basically > all those not running on a recent linux distro, simply because we're starting > to declare that certain environment variables only belong to other operating > systems (which is even worse when such variables are detected when the build > option is *not* set). Also the fact that such variables could be inherited > across multiple layers of processes is even more disgusting. > *snip*
Understood. Coming along is a patch that implements your suggested changes. Two things: - I was unsure whether to renumber the GTUNE_* constants. I opted not to. - I was unsure whether `#if defined(...)` or `#ifdef` is the variant to use for the USE_SYSTEMD option. You use both and I was unable to find out whether there is a scheme you follow. Thanks for your patience and really helpful feedback so far :-) Tim Duesterhus (1): MEDIUM: mworker: Add systemd `Type=notify` support Makefile | 7 +++++++ contrib/systemd/haproxy.service.in | 4 ++-- include/types/global.h | 1 + src/haproxy.c | 29 +++++++++++++++++++++++++++++ 4 files changed, 39 insertions(+), 2 deletions(-) -- 2.15.0
