[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/1] gnu: Add Anonymous Pro fonts.
From: |
Alex Kost |
Subject: |
Re: [PATCH 1/1] gnu: Add Anonymous Pro fonts. |
Date: |
Sat, 05 Sep 2015 16:34:16 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Leo Famulari (2015-09-04 00:53 +0300) wrote:
> * gnu/packages/fonts.scm (font-anonymous-pro): New variable.
> ---
> gnu/packages/fonts.scm | 42 ++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 42 insertions(+)
>
>
> diff --git a/gnu/packages/fonts.scm b/gnu/packages/fonts.scm
> index a78995c..75e6fe8 100644
> --- a/gnu/packages/fonts.scm
> +++ b/gnu/packages/fonts.scm
> @@ -524,3 +524,45 @@ distributed with Ghostscript version 4.00. The
> collection contains the
> following fonts in the OpenType format: Adventor, Bonum, Chorus, Cursor,
> Heros, Pagella, Schola, Termes.")
> (license license:gfl1.0)))
> +
> +(define-public font-anonymous-pro
> + (package
> + (name "font-anonymous-pro")
> + (version "1.002")
> + (source (origin
> + (method url-fetch)
> + (uri (string-append
> "http://www.marksimonson.com/assets/content/fonts/"
This line is too long, we prefer to stay within 78 chars. I would make it:
(string-append
"http://www.marksimonson.com/assets/content/fonts/"
> + "AnonymousPro-" version ".zip"))
> + (sha256
> + (base32
> "1asj6lykvxh46czbal7ymy2k861zlcdqpz8x3s5bbpqwlm3mhrl6"))))
Such (base32 "…") lines also look too long for me, so I prefer to move a
hash on another line (but some people leave it this way).
> + (build-system trivial-build-system)
> + (arguments
> + `(#:modules ((guix build utils))
> + #:builder
> + (begin
> + (use-modules (guix build utils))
> + (let ((unzip (string-append (assoc-ref %build-inputs "unzip")
> + "/bin/unzip"))
> + (font-dir (string-append %output "/share/fonts/truetype"))
> + (doc-dir (string-append %output "/share/doc/" ,name)))
> + (system* unzip (assoc-ref %build-inputs "source"))
> + (mkdir-p font-dir)
> + (mkdir-p doc-dir)
> + (chdir (string-append "AnonymousPro-" ,version ".001"))
> + (for-each (lambda (ttf)
> + (copy-file ttf
> + (string-append font-dir "/" ttf)))
> + (find-files "." "\\.ttf$"))
> + (for-each (lambda (doc)
> + (copy-file doc
> + (string-append doc-dir "/" doc)))
> + (find-files "." "\\.txt$"))))))
This (find-files …) should be shifted 2 chars left (the same as
'find-files' in the previous 'for-each').
Otherwise, LGTM. So if there will be no other comments and objections
on the package name, I will push it. No need to resend the patch — I'll
fix these small issues I mentioned. Thank you.
--
Alex