From: "Jérémy Compostella" <jeremy.compostella@gmail.com>
To: 10598@debbugs.gnu.org
Subject: bug#10598: 24.0.92; run dired-do-async-shell-command on multiple files individually
Date: Wed, 25 Jan 2012 23:48:51 +0100 [thread overview]
Message-ID: <87hazjtn3g.fsf@Apollo.jerryland.fr> (raw)
In-Reply-To: <87liow9jdf.fsf@niu.edu>
[-- Attachment #1: Type: text/plain, Size: 541 bytes --]
Hi,
This bug was really annoying me tonight so I wrote the attached patch
which fix it by translating `gv foo.ps&;gv bar.ps&' to `gv foo.ps&gv
bar.ps&'.
I do agree it should be better to provide a way to choice between the
sequential or parallel solution but I have no idea how to implement it
in a clean fashion for now.
Anyway, the current behavior looks buggy and I propose to merge the
attached patch in a first time in order to have a acceptable behavior
that is far better than an error.
Best regards,
Jérémy
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: Fix dired-do-async-shell-command on multiple files individually (bug#10598) patch --]
[-- Type: text/x-diff, Size: 1186 bytes --]
From aefc85c364f6e51676ffc9dc44deb9a4d43a128c Mon Sep 17 00:00:00 2001
From: Jeremy Compostella <jeremy.compostella@gmail.com>
Date: Wed, 25 Jan 2012 23:38:54 +0100
Subject: [PATCH] Fix dired-do-async-shell-command on multiple files individually (bug#10598)
The `dired-shell-stuff-it' should not insert the ";" shell command
separator when called with a command argument ending with the "&"
character.
Signed-off-by: Jeremy Compostella <jeremy.compostella@gmail.com>
---
lisp/dired-aux.el | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/lisp/dired-aux.el b/lisp/dired-aux.el
index 2d05be3..74f0de9 100644
--- a/lisp/dired-aux.el
+++ b/lisp/dired-aux.el
@@ -659,7 +659,8 @@ can be produced by `dired-get-marked-files', for example."
retval))
(lambda (x) (concat command dired-mark-separator x)))))
(if on-each
- (mapconcat stuff-it (mapcar 'shell-quote-argument file-list) ";")
+ (mapconcat stuff-it (mapcar 'shell-quote-argument file-list)
+ (if (string-match "[ \t]*&[ \t]*\\'" command) "" ";"))
(let ((files (mapconcat 'shell-quote-argument
file-list dired-mark-separator)))
(if (> (length file-list) 1)
--
1.7.2.5
prev parent reply other threads:[~2012-01-25 22:48 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-25 10:17 bug#10598: 24.0.92; run dired-do-async-shell-command on multiple files individually Roland Winkler
2012-01-25 11:14 ` Juri Linkov
2012-01-25 11:47 ` Roland Winkler
2012-07-12 9:01 ` Juri Linkov
2012-07-12 10:26 ` Roland Winkler
2012-07-13 8:30 ` Juri Linkov
2012-07-13 9:28 ` Roland Winkler
2012-07-14 9:31 ` Juri Linkov
2012-07-15 7:51 ` Roland Winkler
2012-07-15 8:29 ` Juri Linkov
2012-07-17 7:17 ` Roland Winkler
2012-07-17 18:41 ` Juri Linkov
2012-01-25 22:48 ` Jérémy Compostella [this message]
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=87hazjtn3g.fsf@Apollo.jerryland.fr \
--to=jeremy.compostella@gmail.com \
--cc=10598@debbugs.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 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.