KhronosGroup / OpenCL-ICD-Loader

The OpenCL ICD Loader project.
Apache License 2.0
246 stars 136 forks source link

Spack package #212

Open koysean opened 1 year ago

koysean commented 1 year ago

Hello, has any work been done on creating a Spack package for the OpenCL ICD Loader? It would be helpful to have as there are many Spack packages (such as double-batched-fft-library) which depend on OpenCL but there is no package for the Khronos loader. I am happy to assist if needed. Thank you!

bashbaug commented 1 year ago

Hello! I'm not aware of any work that has been done for Spack packages for this repo.

@Kerilk do you know if this has been done for the ocl-icd loader?

Regardless, if this is something you could help with it would be much appreciated - thanks!

Kerilk commented 1 year ago

@Kerilk do you know if this has been done for the ocl-icd loader?

ocl-icd is indeed packaged in spack: https://github.com/spack/spack/tree/develop/var/spack/repos/builtin/packages/ocl-icd/ I am not the maintainer though.

koysean commented 1 year ago

Thanks for the response. I saw the ocl-icd Spack package while looking for a Spack package for this repo. I'm not clear on the relationship between this project and that one as they appear to be very similar - could I get some clarity on that?

Kerilk commented 1 year ago

ocl-icd is a maintained drop in replacement for the OpenCL loader which was created when this one was not open sourced. It is still maintained as it exposes a few configuration options that are not supported by the official loader (or with different name and syntax), and some project depend on these.

In practice, it should work fine in your case.

koysean commented 1 year ago

Thank you - it seems like the ocl-icd package is sufficient for my purposes (and if I had to guess, for most users) since I'm not building from source, just pointing to an external installation. If a need arises to have a distinct package for this repo I'd be happy to provide input, but it doesn't seem to be the case at the moment.

Thanks again for the help!