From: Thierry Volpiatto <thierry.volpiatto@gmail.com>
To: bug-gnu-emacs@gnu.org
Subject: bug#6995: 24.0.50; get-free-disk-space doesn't return available space in dired
Date: Mon, 13 Sep 2010 15:22:17 +0200 [thread overview]
Message-ID: <87pqwh3hiu.fsf@tux.homenetwork> (raw)
In-Reply-To: <87d3sp5olh.fsf@tux.homenetwork>
Stefan Monnier <monnier@iro.umontreal.ca> writes:
>> How about dropping the use of `df', at least on GNU/Linux? It's IMO
>> ridiculous to invoke an external program for a single system call. I
>> know that this system call is notoriously non-portable, and that doing
>> in Emacs all that system-dependent stuff that `df' does is not a good
>> idea. But at least on the most popular free system (and perhaps on a
>> few compatible ones) we could do it right without all that labor, and
>> gain stability and reliability that depending on `df' will never
>> achieve.
>
> That would make a lot of sense, yes,
I agree too, but waiting such a change, you should rewrite
`get-free-disk-space' as it can't work correctly on the output of df.
,----
| ;; Usual format is as follows:
| ;; Filesystem ... Used Available Capacity ...
| ;; /dev/sda6 ...48106535 35481255 10669850 ...
| (goto-char (point-min))
| (when (re-search-forward " +Avail[^ \n]*"
`----
But usual format is not the same on all locales:
Here it is:
,----
| Sys. de fichiers 1024-blocs Utilisé Dispo. Capacité Monté sur
| /dev/sda7 48947688 21618816 24842416 47% /home
`----
So (re-search-forward " +Avail[^ \n]*" is not portable here.
I use this simplified version of get-free-disk-space if it can help:
,----
| (defun get-free-disk-space (dir)
| (let* ((data (with-temp-buffer
| (call-process directory-free-space-program
| nil t nil
| directory-free-space-args
| dir)
| (split-string (buffer-string) "\n" t)))
| (values (cdr (split-string (second data)))))
| (nth 2 values)))
`----
Tough the version on 23.2 work also.
Note that the value of "total used in directory" is always wrong.
How do you get this value?
--
A+ Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997
next prev parent reply other threads:[~2010-09-13 13:22 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-07 19:40 bug#6995: 24.0.50; get-free-disk-space doesn't return available space in dired Thierry Volpiatto
2010-09-07 20:36 ` Glenn Morris
2010-09-13 6:56 ` Eli Zaretskii
2010-09-13 10:14 ` Stefan Monnier
2010-09-13 13:22 ` Thierry Volpiatto [this message]
2010-09-15 18:39 ` Thierry Volpiatto
2010-09-18 13:54 ` Thierry Volpiatto
2010-09-25 20:17 ` Chong Yidong
2010-09-27 8:37 ` Thierry Volpiatto
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
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87pqwh3hiu.fsf@tux.homenetwork \
--to=thierry.volpiatto@gmail.com \
--cc=bug-gnu-emacs@gnu.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 public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).