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
signature.asc
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev