Subject: Re: [bug#55960] [PATCH 5/9] gnu: Add node-ieee754.

From: Marius Bakke <[email-masked]>
Date: Thu, 23 Jun 2022 23:07:20 +0200

Nicolas Graves via Guix-patches via <[email-masked]> skriver:

> * gnu/packages/node-xyz.scm (node-ieee754): New variable.
> ---
>  gnu/packages/node-xyz.scm | 31 +++++++++++++++++++++++++++++++
>  1 file changed, 31 insertions(+)
>
> diff --git a/gnu/packages/node-xyz.scm b/gnu/packages/node-xyz.scm
> index 8f3288f219..3d945871cd 100644
> --- a/gnu/packages/node-xyz.scm
> +++ b/gnu/packages/node-xyz.scm
> @@ -458,6 +458,37 @@ (define-public node-once
>  if desired.")
>      (license license:isc)))
>  
> +(define-public node-ieee754
> +  (package
> +    (name "node-ieee754")
> +    (version "1.2.1")
> +    (source
> +     (origin
> +       (method git-fetch)
> +       (uri (git-reference
> +             (url "https://github.com/feross/ieee754")
> +             (commit (string-append "v" version))))
> +       (file-name (git-file-name name version))
> +       (sha256
> +          (base32
> +           "19rlg59lavnwsvbblhvrqwinz2wzqlxhddqpwrc3cyqkscjgza7i"))))
> +    (build-system node-build-system)
> +    (arguments
> +     '(#:phases
> +       (modify-phases %standard-phases
> +         (delete 'check)
> +         (replace 'configure
> +           (lambda* (#:key inputs #:allow-other-keys)
> +             (invoke (string-append (assoc-ref inputs "node") "/bin/npm")
> +                     "--offline" "--ignore-scripts" "install" "--production")
> +             #t)))))

I realize all of these were copy-pasted so I won't repeat any more.  ;-)

> +    (home-page "https://github.com/feross/ieee754")
> +    (synopsis "Read/write IEEE754 floating point numbers from/to a Buffer or
> +array-like object")
> +    (description "This package allows to read/write IEEE754 floating point
> +numbers from/to a Buffer or array-like object in Javascript.")

Can you try to either shorten the synopsis or expand on the description
to avoid duplication?

Writing good descriptions is often the most difficult part of writing a
package definition...  The upstream documentation is not very helpful in
this case.  :-/