So, the issue is that i-g-t will not build on our builder system, with the current version of librm-intel. It just happens to work because I took the liberity of upgrading the entire system, a while ago. No build, no newer i-g-t, no tests, no CI. There are a few of _PINNED macros defined by Chris all over the gem tests. Thought of doing just that, figured it would be better to bump the library instead. On Thu, Jul 14, 2016 at 05:29:55PM +0100, Emil Velikov wrote: > Hi Marius, > > Just double-checking - this is an i-g-t patch, isn't it ? Sorry, my bad! > > On 14 July 2016 at 11:39, Marius Vlad <marius.c.vlad@xxxxxxxxx> wrote: > > Required by commit 2603b98ca (aubdump: Support softpin bos). > > > > Signed-off-by: Marius Vlad <marius.c.vlad@xxxxxxxxx> > > CC: Kristian Høgsberg Kristensen <kristian.h.kristensen@xxxxxxxxx> > > --- > > configure.ac | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/configure.ac b/configure.ac > > index f05bcb0..ade9756 100644 > > --- a/configure.ac > > +++ b/configure.ac > > @@ -119,7 +119,7 @@ if test "x$GCC" = "xyes"; then > > fi > > AC_SUBST(ASSEMBLER_WARN_CFLAGS) > > > > -PKG_CHECK_MODULES(DRM, [libdrm_intel >= 2.4.64 libdrm]) > > +PKG_CHECK_MODULES(DRM, [libdrm_intel >= 2.4.68 libdrm]) > Yes please. As you do that one can nuke most/all the "define LOCAL_" > and "struct local_" (in lib/ioctl_wrappers.h) > and replace with with the official symbols. A very nice plus imho ;-) > > Side note: this will conflict with Robert Foss's work to make > libdrm_intel an optional dependency. Have you/others had the chance to > look at the series ? What can we do to get that moving/accepted ? Yes I've seen them. Someone to Ack'em :-). > > Thanks > Emil
Attachment:
signature.asc
Description: Digital signature
_______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
