* Re: master b968323: Fix previous face.el change
[not found] ` <20210322191809.D85B920D10@vcs0.savannah.gnu.org>
@ 2021-03-24 5:03 ` Stefan Kangas
2021-03-24 8:23 ` Lars Ingebrigtsen
0 siblings, 1 reply; 4+ messages in thread
From: Stefan Kangas @ 2021-03-24 5:03 UTC (permalink / raw)
To: Lars Ingebrigtsen; +Cc: emacs-devel
larsi@gnus.org (Lars Ingebrigtsen) writes:
> branch: master
> commit b9683230ecaada87f23bdba59ef044c4b0374216
> Author: Lars Ingebrigtsen <larsi@gnus.org>
> Commit: Lars Ingebrigtsen <larsi@gnus.org>
>
> Fix previous face.el change
>
> * lisp/faces.el (require): Fix compilation warning from previous
> face.el change.
> ---
> lisp/faces.el | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/lisp/faces.el b/lisp/faces.el
> index 3ea4c94..1067556 100644
> --- a/lisp/faces.el
> +++ b/lisp/faces.el
> @@ -25,6 +25,8 @@
>
> ;;; Code:
>
> +(eval-when-compile (require 'subr-x))
> +
This commit breaks bootstrap here:
Loading /home/skangas/wip/emacs/lisp/faces.el (source)...
Eager macro-expansion failure: (void-variable
read-file-name-completion-ignore-case)
Eager macro-expansion failure: (void-variable
read-file-name-completion-ignore-case)
Loading /home/skangas/wip/emacs/lisp/cus-face.el (source)...
Eager macro-expansion failure: (void-variable
read-file-name-completion-ignore-case)
Eager macro-expansion failure: (void-variable
read-file-name-completion-ignore-case)
Loading /home/skangas/wip/emacs/lisp/faces.el (source)...
Eager macro-expansion failure: (void-variable
read-file-name-completion-ignore-case)
Eager macro-expansion failure: (void-variable
read-file-name-completion-ignore-case)
Eager macro-expansion failure: (void-variable
read-file-name-completion-ignore-case)
[... snip ...]
Eager macro-expansion failure: (void-variable
read-file-name-completion-ignore-case)
Eager macro-expansion failure: (void-variable
read-file-name-completion-ignore-case)
make[3]: *** [Makefile:279: emacs-lisp/autoload.elc] Segmentation fault
I find `string-trim' an obviously useful addition to Emacs Lisp, so I
suggest moving it to subr.el instead.
That might call for adding it to the ELisp Manual as well.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-03-25 9:07 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20210322191808.723.93761@vcs0.savannah.gnu.org>
[not found] ` <20210322191809.D85B920D10@vcs0.savannah.gnu.org>
2021-03-24 5:03 ` master b968323: Fix previous face.el change Stefan Kangas
2021-03-24 8:23 ` Lars Ingebrigtsen
2021-03-24 20:42 ` Basil L. Contovounesios
2021-03-25 9:07 ` Lars Ingebrigtsen
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.