[Intel-gfx] [PATCH i-g-t] igt: Add LOCAL defines for distro compatibility
Arkadiusz Hiler
arkadiusz.hiler at intel.com
Tue Jul 11 15:33:13 UTC 2017
On Tue, Jul 11, 2017 at 04:12:18PM +0100, Emil Velikov wrote:
> Hi Arkadiusz,
>
> On 11 July 2017 at 12:48, Arkadiusz Hiler <arkadiusz.hiler at intel.com> wrote:
> > Each Linux distro takes a different spin on providing kernel's uapi
> > headers (especialy the *drm*.h).
> >
> > You can get them with linux-headers, you can get them with libdrm.
> > Sometime you can even get them twice, from both sources.
> >
> Your distro should _not_ provide the files with linux-headers, exactly
> for the reasons mentioned.
> Please speak with your distro to fix their packaging.
That fix is not actually for me. I am just maintaining stuff.
My distors are fine, but the header thing seems to be a common headache.
> > Sometimes the headers match your kernel version, sometimes you end up
> > stuck with headers matching the kernel that the given release of the
> > distro came out with.
> >
> > This makes things harder for code that does not depend on libdrm, as we
> > cannot have sane ./configure-time checks.
> >
> > So let's define LOCAL_ version for FENCE and EXECBUFFER2_WB defines
> > until all the stable distros catch up (+/- some epsilon).
> >
> One could bump the libdrm version in configure, but some distros (same
> one as the one doing linux-headers goofups?) are slow to ship new
> versions.
The version is correct and it is in effect for components that do depend
on it.
This is for the kernel uapi headers which suffer for being both in
kernel-headers (some distro butcher them out though) and in
libdrm-dev(el).
> Please consider voting for the package so it can be updated in timely
> manner, or even lend packagers a hand ;-)
> Note this is _not_ an objection to the patch. Adding [temporary] local
> makes perfect sense.
>
> FWIW
> Reviewed-by: Emil Velikov <emil.l.velikov at gmail.com>
Thanks, I'll push it in a moment.
--
Cheers,
Arek
More information about the Intel-gfx
mailing list