From: Michael Albinus <michael.albinus@gmx.de>
To: Lars Ingebrigtsen <larsi@gnus.org>
Cc: emacs-devel@gnu.org
Subject: Re: master 1914d94 2/2: Change how Dired displays available space
Date: Thu, 02 Dec 2021 17:09:36 +0100 [thread overview]
Message-ID: <87sfvbm1j3.fsf@gmx.de> (raw)
In-Reply-To: <87v90710bv.fsf@gnus.org> (Lars Ingebrigtsen's message of "Thu, 02 Dec 2021 16:41:08 +0100")
[-- Attachment #1: Type: text/plain, Size: 1699 bytes --]
Lars Ingebrigtsen <larsi@gnus.org> writes:
Hi Lars,
>> Hmm. insert-directory can be used w/o being called from dired. And there
>> are further implementations of insert-directory, because it uses file
>> name handlers. So these functions must be adapted as well.
>
> My assumption was that (as for the first bit) that not translating the
> "total x" thing wouldn't be catastrophic, so that could be adjusted
> afterwards.
"Afterwards" is in dired-insert-directory? As said, this is not always
in play.
> As for the other implementations, I assumed that they would just
> continue to work as is, and if they do that "total" transformation, it
> would be trivial to just remove that bit.
>
> If this causes major problems, I'm sorry.
Don't know whether it causes major problems. I guess the cases in core
Emacs can be fixed easily. For possible other implementations in the
wild, I don't know.
>> Granted, most of them are from Tramp, but there could be also other
>> implementations in the wild.
>>
>> Is there a shiny reason to move this functionality from insert-directory
>> to dired-insert-directory? A discussion about, or at least a trigger for
>> this change (aka announcement in etc/NEWS), would have been appreciated.
>
> I implemented this first as a part of insert-directory. But the problem
> is that it's dired itself that prepends the /current/path/name at the
> top, so the thing had to go to dired.
I do not oppose heavily to this change. The advantage is, that the
boring "adjust the free disk space" must not be implemented again and
again in all insert-directory incarnations. But a coordinated change
would have been better ...
Well, what about this change in etc/NEWS:
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Type: text/x-patch, Size: 458 bytes --]
diff --git a/etc/NEWS b/etc/NEWS
index d783fc019c..304f707f2e 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -741,6 +741,10 @@ with recent versions of Firefox.
** The function 'image-dired-get-exif-data' is now obsolete.
Use 'exif-parse-file' and 'exif-field' instead.
+---
+** 'insert-directory' alternatives should not change the free disk space line.
+This change is now applied in 'dired-insert-directory'.
+
\f
* Lisp Changes in Emacs 29.1
[-- Attachment #3: Type: text/plain, Size: 83 bytes --]
And I will see what must be done in Tramp and ange-ftp.el
Best regards, Michael.
next prev parent reply other threads:[~2021-12-02 16:09 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20211201222724.19544.19533@vcs0.savannah.gnu.org>
[not found] ` <20211201222726.7AFF521362@vcs0.savannah.gnu.org>
2021-12-02 14:55 ` master 1914d94 2/2: Change how Dired displays available space Michael Albinus
2021-12-02 15:41 ` Lars Ingebrigtsen
2021-12-02 16:09 ` Michael Albinus [this message]
2021-12-02 16:47 ` Lars Ingebrigtsen
2021-12-02 17:21 ` [External] : " Drew Adams
2021-12-02 18:46 ` Michael Albinus
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87sfvbm1j3.fsf@gmx.de \
--to=michael.albinus@gmx.de \
--cc=emacs-devel@gnu.org \
--cc=larsi@gnus.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.