emacs m17n support

emacs m17n support

[Arun Isaac]

[-- Attachment #1: Type: text/plain, Size: 243 bytes --]


The emacs package is currently missing m17n (multilingualization)
support, and is therefore not able to render Indic scripts properly. I
have rebuilt emacs with m17n support in my GUIX_PACKAGE_PATH. Shall I
send patches?

Thanks,
Arun Isaac.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 472 bytes --]

emacs m17n support

[Arun Isaac]

These are the patches to build emacs with m17n support. Please review and
consider for inclusion.

Thanks.

[PATCH 1/3] gnu: Add m17n-db.

[Arun Isaac]

* gnu/packages/emacs.scm (m17n-db): New variable.
---
 gnu/packages/emacs.scm | 36 ++++++++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 57fe8a9..2506d3e 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -16,6 +16,7 @@
 ;;; Copyright © 2016 Alex Griffin <a@ajgrf.com>
 ;;; Copyright © 2016 Nicolas Goaziou <mail@nicolasgoaziou.fr>
 ;;; Copyright © 2016 Alex Vong <alexvong1995@gmail.com>
+;;; Copyright © 2016 Arun Isaac <arunisaac@systemreboot.net>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -74,6 +75,7 @@
   #:use-module (gnu packages statistics)
   #:use-module (gnu packages xiph)
   #:use-module (gnu packages mp3)
+  #:use-module (gnu packages gettext)
   #:use-module (guix utils)
   #:use-module (srfi srfi-1))
 
@@ -3104,3 +3106,37 @@ that allows users to concentrate more on their own work.  Its features are:
 a visual interface, reduce overhead of completion by using statistic method,
 extensibility.")
     (license license:gpl3+)))
+
+(define-public m17n-db
+  (package
+    (name "m17n-db")
+    (version "1.7.0")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append
+             "http://download.savannah.gnu.org/releases/m17n/m17n-db-" version ".tar.gz"))
+       (sha256
+        (base32 "1w08hnsbknrcjlzp42c99bgwc9hzsnf5m4apdv0dacql2s09zfm2"))))
+    (build-system gnu-build-system)
+    (inputs
+     `(("gettext" ,gnu-gettext)))
+    (arguments
+     `(#:configure-flags
+       (list (string-append "--with-charmaps="
+                            (assoc-ref %build-inputs "libc")
+                            "/share/i18n/charmaps"))))
+    ;; With `guix lint' the home-page URI returns a small page saying
+    ;; that your browser does not handle frames. This triggers the "URI
+    ;; returns suspiciously small file" warning.
+    (home-page "http://www.nongnu.org/m17n/")
+    (synopsis "Multilingual text processing library (database)")
+    (description "The m17n library realizes multilingualization of
+many aspects of applications.  The m17n library represents
+multilingual text as an object named M-text.  M-text is a string with
+attributes called text properties, and designed to substitute for
+string in C.  Text properties carry any information required to input,
+display and edit the text.
+
+This package contains the library database.")
+    (license license:lgpl2.1+)))
-- 
2.10.0

Re: [PATCH 1/3] gnu: Add m17n-db.

[Ludovic Courtès]

Arun Isaac <arunisaac@systemreboot.net> skribis:

> * gnu/packages/emacs.scm (m17n-db): New variable.

[...]

> +       (uri (string-append
> +             "http://download.savannah.gnu.org/releases/m17n/m17n-db-" version ".tar.gz"))

Changed to mirror://savannah and applied, thanks!

Ludo’.

[PATCH 2/3] gnu: Add m17n-lib.

[Arun Isaac]

* gnu/packages/emacs.scm (m17n-lib): New variable.
---
 gnu/packages/emacs.scm | 40 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 40 insertions(+)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 2506d3e..7d10492 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -76,6 +76,9 @@
   #:use-module (gnu packages xiph)
   #:use-module (gnu packages mp3)
   #:use-module (gnu packages gettext)
+  #:use-module (gnu packages fribidi)
+  #:use-module (gnu packages gd)
+  #:use-module (gnu packages fontutils)
   #:use-module (guix utils)
   #:use-module (srfi srfi-1))
 
@@ -3140,3 +3143,40 @@ display and edit the text.
 
 This package contains the library database.")
     (license license:lgpl2.1+)))
+
+(define-public m17n-lib
+  (package
+    (name "m17n-lib")
+    (version "1.7.0")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append
+             "http://download.savannah.gnu.org/releases/m17n/m17n-lib-"
+             version ".tar.gz"))
+       (sha256
+        (base32 "10yv730i25g1rpzv6q49m6xn4p8fjm7jdwvik2h70sn8w3hm7f4f"))))
+    (build-system gnu-build-system)
+    (inputs
+     `(("fribidi" ,fribidi)
+       ("gd" ,gd)
+       ("libotf" ,libotf)
+       ("libxft" ,libxft)
+       ("libxml2" ,libxml2)
+       ("m17n-db" ,m17n-db)))
+    (arguments
+     `(#:parallel-build? #f))
+    ;; With `guix lint' the home-page URI returns a small page saying
+    ;; that your browser does not handle frames. This triggers the "URI
+    ;; returns suspiciously small file" warning.
+    (home-page "http://www.nongnu.org/m17n/")
+    (synopsis "Multilingual text processing library (runtime)")
+    (description "The m17n library realizes multilingualization of
+many aspects of applications.  The m17n library represents
+multilingual text as an object named M-text.  M-text is a string with
+attributes called text properties, and designed to substitute for
+string in C.  Text properties carry any information required to input,
+display and edit the text.
+
+This package contains the library runtime.")
+    (license license:lgpl2.1+)))
-- 
2.10.0

