Re: [PATCH] gnu: font-gnu-freefont-ttf: Update to 20120503 and build from source.

[ludo@gnu.org (Ludovic Courtès)]

alírio eyng <alirioeyng@gmail.com> skribis:

>>> * gnu/packages/fonts.scm (font-gnu-freefont-ttf): Update to 20120503.
>>> [arguments]: Build from source.
>>> * guix/gnu-maintenance.scm (%tarball-rx): Remove comment.
>> Not clear, how about “Remove comment about freefont-ttf.”?
> ok
>>> […]
>>> +    (native-inputs `(("fontforge" ,(package (inherit fontforge)
>>> +                                     (inputs `(("python-2" ,python-2)
>>> +                                     ,@(package-inputs fontforge)))))))
>> Why not add ‘python-2’ to the fontforge package directly?
> see inline references in the updated patch
>
> From ab7a628f7d8fc9f68f0ebfdd27b3bcb32a607724 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?al=C3=ADrio=20eyng?= <alirioeyng@gmail.com>
> Date: Tue, 29 Mar 2016 03:32:58 +0000
> Subject: [PATCH] gnu: font-gnu-freefont-ttf: Update to 20120503 and build from
>  source.
>
> * gnu/packages/fonts.scm (font-gnu-freefont-ttf): Update to 20120503.
> [arguments]: Build from source.
> * guix/gnu-maintenance.scm (%tarball-rx): Remove comment about freefont-ttf.

[...]

> +    ;; https://github.com/fontforge/fontforge/pull/109
> +    ;;  "fontforge has to be compiled against only one python version"
> +    ;; https://github.com/fontforge/fontforge/issues/1731
> +    ;;  "default python version that fontforge links with is python 2"
> +    (native-inputs `(("fontforge" ,(package (inherit fontforge)
> +                                     (inputs `(("python-2" ,python-2)
> +                                     ,@(package-inputs fontforge)))))))

I understand cannot support Python 2 and 3 simultaneously, but I think
Leo was suggesting changing the ‘fontforge’ package to use Python 2.x
instead of 3.x.  WDYT?

Also, could you explain why we need fontforge-with-python2 instead of
fontforge-with-python3?

> diff --git a/guix/gnu-maintenance.scm b/guix/gnu-maintenance.scm
> index 9d720ca..eb9dcf8 100644
> --- a/guix/gnu-maintenance.scm
> +++ b/guix/gnu-maintenance.scm
> @@ -243,7 +243,6 @@ stored."
>      (substring tarball 0 end)))
>  
>  (define %tarball-rx
> -  ;; The .zip extensions is notably used for freefont-ttf.
>    ;; The "-src" pattern is for "TeXmacs-1.0.7.9-src.tar.gz".
>    ;; The "-gnu[0-9]" pattern is for "icecat-38.4.0-gnu1.tar.bz2".
>    (make-regexp "^([^.]+)-([0-9]|[^-])+(-(src|gnu[0-9]))?\\.(tar\\.|zip$)"))

This comment remains valid because there are .zip files in
ftp.gnu.org/gnu/freefont, so I think we should keep it.

Thanks, it’s a nice to have these fonts built from source!

Ludo’.