On Mon, Dec 15, 2014 at 02:51:03PM +0000, Gore, Tim wrote: > > > > -----Original Message----- > > From: Daniel Vetter [mailto:daniel.vetter@xxxxxxxx] On Behalf Of Daniel > > Vetter > > Sent: Monday, December 15, 2014 2:41 PM > > To: Wood, Thomas > > Cc: Gore, Tim; Intel Graphics Development > > Subject: Re: [PATCH i-g-t] Demos/Android.mk: dont build > > intel_sprite_on > > > > On Fri, Dec 12, 2014 at 11:43:55AM +0000, Thomas Wood wrote: > > > On 12 December 2014 at 10:18, <tim.gore@xxxxxxxxx> wrote: > > > > From: Tim Gore <tim.gore@xxxxxxxxx> > > > > > > > > intel_sprite_on wont build on Android, due to use of a particular > > > > API that has changed in Gmin > > > > > > > > Signed-off-by: Tim Gore <tim.gore@xxxxxxxxx> > > > > --- > > > > demos/Android.mk | 6 ++++-- > > > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > > > > > diff --git a/demos/Android.mk b/demos/Android.mk index > > > > 6227e06..be7f3c2 100644 > > > > --- a/demos/Android.mk > > > > +++ b/demos/Android.mk > > > > @@ -4,11 +4,13 @@ LOCAL_PATH := $(call my-dir) > > > > > > > > include $(CLEAR_VARS) > > > > > > > > -LOCAL_SRC_FILES := intel_sprite_on.c > > > > +# This demo wont build on android (from Gmin on). > > > > +#LOCAL_SRC_FILES := intel_sprite_on.c > > > > + > > > > > > > > LOCAL_CFLAGS += -DHAVE_TERMIOS_H > > > > LOCAL_CFLAGS += -DANDROID -UNDEBUG > > > > -LOCAL_CFLAGS += -std=c99 > > > > +LOCAL_CFLAGS += -std=gnu99 > > > > > > I think this change should have been in your previous patch > > > ("Android.mk: replace std=c99 with std=gnu99")? > > > > btw intel_sprite_on has been contributed by your team (Armin Reese), so > > not sure why you want to disable it. Have you checked with display folks > > whether they don't need it? > > -Daniel > > > > Unfortunately this test wont build, out of the box, for Android. I can patch > This locally but I don't think the patch is suitable for upstream. I'm still confused ... why does it not build on Android? I've thought we have this now all worked out wrt igt build issues. Imo better to address whatever's upsetting your builder than to just disable stuff. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
