mbox series

[bug#70252,v2,0/5] Update and restyle Khronos OpenCL packages

Message ID cover.1712521108.git.a.r.draidi@redscript.org
Headers show
Series Update and restyle Khronos OpenCL packages | expand

Message

Ahmad Draidi April 7, 2024, 8:19 p.m. UTC
Hello Guix,

Version 2 of this patch series uses a common version variable.

While working on Vulkan packages, I noticed the common version variable
%vulkan-sdk-version so I thought I'd use the same pattern here. Upstream
seems to be unified or unifying also[1].

I avoided using the full version string with the "v" included (e.g. v2023.12.14)
to minimize changes, though it might be more future proof to use the whole string.

[1]: https://github.com/KhronosGroup/OpenCL-SDK

Ahmad Draidi (5):
  gnu: opencl-headers, opencl-clhpp and opencl-icd-loader: Replace
    version with %opencl-sdk-version.
  gnu: %opencl-sdk-version: Update to 2023.12.14.
  gnu: opencl-clhpp: Update package style.
  gnu: opencl-headers: Update package style.
  gnu: opencl-icd-loader: Update package style.

 gnu/packages/opencl.scm | 42 +++++++++++++++++++++++------------------
 1 file changed, 24 insertions(+), 18 deletions(-)


base-commit: 298aed72a2a76be33f9a55bed22636acd7a4f9b9

Comments

Ludovic Courtès May 4, 2024, 4:40 p.m. UTC | #1
Hello Ahmad,

Ahmad Draidi <a.r.draidi@redscript.org> skribis:

>   gnu: opencl-headers, opencl-clhpp and opencl-icd-loader: Replace
>     version with %opencl-sdk-version.
>   gnu: %opencl-sdk-version: Update to 2023.12.14.
>   gnu: opencl-clhpp: Update package style.
>   gnu: opencl-headers: Update package style.
>   gnu: opencl-icd-loader: Update package style.

I think this is very much under the scope of the “MESA team”, so I’m
Cc’ing John who’s more familiar with this area than I am.

  https://issues.guix.gnu.org/70252

(BTW, there’s actually no “MESA team” in etc/teams.scm.  Should we add
one, John?)

Ludo’.