On Fri, Aug 14, 2015 at 04:08:35PM +0200, Christophe Fergeau wrote:
> On Fri, Aug 14, 2015 at 03:27:12PM +0200, Victor Toso wrote:
> > Hi,
> >
> > On Fri, Aug 07, 2015 at 12:49:22PM +0200, Christophe Fergeau wrote:
> > > It's seeing regular releases and is API stable, so we don't need to
> > > bundle it with spice-gtk
> > > ---
> > > configure.ac | 8 +++++---
> > > spice-common | 2 +-
> > > 2 files changed, 6 insertions(+), 4 deletions(-)
> > >
> > > diff --git a/configure.ac b/configure.ac
> > > index 1b347c0..855069f 100644
> > > --- a/configure.ac
> > > +++ b/configure.ac
> > > @@ -73,7 +73,9 @@ AC_CHECK_LIBM
> > > AC_SUBST(LIBM)
> > >
> > > AC_CONFIG_SUBDIRS([spice-common])
> > > -COMMON_CFLAGS='-I ${top_srcdir}/spice-common/ -I ${top_srcdir}/spice-common/spice-protocol/'
> > > +PKG_CHECK_MODULES([PROTOCOL], [spice-protocol >= 0.12.10])
> > > +
> >
> > I would go with SPICE_PROTOCOL just to be consistent with spice server
> > as well [0] but feel free to ignore this ;)
> > [0] http://lists.freedesktop.org/archives/spice-devel/2015-August/021516.html
>
> Ah sure, initially I wanted to keep the name used in SPICE_GLIB_CFLAGS
> below, to make the diff shorter, but in the end PROTOCOL_CFLAGS need to
> be in COMMON_CFLAGS because of src/controller/ (unless I patch
> src/controller/Makefile.am too).
Pushed with this change.
Christophe
Attachment:
pgp9VHP6Ed_Ww.pgp
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel