Re: [Spice-devel] [PATCH spice-gtk] Drop expansions of removed CFLAGS/LIBS variables

2015-05-21 Thread Christophe Fergeau
Hey, On Wed, May 20, 2015 at 07:01:12PM -0400, Benjamin Gilbert wrote: > On 05/20/2015 06:42 PM, Marc-André Lureau wrote: > After the recent comment on this list that celt051 is no longer required, NB: it's optional, support has not been removed. But the bit of code which need celt are all in spi

Re: [Spice-devel] [PATCH spice-gtk] Drop expansions of removed CFLAGS/LIBS variables

2015-05-20 Thread Benjamin Gilbert
On 05/20/2015 06:42 PM, Marc-André Lureau wrote: btw, how did you find them? After the recent comment on this list that celt051 is no longer required, I was grepping to learn the current status of that dependency and noticed that the CELT051_* variables were never set. --Benjamin Gilbert _

Re: [Spice-devel] [PATCH spice-gtk] Drop expansions of removed CFLAGS/LIBS variables

2015-05-20 Thread Marc-André Lureau
ack, and pushed btw, how did you find them? thanks On Wed, May 20, 2015 at 11:39 PM, Benjamin Gilbert wrote: > --- > gtk/Makefile.am | 6 -- > 1 file changed, 6 deletions(-) > > diff --git a/gtk/Makefile.am b/gtk/Makefile.am > index e1d5d93..25e2255 100644 > --- a/gtk/Makefile.am > +++ b/g

[Spice-devel] [PATCH spice-gtk] Drop expansions of removed CFLAGS/LIBS variables

2015-05-20 Thread Benjamin Gilbert
--- gtk/Makefile.am | 6 -- 1 file changed, 6 deletions(-) diff --git a/gtk/Makefile.am b/gtk/Makefile.am index e1d5d93..25e2255 100644 --- a/gtk/Makefile.am +++ b/gtk/Makefile.am @@ -88,15 +88,12 @@ SPICE_COMMON_CPPFLAGS = \ -I$(top_srcdir