Github messages for voidlinux
 help / color / mirror / Atom feed
From: ahesford <ahesford@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: Re: New package: OpenCL-SDK
Date: Thu, 18 Jan 2024 13:48:49 +0100	[thread overview]
Message-ID: <20240118124849.5497220251@inbox.vuxu.org> (raw)
In-Reply-To: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-48261@inbox.vuxu.org>

[-- Attachment #1: Type: text/plain, Size: 1424 bytes --]

New comment by ahesford on void-packages repository

https://github.com/void-linux/void-packages/pull/48261#issuecomment-1898418994

Comment:
I wouldn't go that far.

The Khronos ICD loader might be superior to ocl-icd (or at least a desirable alternative and we might still want to package it, but we should consider consider whether 1) if ocl-icd will live on in its own right, 2) if so, whether we want to continue support it, and 3) if so, whether we should move all of its dependents to the new option at this point.

If we offer both ICD loaders, they would either need to conflict or present alternatives. If they conflict, we should build all of our packages with whatever the preferred loader is (otherwise, some combination of consumers would be uninstallable). If they offer alternatives, we will need to alter build process.

As for the utilities offered by the SDK repo, my preference for ease of packaging would be to convince upstream to allow building of the extras against "system" versions of its git submodules, allowing us to build the SDK as an add-on. Submodules are kind of a pain in xbps-src, and hard-requiring them as most of the substance of a project seems like strange project management.

If upstream is recalcitrant but we still find a need or desire for the SDK utilities, then I suppose we bite the bullet and subsume the headers and ICD loader into one template as you've proposed.

  parent reply	other threads:[~2024-01-18 12:48 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-17 23:59 [PR PATCH] " Calandracas606
2024-01-18  0:50 ` Calandracas606
2024-01-18  0:59 ` [PR PATCH] [Updated] " Calandracas606
2024-01-18  1:00 ` Calandracas606
2024-01-18  1:07 ` [PR PATCH] [Updated] " Calandracas606
2024-01-18  3:56 ` Calandracas606
2024-01-18  6:41 ` ahesford
2024-01-18 12:32 ` Calandracas606
2024-01-18 12:32 ` [PR PATCH] [Closed]: " Calandracas606
2024-01-18 12:48 ` ahesford
2024-01-18 12:48 ` ahesford [this message]
2024-01-18 13:41 ` Calandracas606
2024-01-18 15:09 ` [PR PATCH] [Updated] " Calandracas606
2024-01-18 15:17 ` Calandracas606
2024-01-18 15:41 ` [PR PATCH] [Updated] " Calandracas606
2024-02-13 23:28 ` Calandracas606
2024-02-13 23:28 ` [PR PATCH] [Closed]: " Calandracas606

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240118124849.5497220251@inbox.vuxu.org \
    --to=ahesford@users.noreply.github.com \
    --cc=ml@inbox.vuxu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).