Re: [PATCH 2/3] gnu: Add m17n-lib.

[Ludovic Courtès]

Arun Isaac <arunisaac@systemreboot.net> skribis:

> * gnu/packages/emacs.scm (m17n-lib): New variable.

Applied with mirror:// URI, thanks!

Ludo’.

[PATCH 3/3] gnu: emacs: Build with m17n support.

[Arun Isaac]

* gnu/packages/emacs.scm (emacs): Build with m17n support.
---
 gnu/packages/emacs.scm | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 7d10492..d311bcf 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -160,7 +160,11 @@
        ("libsm" ,libsm)
        ("alsa-lib" ,alsa-lib)
        ("dbus" ,dbus)
-       ("guix-src" ,(package-source guix))))
+       ("guix-src" ,(package-source guix))
+
+       ;; multilingualization support
+       ("libotf" ,libotf)
+       ("m17n-lib" ,m17n-lib)))
     (native-inputs
      `(("pkg-config" ,pkg-config)
        ("texinfo" ,texinfo)))
-- 
2.10.0

Re: [PATCH 3/3] gnu: emacs: Build with m17n support.

[Ludovic Courtès]

Arun Isaac <arunisaac@systemreboot.net> skribis:

> * gnu/packages/emacs.scm (emacs): Build with m17n support.
> ---
>  gnu/packages/emacs.scm | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
> index 7d10492..d311bcf 100644
> --- a/gnu/packages/emacs.scm
> +++ b/gnu/packages/emacs.scm
> @@ -160,7 +160,11 @@
>         ("libsm" ,libsm)
>         ("alsa-lib" ,alsa-lib)
>         ("dbus" ,dbus)
> -       ("guix-src" ,(package-source guix))))
> +       ("guix-src" ,(package-source guix))
> +
> +       ;; multilingualization support
> +       ("libotf" ,libotf)
> +       ("m17n-lib" ,m17n-lib)))

The impact in terms of size seems to be quite small:

--8<---------------cut here---------------start------------->8---
$ ./pre-inst-env guix size emacs|tail -1
total: 582.9 MiB
$ ./pre-inst-env guix size emacs libotf m17n-lib|tail -1
total: 595.2 MiB
--8<---------------cut here---------------end--------------->8---

So I went ahead and applied this patch.

How can one check whether it works?  Do users have to install m17n-db in
their profile?

Thanks!

Ludo’.

Re: [PATCH 3/3] gnu: emacs: Build with m17n support.

[Arun Isaac]

[-- Attachment #1: Type: text/plain, Size: 434 bytes --]


> How can one check whether it works?  Do users have to install m17n-db in
> their profile?

No, users don't have to install m17n-db in their profile. They just need
to use the new emacs package built with m17n. m17n-db and m17n-lib are
just "inputs" not "propagated-inputs".

The difference with and without m17n is visible in the rendering of
Indic scripts. Tamil, one of the languages of South India, is the one I
use and tested.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 472 bytes --]