Eshell dependencies: Fix recent regressions

* lisp/dired.el (dired-insert-directory): Tweak bug#27817's ugly hack.

* lisp/eshell/em-ls.el: Refine 'require's.

* lisp/eshell/esh-opt.el: Require esh-util on behalf of its clients.

diff --git a/lisp/dired.el b/lisp/dired.el
index fc0b71238baf075963dac2b46bf7a246e095404b..4c2c3f44e72e1c27e125c8fde4a2f77dacce7b3e 100644 (file)
--- a/lisp/dired.el
+++ b/lisp/dired.el
@@ -1269,8 +1269,8 @@ If HDR is non-nil, insert a header line with the directory name."
         ;; as indicated by `ls-lisp-use-insert-directory-program'.
         (not (and (featurep 'ls-lisp)
                   (null ls-lisp-use-insert-directory-program)))
-         (not (and (featurep 'eshell)
-                   (bound-and-true-p eshell-ls-use-in-dired)))
+         ;; FIXME: Big ugly hack for Eshell's eshell-ls-use-in-dired.
+         (not (bound-and-true-p eshell-ls-use-in-dired))
         (or (file-remote-p dir)
              (if (eq dired-use-ls-dired 'unspecified)
                 ;; Check whether "ls --dired" gives exit code 0, and

diff --git a/lisp/eshell/em-ls.el b/lisp/eshell/em-ls.el
index 5e4bbdc87efb6ea9d8d670e9bbd3dd48cab661aa..89969d3258210d7f3480b2e268fbc123cc2d73f9 100644 (file)
--- a/lisp/eshell/em-ls.el
+++ b/lisp/eshell/em-ls.el
@@ -29,7 +29,8 @@
 (require 'cl-lib)
 (require 'esh-util)
 (require 'esh-opt)
-(eval-when-compile (require 'eshell))
+(require 'esh-proc)
+(require 'esh-cmd)
 
 ;;;###autoload
 (progn

diff --git a/lisp/eshell/esh-opt.el b/lisp/eshell/esh-opt.el
index 5b2693283a729db03c6229a42a15f57783c444d6..3ea5873cafd9e41e80f521928100e1095dac4a7f 100644 (file)
--- a/lisp/eshell/esh-opt.el
+++ b/lisp/eshell/esh-opt.el
@@ -33,6 +33,10 @@
 
 ;;; User Functions:
 
+;; Macro expansion of eshell-eval-using-options refers to eshell-stringify-list
+;; defined in esh-util.
+(require 'esh-util)
+
 (defmacro eshell-eval-using-options (name macro-args options &rest body-forms)
   "Process NAME's MACRO-ARGS using a set of command line OPTIONS.
 After doing so, stores settings in local symbols as declared by OPTIONS;