From: "Lawrence Velázquez" <larryv@zsh.org>
To: zsh-workers@zsh.org
Cc: zeurkous@volny.cz
Subject: Re: twice revised patch: zshmisc(1) clarify non-successful exit statuses
Date: Sun, 18 Jul 2021 18:27:15 -0400 [thread overview]
Message-ID: <9255181c-83e2-4edd-8d36-4a2fcd5433b0@www.fastmail.com> (raw)
In-Reply-To: <20210708225357.58B28287@volny.cz>
On Thu, Jul 8, 2021, at 4:53 PM, zeurkous@volny.cz wrote:
> # One person branded me earlier wording of a sentence ``ungrammatical'';
> # while me doesn't necessarily agree, me sees no harm in the proposed
> # change and this has hence been incorporated.
Thanks, I appreciate it.
> --- Doc/Zsh/..v/5.8/exec.yo Mon Dec 4 14:09:35 2017
> +++ Doc/Zsh/exec.yo Thu Jul 8 20:13:50 2021
> @@ -14,10 +14,21 @@
>
> vindex(path, use of)
> Otherwise, the shell searches each element of tt($path) for a
> -directory containing an executable file by that name. If the
> -search is unsuccessful, the shell prints an error message and returns
> -a nonzero exit status.
> +directory containing an executable file by that name.
>
> +If execution fails: an error message is printed, and one of the
> +following values is returned.
> +
> +startsitem()
> +sitem(127)(The search was unsuccessful. The error message is
> +`tt(command not found:) var(cmd)'.)
> +sitem(126)(The executable file has insufficient permissions, is a
> +directory or special file, or is not a script and is in a format
> +unrecognized by the operating system. The exact conditions and error
> +message are operating system-dependent; see
> +manref(execve)(2).)
> +endsitem()
> +
> If execution fails because the file is not in executable format,
> and the file is not a directory, it is assumed to be a shell
> script. tt(/bin/sh) is spawned to execute it. If the program
> @@ -29,10 +40,6 @@
> If no external command is found but a function tt(command_not_found_handler)
> exists the shell executes this function with all
> command line arguments. The return status of the function becomes the
> -status of the command. If the function wishes to mimic the
> -behaviour of the shell when the command is not found, it should
> -print the message `tt(command not found:) var(cmd)' to standard error
> -and return status 127. Note that the handler is executed in a
> +status of the command. Note that the handler is executed in a
> subshell forked to execute an external command, hence changes to
> directories, shell parameters, etc. have no effect on the main shell.
> -
Anyone have further feedback and/or is up for committing this?
--
vq
next prev parent reply other threads:[~2021-07-18 22:27 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-08 20:53 zeurkous
2021-07-18 22:27 ` Lawrence Velázquez [this message]
2021-07-28 17:48 ` Bart Schaefer
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=9255181c-83e2-4edd-8d36-4a2fcd5433b0@www.fastmail.com \
--to=larryv@zsh.org \
--cc=zeurkous@volny.cz \
--cc=zsh-workers@zsh.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.vuxu.org/mirror/zsh/
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 NNTP newsgroup(s).