[PATCH] Supply all code using dl*() with DLOPEN_LIBS

Dan Nicholson dbn.lists at gmail.com
Thu Oct 29 06:06:10 PDT 2009


On Wed, Oct 28, 2009 at 4:34 PM, Mikhail Gusarov
<dottedmag at dottedmag.net> wrote:
> Previously DLOPEN_LIBS was managed in top-level configure.ac.
> Instead bundle it with the code using dl*() functions to
> avoid breakages in uncommon configurations.
>
> Signed-off-by: Mikhail Gusarov <dottedmag at dottedmag.net>
> ---
>  configure.ac                  |    5 +++--
>  glx/Makefile.am               |    4 ++++
>  hw/xfree86/loader/Makefile.am |    1 +
>  os/Makefile.am                |    2 +-
>  4 files changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 53342c5..2c3b0b8 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -195,6 +195,7 @@ dnl Check to see if dlopen is in default libraries (like Solaris, which
>  dnl has it in libc), or if libdl is needed to get it.
>  AC_CHECK_FUNC([dlopen], [],
>        AC_CHECK_LIB([dl], [dlopen], DLOPEN_LIBS="-ldl"))
> +AC_SUBST(DLOPEN_LIBS)
>
>  dnl Checks for library functions.
>  AC_FUNC_VPRINTF
> @@ -922,7 +923,7 @@ if test "x$GLX" = xyes; then
>        AC_SUBST(XLIB_CFLAGS)
>        AC_DEFINE(GLXEXT, 1, [Build GLX extension])
>        GLX_LIBS='$(top_builddir)/glx/libglx.la'
> -       GLX_SYS_LIBS="$GLX_SYS_LIBS $DLOPEN_LIBS"
> +       GLX_SYS_LIBS="$GLX_SYS_LIBS"
>  else
>         GLX=no
>  fi
> @@ -1513,7 +1514,7 @@ if test "x$XORG" = xyes; then
>        AC_CHECK_FUNCS([pci_device_vgaarb_init])
>        LIBS=$SAVE_LIBS
>        CFLAGS=$SAVE_CFLAGS
> -       XORG_SYS_LIBS="$XORG_SYS_LIBS $PCIACCESS_LIBS $DLOPEN_LIBS $GLX_SYS_LIBS $SELINUX_LIB"
> +       XORG_SYS_LIBS="$XORG_SYS_LIBS $PCIACCESS_LIBS $GLX_SYS_LIBS $SELINUX_LIB"
>        XORG_CFLAGS="$XORG_CFLAGS $PCIACCESS_CFLAGS"
>
>        case $host_os in
> diff --git a/glx/Makefile.am b/glx/Makefile.am
> index a23ae0a..3a83770 100644
> --- a/glx/Makefile.am
> +++ b/glx/Makefile.am
> @@ -58,6 +58,8 @@ if DRI2_AIGLX
>  libglxdri_la_SOURCES += glxdri2.c
>  endif
>
> +libglxdri_la_LIBADD = @DLOPEN_LIBS@

Is there a reason not to just use the variable $(DLOPEN_LIBS)? It will
already be in the Makefile since it's AC_SUBST'd. This just makes it
so -ldl is unnecessarily hardcoded into another variable. I know
you're not the first one to do this. It seems to be a popular exercise
in the server.

Otherwise,
Reviewed-by: Dan Nicholson <dbn.lists at gmail.com>

--
Dan


More information about the xorg-devel mailing list