<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 16 November 2015 at 17:26, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi Jussi,<br>
<br>
I'm not as experienced as other devs here so take all of the following<br>
with a pinch of salt.<br>
<div><div class="h5"><br>
On 16 November 2015 at 14:46, Jussi Kukkonen <<a href="mailto:jussi.kukkonen@intel.com">jussi.kukkonen@intel.com</a>> wrote:<br>
> REQUIRED_LIBS needs to be set to the correct systemd library,<br>
> otherwise the later check will either fail or use the wrong<br>
> pc file.<br>
><br>
> Signed-off-by: Jussi Kukkonen <<a href="mailto:jussi.kukkonen@intel.com">jussi.kukkonen@intel.com</a>><br>
> ---<br>
>  <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> | 13 ++++++++++---<br>
>  1 file changed, 10 insertions(+), 3 deletions(-)<br>
><br>
> diff --git a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> index 96c0242..f63eca1 100644<br>
> --- a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> +++ b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> @@ -837,9 +837,16 @@ AC_ARG_WITH([systemd-daemon],<br>
>                 [support systemd socket activation (default: auto)]),<br>
>         [WITH_SYSTEMD_DAEMON=$withval], [WITH_SYSTEMD_DAEMON=auto])<br>
>  PKG_CHECK_MODULES([SYSTEMD_DAEMON], [libsystemd-daemon],<br>
> -                  [HAVE_SYSTEMD_DAEMON=yes],<br>
> +                  [REQUIRED_SYSTEMD_DAEMON=libsystemd-daemon],<br>
>                    [PKG_CHECK_MODULES([SYSTEMD_DAEMON], [libsystemd],<br>
> -                                     [HAVE_SYSTEMD_DAEMON=yes], [HAVE_SYSTEMD_DAEMON=no])])<br>
> +                                     [REQUIRED_SYSTEMD_DAEMON=libsystemd],<br>
> +                                     [REQUIRED_SYSTEMD_DAEMON=])])<br>
> +if test "x$REQUIRED_SYSTEMD_DAEMON" = x; then<br>
> +        HAVE_SYSTEMD_DAEMON=no<br>
> +else<br>
> +        HAVE_SYSTEMD_DAEMON=yes<br>
> +fi<br>
> +<br>
>  if test "x$WITH_SYSTEMD_DAEMON" = xauto; then<br>
>         WITH_SYSTEMD_DAEMON="$HAVE_SYSTEMD_DAEMON"<br>
>  fi<br>
> @@ -848,7 +855,7 @@ if test "x$WITH_SYSTEMD_DAEMON" = xyes; then<br>
>                 AC_MSG_ERROR([systemd support requested but no library has been found])<br>
>         fi<br>
>         AC_DEFINE(HAVE_SYSTEMD_DAEMON, 1, [Define to 1 if libsystemd-daemon is available])<br>
> -       REQUIRED_LIBS="$REQUIRED_LIBS libsystemd-daemon"<br>
> +       REQUIRED_LIBS="$REQUIRED_LIBS $REQUIRED_SYSTEMD_DAEMON"<br>
</div></div>Rather than flipping things into a different (and somewhat obscure)<br>
way, why don't you just use SYSTEMD_DAEMON_LIBS instead of the<br>
hardcoded libsystemd-daemon ?<br>
<br>
Shorter diff, consistent checks and it should work (although I haven't<br>
checked it).<br></blockquote><div><br></div><div>The way it's done is indeed a bit obscure (might just be my limited experience with xorg build system though). My impression is this:</div><div>Individual dependencies are checked one by one, but libs/cflags are not set at this point. Later in the file there's two PKG_CHECK_MODULES calls that set $<span style="color:rgb(0,0,0);font-size:13.3333px">XSERVERCFLAGS and $</span><span style="color:rgb(0,0,0);font-size:13.3333px">XSERVERLIBS</span> for all <span style="color:rgb(80,0,80)">$REQUIRED_LIBS.</span></div><div><span style="color:rgb(80,0,80)"><br></span></div><div><span style="color:rgb(80,0,80)">I didn't want to change that for systemd alone as I don't know the reasons for this design -- but I can do that if that's preferred.</span><br></div><div><span style="color:rgb(80,0,80)"><br></span></div><div><span style="color:rgb(80,0,80)">regards,</span></div><div>  Jussi</div></div></div></div>