On 2019-01-08 — 15:18, Dylan Baker wrote:
[snip]
> >    dep_clc = dependency('libclc')
> > +  dep_spirv_tools = dependency('SPIRV-Tools', required : false, version : 
> > '>= 2018.0')
> > +  # LLVMSPIRVLib is available at 
> > https://github.com/KhronosGroup/SPIRV-LLVM-Translator
> > +  dep_llvmspirvlib = dependency('LLVMSPIRVLib', required : false, version 
> > : '>= 0.2.1')
> > +  if dep_spirv_tools.found() and dep_llvmspirvlib.found()
> > +    pre_args += '-DCLOVER_ALLOW_SPIRV'
> > +  endif
> 
> As-is this is what distros call "automagical", and they don't like it :). 
> There
> is a case like this is for xlib-lease as well, I'd really like this to have an
> option to control it, something similar to the way that xlib-lease works.
> 
> Dylan
[snip]

Absolutely! I can’t remember why I didn’t went that way. I’ll prepare a v11
that only looks for the dependencies if an enable flag (I’m thinking
`gallium_opencl_spirv` for Meson and `enable-opencl-spirv` for autotools) is
specified, and the dependencies would now be mandatory.

Pierre

Attachment: signature.asc
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to