<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Sat, Feb 2, 2019, 12:41 PM Eric Engestrom <<a href="mailto:eric.engestrom@intel.com">eric.engestrom@intel.com</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Saturday, 2019-02-02 10:32:15 -0500, Marek Olšák wrote:<br>
> On Sat, Feb 2, 2019, 7:17 AM Eric Engestrom <<a href="mailto:eric.engestrom@intel.com" target="_blank" rel="noreferrer">eric.engestrom@intel.com</a> wrote:<br>
> <br>
> > On Friday, 2019-02-01 15:42:17 -0500, Marek Olšák wrote:<br>
> > > If there is no feedback soon, I'll push this.<br>
> ><br>
> > Have you tested that xcb-randr < 1.12 works?<br>
> > Probably shouldn't remove a restriction unless you're sure it isn't<br>
> > needed :)<br>
> ><br>
> <br>
> Is this a joke? I'm just mirroring autotools. Supporting the same linux<br>
> distributions as autotools is a requirement for meson's general acceptance.<br>
<br>
No, I'm being serious: just because a restriction didn't exist on<br>
autotools doesn't mean that code path was exercised by people running<br>
an old xcb-randr, hence the need to test it :)<br>
<br>
I didn't mean to offend you, I was just asking the question to make sure<br>
this was tested before we claim to support xcb-randr < 1.12, as it might<br>
be that autotools was simply missing the version check.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Ok. I use old xcb-xrandr on some of my systems, one of them used to be my main system. Not being able to use meson on those systems without this patch is a big deal for me.</div><div dir="auto"><br></div><div dir="auto">Marek</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> <br>
> Marek<br>
> <br>
> <br>
> > ><br>
> > > Marek<br>
> > ><br>
> > > On Wed, Jan 30, 2019 at 12:44 PM Erik Faye-Lund <<br>
> > > <a href="mailto:erik.faye-lund@collabora.com" target="_blank" rel="noreferrer">erik.faye-lund@collabora.com</a>> wrote:<br>
> > ><br>
> > > > On Wed, 2019-01-30 at 12:32 -0500, Marek Olšák wrote:<br>
> > > > > ping<br>
> > > > ><br>
> > > ><br>
> > > > Probably worth including Keith, who added this line...<br>
> ><br>
> > Added him.<br>
> ><br>
> > > ><br>
> > > > But yeah, I tend to think that this makes sense. The autotools-build<br>
> > > > doesn't seem to tie this to a specific version, and seems to have been<br>
> > > > used without problems for almost a year. But let's hear what Keith<br>
> > > > things first...<br>
> > > ><br>
> > > > > On Fri, Jan 25, 2019 at 5:48 PM Marek Olšák <<a href="mailto:maraeo@gmail.com" target="_blank" rel="noreferrer">maraeo@gmail.com</a>><br>
> > wrote:<br>
> > > > > > From: Marek Olšák <<a href="mailto:marek.olsak@amd.com" target="_blank" rel="noreferrer">marek.olsak@amd.com</a>><br>
> > > > > ><br>
> > > > > > autotools doesn't have any requirement. This fixes meson on Ubuntu<br>
> > > > > > 16.04.<br>
> > > > > > ---<br>
> > > > > >  meson.build | 2 +-<br>
> > > > > >  1 file changed, 1 insertion(+), 1 deletion(-)<br>
> > > > > ><br>
> > > > > > diff --git a/meson.build b/meson.build<br>
> > > > > > index 34e2a032548..7f16c3070fe 100644<br>
> > > > > > --- a/meson.build<br>
> > > > > > +++ b/meson.build<br>
> > > > > > @@ -1389,21 +1389,21 @@ if with_platform_x11<br>
> > > > > >        dep_xxf86vm = dependency('xxf86vm')<br>
> > > > > >      endif<br>
> > > > > >      dep_glproto = dependency('glproto', version : '>= 1.4.14')<br>
> > > > > >    endif<br>
> > > > > >    if (with_egl or (<br>
> > > > > >        with_gallium_vdpau or with_gallium_xvmc or with_gallium_xa<br>
> > > > > > or<br>
> > > > > >        with_gallium_omx != 'disabled'))<br>
> > > > > >      dep_xcb_xfixes = dependency('xcb-xfixes')<br>
> > > > > >    endif<br>
> > > > > >    if with_xlib_lease<br>
> > > > > > -    dep_xcb_xrandr = dependency('xcb-randr', version : '>= 1.12')<br>
> > > > > > +    dep_xcb_xrandr = dependency('xcb-randr')<br>
> > > > > >      dep_xlib_xrandr = dependency('xrandr', version : '>= 1.3')<br>
> > > > > >    endif<br>
> > > > > >  endif<br>
> > > > > ><br>
> > > > > >  if get_option('gallium-extra-hud')<br>
> > > > > >    pre_args += '-DHAVE_GALLIUM_EXTRA_HUD=1'<br>
> > > > > >  endif<br>
> > > > > ><br>
> > > > > >  _sensors = get_option('lmsensors')<br>
> > > > > >  if _sensors != 'false'<br>
> > > > > > _______________________________________________<br>
> > > > > > mesa-dev mailing list<br>
> > > > > > <a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank" rel="noreferrer">mesa-dev@lists.freedesktop.org</a><br>
> > > > > > <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
> > > ><br>
> > > ><br>
> ><br>
</blockquote></div></div></div>