help-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Build failure caused by a .tit file


From: Göktuğ Kayaalp
Subject: Re: Build failure caused by a .tit file
Date: Tue, 18 Apr 2017 16:16:12 +0300
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)

On 2017-04-18 15:41 +0300, Göktuğ Kayaalp <address@hidden> wrote:
> diff --git a/lisp/international/titdic-cnv.el 
> b/lisp/international/titdic-cnv.el
> index 130bc74..6a593e2 100644
> --- a/lisp/international/titdic-cnv.el
> +++ b/lisp/international/titdic-cnv.el
> @@ -474,7 +474,6 @@ titdic-convert
>      (with-temp-file  (tit-make-quail-package-file-name filename dirname)
>        (let ((standard-output (current-buffer)))
>       (with-temp-buffer
> -       (set-buffer-multibyte nil)
>         ;; Here we must use `raw-text' instead of `no-conversion' to
>         ;; enable auto-decoding of eol format (CRLF->LF).
>         (let ((coding-system-for-read 'raw-text))

A better version of this one is as follows, which removes the
complementary set-buffer-multibyte call:

diff --git a/lisp/international/titdic-cnv.el b/lisp/international/titdic-cnv.el
index 130bc74..aa6e4f1 100644
--- a/lisp/international/titdic-cnv.el
+++ b/lisp/international/titdic-cnv.el
@@ -474,7 +474,6 @@ titdic-convert
     (with-temp-file  (tit-make-quail-package-file-name filename dirname)
       (let ((standard-output (current-buffer)))
        (with-temp-buffer
-         (set-buffer-multibyte nil)
          ;; Here we must use `raw-text' instead of `no-conversion' to
          ;; enable auto-decoding of eol format (CRLF->LF).
          (let ((coding-system-for-read 'raw-text))
@@ -504,7 +503,6 @@ titdic-convert
                  (coding-system-change-eol-conversion coding-system 'unix))
            (remove-text-properties (point-min) (point-max) '(charset nil)))
 
-         (set-buffer-multibyte t)
          ;; Set point the starting position of the body part.
          (goto-char (point-min))
          (if (not (search-forward "\nBEGIN" nil t))

BTW I did manage to successfully build with this patch applied.  Maybe
that second call should happen before the first call to the search
forward, i.e. just after the let form in the first hunk.

Now I get this weird warning, I can report that elsewhere if not
expected:

Error (initialization): Creation of the default fontsets failed: (error
Invalid script or charset name: old-italic)

This is from a build of master as of yesterday (edc63bf).  I don't get
this with 25.1.

--gk.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]