[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#26702: [PATCH 1/2] gnu: Add font-cns11643
From: |
Alex Kost |
Subject: |
bug#26702: [PATCH 1/2] gnu: Add font-cns11643 |
Date: |
Thu, 04 May 2017 23:03:42 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Marius Bakke (2017-04-30 17:10 +0200) wrote:
>> + (outputs '("out" "tw-kai" "tw-sung"))
>> + (build-system trivial-build-system)
>> + (native-inputs
>> + `(("unzip" ,unzip)))
>> + (arguments
>> + `(#:modules ((guix build utils))
>> + #:builder
>> + (begin
>> + (use-modules (guix build utils))
>> + (let* ((font-dir "/share/fonts/truetype/cns11643")
>> + (out (string-append %output font-dir))
>
> I've changes this to use (assoc-ref outputs "out") instead of the
> magical %output.
I would leave the magical %output :-) Besides (assoc-ref outputs "out")
wouldn't work here; you probably meant (assoc-ref %outputs "out").
Regarding “%output” vs “(assoc-ref %outputs "out")”: I always prefer the
former (Ludovic prefers the latter). Why does “%output” exist anyway,
if not for using it?
--
Alex