[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/geiser-chicken 26b1f56 059/102: Use (car (process-lines ..
From: |
Philip Kaludercic |
Subject: |
[nongnu] elpa/geiser-chicken 26b1f56 059/102: Use (car (process-lines ...)) instead of (shell-command ...) |
Date: |
Sun, 1 Aug 2021 18:26:55 -0400 (EDT) |
branch: elpa/geiser-chicken
commit 26b1f560f7d462992f0ef82dfa6935bc71baebcf
Author: Jordan Brown <mrhmouse@gmail.com>
Commit: Jose Antonio Ortega Ruiz <jao@gnu.org>
Use (car (process-lines ...)) instead of (shell-command ...)
`shell-command` assumes Bourne-shell-compatible quoting, which
doesn't work when the user isn't using a Bourne-compatible shell.
Instead of futzing about with quoting, we can just use `process-lines`
to execute a process and pass it arguments directly.
---
elisp/geiser-chicken.el | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/elisp/geiser-chicken.el b/elisp/geiser-chicken.el
index 8c10908..60567b7 100644
--- a/elisp/geiser-chicken.el
+++ b/elisp/geiser-chicken.el
@@ -263,10 +263,7 @@ This function uses `geiser-chicken-init-file' if it
exists."
(defconst geiser-chicken-minimum-version "4.8.0.0")
(defun geiser-chicken--version (binary)
- (shell-command-to-string
- (format "%s -e %s"
- (shell-quote-argument binary)
- (shell-quote-argument "(display (chicken-version))"))))
+ (car (process-lines binary "-e" "(display (chicken-version))")))
(defun connect-to-chicken ()
"Start a Chicken REPL connected to a remote process."
- [nongnu] elpa/geiser-chicken 935a114 040/102: Allows redefinition of modules, (continued)
- [nongnu] elpa/geiser-chicken 935a114 040/102: Allows redefinition of modules, Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken 95d19aa 044/102: Minor changes to help with debugging completions, Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken 2098767 046/102: Quoting binary on version checks (issue #117), Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken 42b18d5 045/102: Minor cleanup, Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken 1ff0274 048/102: Performance Improvements, Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken 0bcd595 050/102: Fuzzy matching for Chicken AutoDoc and Completions, Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken 5cc0dd2 052/102: Fix for #127, Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken 402e1bc 054/102: Minor typo, Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken fc0c8e6 056/102: Disable aggressive prefix assumption, Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken 1c969ee 058/102: Allows Chicken to limit the number of symbols provided to Geiser, Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken 26b1f56 059/102: Use (car (process-lines ...)) instead of (shell-command ...),
Philip Kaludercic <=
- [nongnu] elpa/geiser-chicken 17a3799 062/102: Removed rate limit setting., Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken a35ce4e 065/102: Chicken 5 works with Geiser, many features removed, Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken d69c1ea 069/102: Removed more unused code, speeding things up., Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken c85d953 071/102: Only check for specific chicken words, not all their keywords., Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken 3486915 075/102: geiser-chicken.el: Use a more stable API to get CHICKEN's version, Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken 0740b46 076/102: Checking Chicken's version with the proper invocation, Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken 2caa284 072/102: Fixes autodoc in Chicken, #258, Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken d1abcb1 074/102: geiser-chicken.el: Fix jao's name, Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken a6f88ca 081/102: Mark the beginning of code part of elisp libraries with Code: heading, Philip Kaludercic, 2021/08/01
- [nongnu] elpa/geiser-chicken 616954d 078/102: Fix latest chicken. Add debug level logging. Fixed typo., Philip Kaludercic, 2021/08/01