[PATCH v2] configure.ac: Use libsystemd in REQUIRED_LIBS check

Emil Velikov emil.l.velikov at gmail.com
Wed Nov 18 08:25:43 PST 2015


On 17 November 2015 at 08:39, Jussi Kukkonen <jussi.kukkonen at intel.com> wrote:
> REQUIRED_LIBS needs to be set to the correct systemd library,
> otherwise the later check will either fail or use the wrong
> pc file.
>
> Signed-off-by: Jussi Kukkonen <jussi.kukkonen at intel.com>
> ---
>
>
> Changes since v1:
> * Set both HAVE_SYSTEMD_DAEMON and REQUIRED_SYSTEMD_DAEMON inside
>   the PKG_CHECK_MODULES as per Emils review. Note that I don't think
>   his version with commas works -- but it's autoconf so not betting
>   any money on that.
I never remember the syntax so using commas might as well be wrong.

> * Check for libsystemd first: it should be preferred over the wrapper
>
Although this one is definitely not a good idea. This will likely
cause breakage with old systemd, where the daemon library is a
separate one - I'd keep it as is.

Regards,
Emil


More information about the xorg-devel mailing list