[Mesa-dev] [PATCH v2 10/21] meson: split and simplify depdendncies

Eric Anholt eric at anholt.net
Thu Oct 12 19:23:19 UTC 2017


Dylan Baker <dylan at pnwbakers.com> writes:

> Rather than group dependencies in complex groups, use a flatter
> structure with split dependencies to avoid checking for the same
> dependencies twice.

meson's going to be caching the dependency checks in the future, so I
wouldn't go out of your way to avoid checking twice, unless it cleans
things up.  In fact, I hope we get to move some of the dependency
checking logic into the subdirs using them, for deps that aren't shared
between many pieces of the build.

However, you've built a lot on top of this patch, and it doesn't hurt,
so I'm fine with it.  A comment below, then r-b.

> @@ -484,10 +515,19 @@ else
>    dep_clock = cc.find_library('rt')
>  endif
>  
> +# TODO: conditionalize libdrm requirement

I think you can drop this comment now.

> +dep_libdrm = dependency('libdrm', version : '>= 2.4.75',
> +                        required : with_dri2 or with_dri3)
> +if dep_libdrm.found()
> +  pre_args += '-DHAVE_LIBDRM'
> +endif
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20171012/2daafa7e/attachment-0001.sig>


More information about the mesa-dev mailing list