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

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

bug#39546: 28.0.50; Do not require subr-x at run time


From: Michael Albinus
Subject: bug#39546: 28.0.50; Do not require subr-x at run time
Date: Mon, 10 Feb 2020 15:54:56 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux)

Tino Calancha <address@hidden> writes:

Hi Tino,

> diff --git a/lisp/net/tramp-compat.el b/lisp/net/tramp-compat.el
> index 87bcd08b85..8f74683dee 100644
> --- a/lisp/net/tramp-compat.el
> +++ b/lisp/net/tramp-compat.el
> @@ -39,7 +39,7 @@ tramp-unload-file-name-handlers
>  (require 'ls-lisp)  ;; Due to `tramp-handle-insert-directory'.
>  (require 'parse-time)
>  (require 'shell)
> -(require 'subr-x)
> +(eval-when-compile (require 'subr-x))
>
>  ;; `temporary-file-directory' as function is introduced with Emacs 26.1.
>  (declare-function tramp-handle-temporary-file-directory "tramp")

Tramp needs `string-join' and `string-empty-p' from subr-x.el. If we
don't load it here, we must eval-when-compile it in almost all Tramp
files, I believe.

Hmm, what's so bad requiring it? (I know the general answer because both
are defsubst'es, but I don't see the point here).





reply via email to

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