unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* #f as a package description, gnu: Add rocminfo.
       [not found] ` <20210808145729.2F8D620BD3@vcs0.savannah.gnu.org>
@ 2021-08-08 20:33   ` Christopher Baines
  2021-08-09  7:21     ` Lars-Dominik Braun
  0 siblings, 1 reply; 3+ messages in thread
From: Christopher Baines @ 2021-08-08 20:33 UTC (permalink / raw)
  To: guix-devel

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


guix-commits@gnu.org writes:

> lbraun pushed a commit to branch master
> in repository guix.
>
> commit 91ce17a53236578f8055a2588460047741983925
> Author: Lars-Dominik Braun <lars@6xq.net>
> AuthorDate: Fri Aug 6 08:29:12 2021 +0200
>
>     gnu: Add rocminfo.
>
>     * gnu/packages/rocm.scm (rocminfo): New variable.
> ---
>  gnu/packages/rocm.scm | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
>
> diff --git a/gnu/packages/rocm.scm b/gnu/packages/rocm.scm
> index 07e395c..0cf5a34 100644
> --- a/gnu/packages/rocm.scm
> +++ b/gnu/packages/rocm.scm
> @@ -304,3 +304,24 @@ allows runtimes to work on Windows as well as on Linux without much effort.")
>      (description "OpenCL 2.0 compatible language runtime, supporting offline
>  and in-process/in-memory compilation.")
>      (license license:ncsa)))
> +
> +(define-public rocminfo
> +  (package
> +    (name "rocminfo")
> +    (version %rocm-version)
> +    (source (origin
> +              (method git-fetch)
> +              (uri (git-reference
> +                    (url "https://github.com/RadeonOpenCompute/rocminfo.git")
> +                    (commit (string-append "rocm-" version))))
> +              (file-name (git-file-name name version))
> +              (sha256
> +               (base32
> +                "0pcm308vwkjrwnrk507iya20mkil8j0vx699w9jk2gas4n4jvkcz"))))
> +    (build-system cmake-build-system)
> +    (arguments `(#:tests? #f)) ; No tests.
> +    (inputs `(("rocr-runtime" ,rocr-runtime)))
> +    (home-page "https://github.com/RadeonOpenCompute/rocminfo")
> +    (synopsis "ROCm Application for Reporting System Info")
> +    (description #f)
> +    (license license:ncsa)))

guix lint notes:
gnu/packages/rocm.scm:309:2: rocminfo@4.3.0: invalid description: #f

I noticed this because it broke an assumption about descriptions in the
Guix Data Service [1].

1: https://data.guix.gnu.org/revision/e81cf4e79a6e297db0ae2a9c39eab495e7e204f0

As this is now part of the history, I'll fix the Guix Data Service to be
able to cope when the description is #f.

Anyway, I wouldn't like for this change to lower the standard though,
it's currently the only package in Guix with an invalid description (as
far as I'm aware), is there some reason why it doesn't have one?

Thanks,

Chris

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 987 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: #f as a package description, gnu: Add rocminfo.
  2021-08-08 20:33   ` #f as a package description, gnu: Add rocminfo Christopher Baines
@ 2021-08-09  7:21     ` Lars-Dominik Braun
  2021-08-09 21:44       ` Christopher Baines
  0 siblings, 1 reply; 3+ messages in thread
From: Lars-Dominik Braun @ 2021-08-09  7:21 UTC (permalink / raw)
  To: Christopher Baines; +Cc: guix-devel

Hi Christopher,

> Anyway, I wouldn't like for this change to lower the standard though,
> it's currently the only package in Guix with an invalid description (as
> far as I'm aware), is there some reason why it doesn't have one?
it simply fell through the cracks[1]. Commit
0a379de3249d5e9ff66fb404f7e5aa8ce2cb3d24 adds a proper descripton.

Sorry for the trouble,
Lars

[1] Unfortunately I cannot run `guix lint` on an entire git changeset,
so instead I have to check each package by hand and I probably missed
rocminfo. I wish someday we can have a branch/pull-request-based
workflow with automated CI checks (linting, `guix pull`, signature
verification, …) *before* merging to master.



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: #f as a package description, gnu: Add rocminfo.
  2021-08-09  7:21     ` Lars-Dominik Braun
@ 2021-08-09 21:44       ` Christopher Baines
  0 siblings, 0 replies; 3+ messages in thread
From: Christopher Baines @ 2021-08-09 21:44 UTC (permalink / raw)
  To: Lars-Dominik Braun; +Cc: guix-devel

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


Lars-Dominik Braun <lars@6xq.net> writes:

> Hi Christopher,
>
>> Anyway, I wouldn't like for this change to lower the standard though,
>> it's currently the only package in Guix with an invalid description (as
>> far as I'm aware), is there some reason why it doesn't have one?
> it simply fell through the cracks[1]. Commit
> 0a379de3249d5e9ff66fb404f7e5aa8ce2cb3d24 adds a proper descripton.
>
> Sorry for the trouble,

No problem, thanks for fixing it.

> [1] Unfortunately I cannot run `guix lint` on an entire git changeset,
> so instead I have to check each package by hand and I probably missed
> rocminfo. I wish someday we can have a branch/pull-request-based
> workflow with automated CI checks (linting, `guix pull`, signature
> verification, …) *before* merging to master.

This is something I've put some time in to, but it needs some general
support before I'm confident that the things I have in mind can become
reality. I recently started a thread on guix-devel [1] about what I have
in mind, so please comment if you're interested in this area.

1: https://lists.gnu.org/archive/html/guix-devel/2021-08/msg00001.html

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 987 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-08-09 21:44 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20210808145723.7941.35341@vcs0.savannah.gnu.org>
     [not found] ` <20210808145729.2F8D620BD3@vcs0.savannah.gnu.org>
2021-08-08 20:33   ` #f as a package description, gnu: Add rocminfo Christopher Baines
2021-08-09  7:21     ` Lars-Dominik Braun
2021-08-09 21:44       ` Christopher Baines

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).