Subject: [bug#72397] [PATCH] gnu: Add fennel-ls.

From: Nicolas Graves <[email-masked]>
Date: Thu, 15 Aug 2024 16:32:01 +0200

On 2024-07-31 00:27, sydney goose wrote:

> * gnu/packages/fennel.scm (fennel-ls): New variable.

> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append "https://git.sr.ht/~xerool/fennel-ls/archive/"
> +                           version ".tar.gz"))
> +       (sha256
> +        (base32 "0nb3yclv9v2mwcnam5djvhg0vgmdsk0gavbvq1ar5v2j0m9pzr7f"))))

Is there a reason why we use archives rather than git fetching here?
Otherwise I would advise against, given the codebase is small.
This would probably also solve the two following lints:

[email-masked]: the source file name should contain the package name
[email-masked]: no updater for fennel-ls

> +    (build-system gnu-build-system)
> +    (arguments
> +     (list
> +      #:test-target "test"
> +      #:phases #~(modify-phases %standard-phases
> +                   (delete 'configure)
> +                   (add-after 'build 'patch-fennel-ls
> +                     (lambda* (#:key inputs #:allow-other-keys)
> +                       (substitute* "fennel-ls"
> +                         (("/usr/bin/env .*lua")
> +                          (search-input-file inputs "/bin/lua"))))))
> +      #:make-flags #~(list (string-append "PREFIX="
> +                                          (assoc-ref %outputs "out")))))
> +    (inputs (list lua-5.4 fennel))
> +    (home-page "https://git.sr.ht/~xerool/fennel-ls")
> +    (license license:expat)
> +    (synopsis "Provides intelligent editing features for Fennel files")

Nitpick/proposition : "A language server for Fennel"

> +    (description "A language server for Fennel.  Provides intelligent editing
> +features for Fennel files.")))

Nitpick/proposition : "This package provides a language server for
Fennel that provides intelligent editing features for Fennel files."

Otherwise LGTM. Please send an updated patch ;)

-- 
Best regards,
Nicolas Graves