[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/with-editor 9097321 112/140: with-editor-export-editor: Fi
From: |
Jonas Bernoulli |
Subject: |
[nongnu] elpa/with-editor 9097321 112/140: with-editor-export-editor: Fix for term-mode |
Date: |
Fri, 6 Aug 2021 12:51:31 -0400 (EDT) |
branch: elpa/with-editor
commit 90973212cee3655746c95d0e0cae2f8f5bf3d97e
Author: Stéphane Soppera <soppera@users.noreply.github.com>
Commit: Jonas Bernoulli <jonas@bernoul.li>
with-editor-export-editor: Fix for term-mode
The previous value of `with-editor-sleeping-editor' variable was using
the \x1f character in the elisp string and sending it to the
shell/term process.
For `shell-mode' this is not an issue since the character is not
interpreted by the shell. But in `term-mode' the terminal emulation
interprets it and remove previous characters, the "$0" string. So we
end up with an EDITOR environment variable that contains an echo
command with "…OPEN IN…" instead of "…OPEN $0^_ IN…" (here "^_" is
used to represent the \x1f byte).
This change fixes the issue by using `-e' flag of `echo' and using the
`\037' octal escape sequence for the \x1f byte.
---
with-editor.el | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/with-editor.el b/with-editor.el
index 7612200..4e439b7 100644
--- a/with-editor.el
+++ b/with-editor.el
@@ -176,7 +176,7 @@ please see
https://github.com/magit/magit/wiki/Emacsclient."))))
(defcustom with-editor-sleeping-editor "\
sh -c '\
-echo \"WITH-EDITOR: $$ OPEN $0 IN $(pwd)\"; \
+echo -e \"WITH-EDITOR: $$ OPEN $0\\037 IN $(pwd)\"; \
sleep 604800 & sleep=$!; \
trap \"kill $sleep; exit 0\" USR1; \
trap \"kill $sleep; exit 1\" USR2; \
- [nongnu] elpa/with-editor 22a581d 076/140: make: Remove deleted pages when publishing manual, (continued)
- [nongnu] elpa/with-editor 22a581d 076/140: make: Remove deleted pages when publishing manual, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor 71e61b9 081/140: with-editor-sleeping-editor: Also print the working directory, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor d3b0bc7 083/140: Update documentation, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor 87c96b3 091/140: with-editor-finish: Preserve local git-commit-post-finish-hook, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor ef0ef43 093/140: manual: Call ox-texinfo+-update-copyright-years while exporting, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor 513e13f 097/140: Fix previous commit, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor ff23166 099/140: Add .github/FUNDING.yml, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor f5a8d96 100/140: with-editor-usage-message: Delay message a little longer, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor ba4ca8a 108/140: with-editor-finish: Fix bug concerning git-commit-post-finish-hook, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor 324285f 109/140: Add htmlxref.cnf, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor 9097321 112/140: with-editor-export-editor: Fix for term-mode,
Jonas Bernoulli <=
- [nongnu] elpa/with-editor d8f2e99 116/140: Fix typo, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor f2a9b02 121/140: manual: Add indices, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor 7c51288 122/140: make-process--with-editor-process-filter: &allow-other-keys, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor 236c158 127/140: Require subr-x for when-let, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor 36e163c 132/140: manual: Reorder header properties a bit, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor ebcbd3b 134/140: Use defvar-local instead of make-variable-buffer-local, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor 48e1b2d 082/140: with-editor-shell-command-use-emacsclient: Fix last modified info, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor ac6b2e3 085/140: with-editor*: New macro, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor 53dcd4d 106/140: Fix typos, Jonas Bernoulli, 2021/08/06
- [nongnu] elpa/with-editor 590ff0b 124/140: manual: Use a favicon in html export, Jonas Bernoulli, 2021/08/06