From: Ihor Radchenko <yantar92@posteo.net>
To: Eli Zaretskii <eliz@gnu.org>
Cc: dmitry@gutov.dev, 71081@debbugs.gnu.org, matt@excalamus.com
Subject: bug#71081: 30.0.50; shell-command-on-region outputs boilerplate text on Windows
Date: Thu, 23 May 2024 14:36:22 +0000 [thread overview]
Message-ID: <87y180wq6x.fsf@localhost> (raw)
In-Reply-To: <867cfk4nus.fsf@gnu.org>
Eli Zaretskii <eliz@gnu.org> writes:
>> Should it be documented in the docstring of `shell-command-to-string'?
>
> We could, but IMO such tangential information belongs to the manual,
> not to the doc string. We should also somehow avoid the slippery
> slope of describing all the differences between the Posix shells and
> the Windows shell. E.g., newlines are also not allowed in Windows
> file names, but we don't mention that in doc strings of every function
> that deals with file names.
Understood.
>> Why would anyone assume that `shell-command-to-string' is
>> passed as an argument to cmd.exe and not piped as input?
>
> Because the doc string says so:
>
> Execute shell command COMMAND and return its output as a string.
>
> "Execute COMMAND" means run it as "SHELL -c COMMAND", in any
> reasonable interpretation. Anything else is not the usual way of
> running commands.
Another reasonable interpretation is: "do the same thing as if I type
COMMAND into shell prompt". But that's me. If you think that my
interpretation is uncommon, so be it.
>> more over, on Linux, different shells have different behaviors wrt
>> input/command argument/script file).
>
> Same on Windows. Try PowerShell some day. Does it mean we need to
> document all of that in our documentation?
It would be nice, yes. Without details - just point that there is a
difference. Maybe in the "41.3 Creating a Synchronous Process" section
of the manual.
--
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>
next prev parent reply other threads:[~2024-05-23 14:36 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-05-20 18:34 bug#71081: 30.0.50; shell-command-on-region outputs boilerplate text on Windows Ihor Radchenko
2024-05-20 18:59 ` Eli Zaretskii
2024-05-21 20:18 ` Dmitry Gutov
2024-05-22 2:30 ` Eli Zaretskii
2024-05-22 11:45 ` Ihor Radchenko
2024-05-22 13:33 ` Eli Zaretskii
2024-05-22 14:26 ` Ihor Radchenko
2024-05-22 14:53 ` Eli Zaretskii
2024-05-23 12:06 ` Ihor Radchenko
2024-05-23 13:37 ` Eli Zaretskii
2024-05-23 14:00 ` Ihor Radchenko
2024-05-23 14:14 ` Eli Zaretskii
2024-05-23 14:36 ` Ihor Radchenko [this message]
2024-05-25 10:03 ` Eli Zaretskii
2024-05-25 10:03 ` Eli Zaretskii
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=87y180wq6x.fsf@localhost \
--to=yantar92@posteo.net \
--cc=71081@debbugs.gnu.org \
--cc=dmitry@gutov.dev \
--cc=eliz@gnu.org \
--cc=matt@excalamus.com \
/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.