* bug#27817: 26.0.50; Dired: Support eshell/ls from the beginning if user want to
@ 2017-07-25 6:52 Tino Calancha
2017-07-25 7:00 ` Tino Calancha
2017-07-25 14:33 ` Eli Zaretskii
0 siblings, 2 replies; 11+ messages in thread
From: Tino Calancha @ 2017-07-25 6:52 UTC (permalink / raw
To: 27817
An user must be able to use Dired with eshell/ls at the
beginning even if s?he doesn't have installed an external
"ls" program.
I) emacs -Q -l em-ls
M-: (progn
(setq insert-directory-program "foobar-ls"
eshell-ls-use-in-dired t)
(dired source-directory)) RET
;; We got an error because `insert-directory-program' was called.
;; Now compare with ls-lisp; in this case `insert-directory-program'
;; is not called.
II) emacs -Q -l ls-lisp
M-: (progn
(setq insert-directory-program "foobar-ls"
ls-lisp-use-insert-directory-program nil)
(dired source-directory)) RET
--8<-----------------------------cut here---------------start------------->8---
commit 6322f2932e9a083e01bb86aa219fc8256443c33f
Author: Tino Calancha <tino.calancha@gmail.com>
Date: Tue Jul 25 15:46:54 2017 +0900
Dired: Support eshell/ls from the beginning if user want to
* lisp/dired.el (dired-insert-directory): Check for em-ls as well.
* test/lisp/dired-tests.el (dired-test-bug27817): Add test.
diff --git a/lisp/dired.el b/lisp/dired.el
index 9d500a9f52..3b29c7129d 100644
--- a/lisp/dired.el
+++ b/lisp/dired.el
@@ -1207,6 +1207,7 @@ dired-insert-directory
;; 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)))
(or (if (eq dired-use-ls-dired 'unspecified)
;; Check whether "ls --dired" gives exit code 0, and
;; save the answer in `dired-use-ls-dired'.
diff --git a/test/lisp/dired-tests.el b/test/lisp/dired-tests.el
index 69331457c0..601d65768b 100644
--- a/test/lisp/dired-tests.el
+++ b/test/lisp/dired-tests.el
@@ -175,5 +175,18 @@
(should (looking-at "src")))
(when (buffer-live-p buf) (kill-buffer buf)))))
+(ert-deftest dired-test-bug27817 ()
+ "Test for http://debbugs.gnu.org/27817 ."
+ (require 'em-ls)
+ (let ((orig eshell-ls-use-in-dired)
+ (dired-use-ls-dired 'unspecified)
+ buf insert-directory-program)
+ (unwind-protect
+ (progn
+ (customize-set-variable 'eshell-ls-use-in-dired t)
+ (should (setq buf (dired source-directory))))
+ (customize-set-variable 'eshell-ls-use-in-dired orig)
+ (and (buffer-live-p buf) (kill-buffer)))))
+
(provide 'dired-tests)
;; dired-tests.el ends here
--8<-----------------------------cut here---------------end--------------->8---
In GNU Emacs 26.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.22.11)
of 2017-07-25
Repository revision: 565cfd9f6c19e4d2aa318efdf19bdc56175bd153
^ permalink raw reply related [flat|nested] 11+ messages in thread
* bug#27817: 26.0.50; Dired: Support eshell/ls from the beginning if user want to
2017-07-25 6:52 bug#27817: 26.0.50; Dired: Support eshell/ls from the beginning if user want to Tino Calancha
@ 2017-07-25 7:00 ` Tino Calancha
2017-07-25 14:33 ` Eli Zaretskii
1 sibling, 0 replies; 11+ messages in thread
From: Tino Calancha @ 2017-07-25 7:00 UTC (permalink / raw
To: 27817
Tino Calancha <tino.calancha@gmail.com> writes:
> An user must be able to use Dired with eshell/ls at the
> beginning even if s?he doesn't have installed an external
> "ls" program.
>
> I) emacs -Q -l em-ls
> M-: (progn
> (setq insert-directory-program "foobar-ls"
> eshell-ls-use-in-dired t)
> (dired source-directory)) RET
;; Sorry, previous form should be as follows (in order to
;; `insert-directory' get the advice):
M-x: (progn
(setq insert-directory-program "foobar-ls")
(customize-set-variable 'eshell-ls-use-in-dired t)
(dired source-directory))
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#27817: 26.0.50; Dired: Support eshell/ls from the beginning if user want to
2017-07-25 6:52 bug#27817: 26.0.50; Dired: Support eshell/ls from the beginning if user want to Tino Calancha
2017-07-25 7:00 ` Tino Calancha
@ 2017-07-25 14:33 ` Eli Zaretskii
2017-07-25 14:38 ` Tino Calancha
2017-07-26 7:57 ` Tino Calancha
1 sibling, 2 replies; 11+ messages in thread
From: Eli Zaretskii @ 2017-07-25 14:33 UTC (permalink / raw
To: Tino Calancha; +Cc: 27817
> From: Tino Calancha <tino.calancha@gmail.com>
> Date: Tue, 25 Jul 2017 15:52:29 +0900
>
> commit 6322f2932e9a083e01bb86aa219fc8256443c33f
> Author: Tino Calancha <tino.calancha@gmail.com>
> Date: Tue Jul 25 15:46:54 2017 +0900
>
> Dired: Support eshell/ls from the beginning if user want to
>
> * lisp/dired.el (dired-insert-directory): Check for em-ls as well.
> * test/lisp/dired-tests.el (dired-test-bug27817): Add test.
Looks okay, but (a) please cite the bug number in the commit log
message, and (b) shouldn't this:
> + (not (and (featurep 'eshell) (bound-and-true-p eshell-ls-use-in-dired)))
^^^^^^^^^^^^^^^^^^^^^^
quote eshell-ls-use-in-dired?
Thanks.
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#27817: 26.0.50; Dired: Support eshell/ls from the beginning if user want to
2017-07-25 14:33 ` Eli Zaretskii
@ 2017-07-25 14:38 ` Tino Calancha
2017-07-25 14:48 ` Noam Postavsky
2017-07-25 14:48 ` Eli Zaretskii
2017-07-26 7:57 ` Tino Calancha
1 sibling, 2 replies; 11+ messages in thread
From: Tino Calancha @ 2017-07-25 14:38 UTC (permalink / raw
To: Eli Zaretskii; +Cc: 27817
On Tue, 25 Jul 2017, Eli Zaretskii wrote:
>> From: Tino Calancha <tino.calancha@gmail.com>
>> Date: Tue, 25 Jul 2017 15:52:29 +0900
>>
>> commit 6322f2932e9a083e01bb86aa219fc8256443c33f
>> Author: Tino Calancha <tino.calancha@gmail.com>
>> Date: Tue Jul 25 15:46:54 2017 +0900
>>
>> Dired: Support eshell/ls from the beginning if user want to
>>
>> * lisp/dired.el (dired-insert-directory): Check for em-ls as well.
>> * test/lisp/dired-tests.el (dired-test-bug27817): Add test.
>
> Looks okay, but (a) please cite the bug number in the commit log
> message, and (b) shouldn't this:
Thank you. i will do.
>
>> + (not (and (featurep 'eshell) (bound-and-true-p eshell-ls-use-in-dired)))
> quote eshell-ls-use-in-dired?
I don't think so. But i drank 2 beer so i migh be wrong...
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#27817: 26.0.50; Dired: Support eshell/ls from the beginning if user want to
2017-07-25 14:38 ` Tino Calancha
@ 2017-07-25 14:48 ` Noam Postavsky
2017-07-25 15:02 ` Eli Zaretskii
2017-07-25 15:09 ` Tino Calancha
2017-07-25 14:48 ` Eli Zaretskii
1 sibling, 2 replies; 11+ messages in thread
From: Noam Postavsky @ 2017-07-25 14:48 UTC (permalink / raw
To: Tino Calancha; +Cc: 27817
On Tue, Jul 25, 2017 at 10:38 AM, Tino Calancha <tino.calancha@gmail.com> wrote:
>>
>>
>>> + (not (and (featurep 'eshell) (bound-and-true-p
>>> eshell-ls-use-in-dired)))
>>
>> quote eshell-ls-use-in-dired?
>
> I don't think so. But i drank 2 beer so i migh be wrong...
bound-and-true-p is a macro, its argument doesn't need quoting. boundp
is a function, its argument does need quoting. Trips me up too.
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#27817: 26.0.50; Dired: Support eshell/ls from the beginning if user want to
2017-07-25 14:38 ` Tino Calancha
2017-07-25 14:48 ` Noam Postavsky
@ 2017-07-25 14:48 ` Eli Zaretskii
1 sibling, 0 replies; 11+ messages in thread
From: Eli Zaretskii @ 2017-07-25 14:48 UTC (permalink / raw
To: Tino Calancha; +Cc: 27817
> From: Tino Calancha <tino.calancha@gmail.com>
> Date: Tue, 25 Jul 2017 23:38:56 +0900 (JST)
> cc: 27817@debbugs.gnu.org
>
> >> + (not (and (featurep 'eshell) (bound-and-true-p eshell-ls-use-in-dired)))
> > quote eshell-ls-use-in-dired?
> I don't think so. But i drank 2 beer so i migh be wrong...
Ah, I see bound-and-true-p confusingly deviates from boundp by quoting
its argument. So much for mnemonic value...
Sorry.
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#27817: 26.0.50; Dired: Support eshell/ls from the beginning if user want to
2017-07-25 14:48 ` Noam Postavsky
@ 2017-07-25 15:02 ` Eli Zaretskii
2017-07-25 15:09 ` Tino Calancha
1 sibling, 0 replies; 11+ messages in thread
From: Eli Zaretskii @ 2017-07-25 15:02 UTC (permalink / raw
To: Noam Postavsky; +Cc: 27817, tino.calancha
> From: Noam Postavsky <npostavs@users.sourceforge.net>
> Date: Tue, 25 Jul 2017 10:48:24 -0400
> Cc: Eli Zaretskii <eliz@gnu.org>, 27817@debbugs.gnu.org
>
> Trips me up too.
Then I'm in good company ;-)
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#27817: 26.0.50; Dired: Support eshell/ls from the beginning if user want to
2017-07-25 14:48 ` Noam Postavsky
2017-07-25 15:02 ` Eli Zaretskii
@ 2017-07-25 15:09 ` Tino Calancha
2017-07-25 15:27 ` Noam Postavsky
1 sibling, 1 reply; 11+ messages in thread
From: Tino Calancha @ 2017-07-25 15:09 UTC (permalink / raw
To: Noam Postavsky; +Cc: 27817
On Tue, 25 Jul 2017, Noam Postavsky wrote:
> On Tue, Jul 25, 2017 at 10:38 AM, Tino Calancha <tino.calancha@gmail.com> wrote:
>>>
>>>
>>>> + (not (and (featurep 'eshell) (bound-and-true-p
>>>> eshell-ls-use-in-dired)))
>>>
>>> quote eshell-ls-use-in-dired?
>>
>> I don't think so. But i drank 2 beer so i migh be wrong...
>
> bound-and-true-p is a macro, its argument doesn't need quoting. boundp
> is a function, its argument does need quoting. Trips me up too.
Thank you for the support! I understand you until the second period.
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#27817: 26.0.50; Dired: Support eshell/ls from the beginning if user want to
2017-07-25 15:09 ` Tino Calancha
@ 2017-07-25 15:27 ` Noam Postavsky
2017-07-25 15:34 ` Tino Calancha
0 siblings, 1 reply; 11+ messages in thread
From: Noam Postavsky @ 2017-07-25 15:27 UTC (permalink / raw
To: Tino Calancha; +Cc: 27817
On Tue, Jul 25, 2017 at 11:09 AM, Tino Calancha <tino.calancha@gmail.com> wrote:
>> bound-and-true-p is a macro, its argument doesn't need quoting. boundp
>> is a function, its argument does need quoting. Trips me up too.
>
> Thank you for the support! I understand you until the second period.
Oh, I just meant that I also often get the quoting for
boundp/bound-and-true-p wrong.
https://en.wiktionary.org/wiki/trip_up
trip up: 2. (transitive) To cause (someone) to commit an error,
trick into a mistake.
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#27817: 26.0.50; Dired: Support eshell/ls from the beginning if user want to
2017-07-25 15:27 ` Noam Postavsky
@ 2017-07-25 15:34 ` Tino Calancha
0 siblings, 0 replies; 11+ messages in thread
From: Tino Calancha @ 2017-07-25 15:34 UTC (permalink / raw
To: Noam Postavsky; +Cc: 27817
[-- Attachment #1: Type: text/plain, Size: 312 bytes --]
On Tue, 25 Jul 2017, Noam Postavsky wrote:
> https://en.wiktionary.org/wiki/trip_up
>
> trip up: 2. (transitive) To cause (someone) to commit an error,
> trick into a mistake.
google.translate (from english to spanish):
"Trips me up too" ===> "Me dispara también"
I read the second as: "It shoots me too".
^ permalink raw reply [flat|nested] 11+ messages in thread
* bug#27817: 26.0.50; Dired: Support eshell/ls from the beginning if user want to
2017-07-25 14:33 ` Eli Zaretskii
2017-07-25 14:38 ` Tino Calancha
@ 2017-07-26 7:57 ` Tino Calancha
1 sibling, 0 replies; 11+ messages in thread
From: Tino Calancha @ 2017-07-26 7:57 UTC (permalink / raw
To: 27817-done
Eli Zaretskii <eliz@gnu.org> writes:
>> Dired: Support eshell/ls from the beginning if user want to
> Looks okay
Fixed in master branch as commit d5c41e99a2071e3ee491a53a0f9506f62fa6ae54
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2017-07-26 7:57 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-07-25 6:52 bug#27817: 26.0.50; Dired: Support eshell/ls from the beginning if user want to Tino Calancha
2017-07-25 7:00 ` Tino Calancha
2017-07-25 14:33 ` Eli Zaretskii
2017-07-25 14:38 ` Tino Calancha
2017-07-25 14:48 ` Noam Postavsky
2017-07-25 15:02 ` Eli Zaretskii
2017-07-25 15:09 ` Tino Calancha
2017-07-25 15:27 ` Noam Postavsky
2017-07-25 15:34 ` Tino Calancha
2017-07-25 14:48 ` Eli Zaretskii
2017-07-26 7:57 ` Tino Calancha
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.