From 9992bdcbe6b4071c7d04059955171dec183cacfe Mon Sep 17 00:00:00 2001 From: Quentin Glidic Date: Sat, 23 Apr 2016 12:02:58 +0200 Subject: build: Add (and use) an AC_SEARCH_LIBS wrapper AC_SEARCH_LIBS is the recommended macro for these checks, unfortunately, we use AC_CHECK_LIB instead, and even AC_CHECK_FUNC, when only one AC_SEARCH_LIBS would be enough. This wrapper macro is used much like PKG_CHECK_MODULES, as it defines (and AC_SUBST) the PREFIX_LIBS variable itself. It also avoids adding unnecessary stuff to LIBS. Signed-off-by: Quentin Glidic Tested-by: Gustavo Zacarias Reviewed-by: Pekka Paalanen --- configure.ac | 18 +++++------------- 1 file changed, 5 insertions(+), 13 deletions(-) (limited to 'configure.ac') diff --git a/configure.ac b/configure.ac index 670200cf..41996168 100644 --- a/configure.ac +++ b/configure.ac @@ -42,14 +42,10 @@ AC_ARG_VAR([WESTON_SHELL_CLIENT], PKG_PROG_PKG_CONFIG() -AC_CHECK_FUNC([dlopen], [], - AC_CHECK_LIB([dl], [dlopen], DLOPEN_LIBS="-ldl")) -AC_SUBST(DLOPEN_LIBS) +WESTON_SEARCH_LIBS([DLOPEN], [dl], [dlopen]) # In old glibc versions (< 2.17) clock_gettime() is in librt -AC_SEARCH_LIBS([clock_gettime], [rt], - [CLOCK_GETTIME_LIBS="-lrt"]) -AC_SUBST([CLOCK_GETTIME_LIBS]) +WESTON_SEARCH_LIBS([CLOCK_GETTIME], [rt], [clock_gettime]) AC_CHECK_DECL(SFD_CLOEXEC,[], [AC_MSG_ERROR("SFD_CLOEXEC is needed to compile weston")], @@ -303,12 +299,10 @@ AC_ARG_WITH([jpeg], AS_HELP_STRING([--without-jpeg], [Use jpeglib for JPEG decoding support [default=auto]])) AS_IF([test "x$with_jpeg" != "xno"], - [AC_CHECK_LIB([jpeg], [jpeg_CreateDecompress], [have_jpeglib=yes], [have_jpeglib=no])], + [WESTON_SEARCH_LIBS([JPEG], [jpeg], [jpeg_CreateDecompress], [have_jpeglib=yes], [have_jpeglib=no])], [have_jpeglib=no]) AS_IF([test "x$have_jpeglib" = "xyes"], - [JPEG_LIBS="-ljpeg" - AC_SUBST([JPEG_LIBS]) - AC_DEFINE([HAVE_JPEG], [1], [Have jpeglib])], + [AC_DEFINE([HAVE_JPEG], [1], [Have jpeglib])], [AS_IF([test "x$with_jpeg" = "xyes"], [AC_MSG_ERROR([JPEG support explicitly requested, but jpeglib couldn't be found])])]) @@ -418,12 +412,10 @@ AS_IF([test "x$enable_resize_optimization" = "xyes"], AC_ARG_ENABLE(weston-launch, [ --enable-weston-launch],, enable_weston_launch=yes) AM_CONDITIONAL(BUILD_WESTON_LAUNCH, test x$enable_weston_launch == xyes) if test x$enable_weston_launch == xyes; then - AC_CHECK_LIB([pam], [pam_open_session], [have_pam=yes], [have_pam=no]) + WESTON_SEARCH_LIBS([PAM], [pam], [pam_open_session], [have_pam=yes], [have_pam=no]) if test x$have_pam == xno; then AC_ERROR([weston-launch requires pam]) fi - PAM_LIBS=-lpam - AC_SUBST(PAM_LIBS) fi AM_CONDITIONAL(HAVE_PANGO, test "x$have_pango" = "xyes") -- cgit v1.2.1