#f as a package description, gnu: Add rocminfo.

[Christopher Baines <mail@cbaines.net>]

[-- 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 --]