Re: [client] build-sys: Prefix the configure audio GStreamer variables with GSTAUDIO_

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Mon, Nov 23, 2015 at 05:33:56PM +0100, Francois Gouget wrote:
> This lets us avoid naming conflicts when using GStreamer for other
> purposes.
>

Yes, I remember to ack those but it didn't get upstream yet due the
video part.

Acked-by: Victor Toso <victortoso@xxxxxxxxxx>

I'll be pushing this shortly.

> Signed-off-by: Francois Gouget <fgouget@xxxxxxxxxxxxxxx>
> ---
>  configure.ac    | 18 +++++++++---------
>  src/Makefile.am |  4 ++--
>  2 files changed, 11 insertions(+), 11 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index db2891f..dea9a30 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -298,7 +298,7 @@ AC_ARG_WITH([audio],
>  
>  case "$with_audio" in
>    gstreamer|pulse|auto*)
> -    PKG_CHECK_MODULES(GST, gstreamer-1.0 gstreamer-base-1.0 gstreamer-app-1.0 gstreamer-audio-1.0, [have_gst=yes], [have_gst=no])
> +    PKG_CHECK_MODULES(GSTAUDIO, gstreamer-1.0 gstreamer-base-1.0 gstreamer-app-1.0 gstreamer-audio-1.0, [have_gstaudio=yes], [have_gstaudio=no])
>      PKG_CHECK_MODULES(PULSE, libpulse libpulse-mainloop-glib, [have_pulse=yes], [have_pulse=no])
>      ;;
>    no*)
> @@ -309,7 +309,7 @@ esac
>  AS_IF([test "x$with_audio" = "xauto" && test "x$have_pulse" = "xyes"],
>        [with_audio=pulse])
>  
> -AS_IF([test "x$with_audio" = "xauto" && test "x$have_gst" = "xyes"],
> +AS_IF([test "x$with_audio" = "xauto" && test "x$have_gstaudio" = "xyes"],
>        [with_audio=gstreamer])
>  
>  AS_IF([test "x$with_audio" = "xauto"],
> @@ -326,14 +326,14 @@ AC_SUBST(PULSE_CFLAGS)
>  AC_SUBST(PULSE_LIBS)
>  
>  AS_IF([test "x$with_audio" = "xgstreamer"],
> -      [AS_IF([test "x$have_gst" = "xyes"],
> -             [AC_DEFINE([WITH_GSTAUDIO], 1, [Have GStreamer 1.0?])],
> -             [AC_MSG_ERROR([GStreamer 1.0 requested but not found])
> +      [AS_IF([test "x$have_gstaudio" = "xyes"],
> +             [AC_DEFINE([WITH_GSTAUDIO], 1, [Have GStreamer 1.0 audio?])],
> +             [AC_MSG_ERROR([GStreamer 1.0 audio requested but not found])
>        ])
>  ])
> -AM_CONDITIONAL([WITH_GSTAUDIO], [test "x$have_gst" = "xyes"])
> -AC_SUBST(GST_CFLAGS)
> -AC_SUBST(GST_LIBS)
> +AM_CONDITIONAL([WITH_GSTAUDIO], [test "x$have_gstaudio" = "xyes"])
> +AC_SUBST(GSTAUDIO_CFLAGS)
> +AC_SUBST(GSTAUDIO_LIBS)
>  
>  AC_CHECK_LIB(jpeg, jpeg_destroy_decompress,
>      AC_MSG_CHECKING([for jpeglib.h])
> @@ -697,7 +697,7 @@ SPICE_CFLAGS="$SPICE_CFLAGS $WARN_CFLAGS"
>  
>  AC_SUBST(SPICE_CFLAGS)
>  
> -SPICE_GLIB_CFLAGS="$PIXMAN_CFLAGS $PULSE_CFLAGS $GST_CFLAGS $GLIB2_CFLAGS $GIO_CFLAGS $GOBJECT2_CFLAGS $SSL_CFLAGS $SASL_CFLAGS"
> +SPICE_GLIB_CFLAGS="$PIXMAN_CFLAGS $PULSE_CFLAGS $GSTAUDIO_CFLAGS $GLIB2_CFLAGS $GIO_CFLAGS $GOBJECT2_CFLAGS $SSL_CFLAGS $SASL_CFLAGS"
>  SPICE_GTK_CFLAGS="$SPICE_GLIB_CFLAGS $GTK_CFLAGS "
>  
>  AC_SUBST(SPICE_GLIB_CFLAGS)
> diff --git a/src/Makefile.am b/src/Makefile.am
> index 0c40c48..99a8a1f 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -97,7 +97,7 @@ SPICE_COMMON_CPPFLAGS =						\
>  	$(GOBJECT2_CFLAGS)					\
>  	$(SSL_CFLAGS)						\
>  	$(SASL_CFLAGS)						\
> -	$(GST_CFLAGS)						\
> +	$(GSTAUDIO_CFLAGS)					\
>  	$(SMARTCARD_CFLAGS)					\
>  	$(USBREDIR_CFLAGS)					\
>  	$(GUDEV_CFLAGS)						\
> @@ -210,7 +210,7 @@ libspice_client_glib_2_0_la_LIBADD =					\
>  	$(PIXMAN_LIBS)							\
>  	$(SSL_LIBS)							\
>  	$(PULSE_LIBS)							\
> -	$(GST_LIBS)							\
> +	$(GSTAUDIO_LIBS)						\
>  	$(SASL_LIBS)							\
>  	$(SMARTCARD_LIBS)						\
>  	$(USBREDIR_LIBS)						\
> -- 
> 2.6.2
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]