[Spice-devel] [PATCH 1/3] build-sys: Use spice-common m4 macro for smartcard
Victor Toso
victortoso at redhat.com
Mon Jun 22 09:19:03 PDT 2015
On Mon, Jun 22, 2015 at 06:17:35PM +0200, Victor Toso wrote:
> Hey,
>
> This series and moving the --enable-lz4 to spice-common looks good to
> me.
+ version check on lz4 (> 118)
>
> - toso
>
> On Thu, Jun 18, 2015 at 11:58:28AM +0200, Christophe Fergeau wrote:
> > Besides the code factorization, this will allow smartcard support to be
> > automatically enabled if libcacard is present and --disable-smartcard is
> > not used.
> > ---
> > configure.ac | 24 ++++++------------------
> > 1 file changed, 6 insertions(+), 18 deletions(-)
> >
> > diff --git a/configure.ac b/configure.ac
> > index 2aaf88d..e17911e 100644
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -29,7 +29,7 @@ fi
> > SPICE_SERVER_VERSION=`printf "0x%02x%02x%02x" $major $minor $micro`
> > AC_SUBST(SPICE_SERVER_VERSION)
> >
> > -AC_CONFIG_MACRO_DIR([m4])
> > +AC_CONFIG_MACRO_DIRS([m4 spice-common/m4])
> > AM_CONFIG_HEADER([config.h])
> > AC_CONFIG_AUX_DIR(.)
> >
> > @@ -84,14 +84,8 @@ AC_ARG_ENABLE([lz4],
> > AS_IF([test x"$enable_lz4" != "xno"], [enable_lz4="yes"])
> > AM_CONDITIONAL(SUPPORT_LZ4, test "x$enable_lz4" = "xyes")
> >
> > -AC_ARG_ENABLE([smartcard],
> > - AS_HELP_STRING([--enable-smartcard], [Enable network redirection]),,
> > - [enable_smartcard="no"])
> > -AS_IF([test x"$enable_smartcard" != "xno"], [enable_smartcard="yes"])
> > -AM_CONDITIONAL(SUPPORT_SMARTCARD, test "x$enable_smartcard" != "xno")
> > -if test "x$enable_smartcard" = "xyes"; then
> > - AC_DEFINE([USE_SMARTCARD], [1], [Define if supporting smartcard proxying])
> > -fi
> > +SPICE_CHECK_SMARTCARD([SMARTCARD])
> > +AM_CONDITIONAL(SUPPORT_SMARTCARD, test "x$have_smartcard" = "xyes")
> >
> > AC_ARG_ENABLE([automated_tests],
> > AS_HELP_STRING([--enable-automated-tests], [Enable automated tests using spicy-screenshot (part of spice--gtk)]),,
> > @@ -120,15 +114,9 @@ AS_VAR_APPEND([SPICE_NONPKGCONFIG_LIBS], [" -pthread $LIBM $LIBRT"])
> >
> > SPICE_REQUIRES=""
> >
> > -if test "x$enable_smartcard" = "xyes"; then
> > - PKG_CHECK_MODULES(CAC_CARD, libcacard >= 0.1.2)
> > - SMARTCARD_LIBS="$CAC_CARD_LIBS"
> > - SMARTCARD_CFLAGS="$CAC_CARD_CFLAGS"
> > - AC_SUBST(SMARTCARD_LIBS)
> > - AC_SUBST(SMARTCARD_CFLAGS)
> > +AS_IF([test x"$have_smartcard" = "xyes"], [
> > AS_VAR_APPEND([SPICE_REQUIRES], [" libcacard >= 0.1.2"])
> > -fi
> > -
> > +])
> >
> > PKG_CHECK_MODULES([GLIB2], [glib-2.0 >= 2.22])
> > AS_VAR_APPEND([SPICE_REQUIRES], [" glib-2.0 >= 2.22"])
> > @@ -359,7 +347,7 @@ echo "
> >
> > LZ4 support: ${enable_lz4}
> >
> > - Smartcard: ${enable_smartcard}
> > + Smartcard: ${have_smartcard}
> >
> > SASL support: ${enable_sasl}
> >
> > --
> > 2.4.3
> >
> > _______________________________________________
> > Spice-devel mailing list
> > Spice-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/spice-devel
More information about the Spice-devel
mailing list