GNU bug report logs - #71319
[PATCH 0/2] gnu: emacs-ddskk: Update to 17.1 revision 0.

Previous Next

Full log

View this message in rfc822 format

From: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
To: gemmaro <gemmaro.dev <at> gmail.com>
Cc: 71319 <at> debbugs.gnu.org, Katherine Cox-Buday <cox.katherine.e+guix <at> gmail.com>, Liliana Marie Prikler <liliana.prikler <at> gmail.com>, Andrew Tropin <andrew <at> trop.in>
Subject: [bug#71319] [PATCH v2 2/3] gnu: emacs-ddskk: Improve package style.
Date: Wed, 19 Jun 2024 13:16:35 +0200

Hello,

gemmaro <gemmaro.dev <at> gmail.com> writes:

> * gnu/packages/emacs-xyz.scm (emacs-ddskk)[arguments]: Use newer
> package style.

Thanks. Some comments follow.

> +       (list #:modules '((guix build gnu-build-system)
> +                         (guix build utils)
> +                         (guix build emacs-utils))
> +             #:imported-modules `(,@%gnu-build-system-modules
> +                                  (guix build emacs-utils))
> +             #:test-target "test"
> +             #:phases
> +             #~(modify-phases %standard-phases
> +                 (replace 'configure
> +                   (lambda* (#:key outputs #:allow-other-keys)

Use (lambda _ ...) here.

> +                     (make-file-writable "SKK-MK")

I don't think this line is necessary. Could you double-check?

> +                     (emacs-substitute-variables "SKK-MK"
> +                       ("PREFIX" (assoc-ref outputs "out"))

(assoc-ref outputs "out") -> #$output

> +                       ("LISPDIR" '(expand-file-name "/share/emacs/site-lisp" PREFIX))
> +                       ("SKK_PREFIX" "")
> +                       ("SKK_INFODIR" '(expand-file-name "info" PREFIX)))
> +                     (for-each make-file-writable (find-files "./doc"))
> +                     #t))

Could you remove this trailing #T, too?

Regards,
-- 
Nicolas Goaziou

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.