linux-man.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] wordexp.3: Make it clear that WRDE_NOCMD prevents command substitution.
@ 2014-11-19 19:45 Carlos O'Donell
       [not found] ` <546CF348.5080400-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
  0 siblings, 1 reply; 2+ messages in thread
From: Carlos O'Donell @ 2014-11-19 19:45 UTC (permalink / raw)
  To: Michael Kerrisk; +Cc: linux-man-u79uwXL29TY76Z2rM5mHXA@public.gmane.org

The use of WRDE_NOCMD prevents command substitution. If the flag
WRDE_NOCMD is set then no command substitution shall occur and
the error WRDE_CMDSUB will be returned if such substitution is
requested when processing the words.

The manual page as-is makes it seem like the command substitution
occurs, and an error is returned *after* the substitution.

This patch clarifies that.

Patch against trunk.

Signed-off-by: Carlos O'Donell <carlos-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

diff --git a/man3/wordexp.3 b/man3/wordexp.3
index c695c08..e457943 100644
--- a/man3/wordexp.3
+++ b/man3/wordexp.3
@@ -169,7 +169,7 @@ flag
 told us to consider this an error.
 .TP
 .B WRDE_CMDSUB
-Command substitution occurred, and the
+Command substitution requested, but the
 .B WRDE_NOCMD
 flag told us to consider this an error.
 .TP
---

Cheers,
Carlos.
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] wordexp.3: Make it clear that WRDE_NOCMD prevents command substitution.
       [not found] ` <546CF348.5080400-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
@ 2014-12-26  6:22   ` Michael Kerrisk (man-pages)
  0 siblings, 0 replies; 2+ messages in thread
From: Michael Kerrisk (man-pages) @ 2014-12-26  6:22 UTC (permalink / raw)
  To: Carlos O'Donell
  Cc: mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w,
	linux-man-u79uwXL29TY76Z2rM5mHXA@public.gmane.org

Hi Carlos,

On 11/19/2014 08:45 PM, Carlos O'Donell wrote:
> The use of WRDE_NOCMD prevents command substitution. If the flag
> WRDE_NOCMD is set then no command substitution shall occur and
> the error WRDE_CMDSUB will be returned if such substitution is
> requested when processing the words.
> 
> The manual page as-is makes it seem like the command substitution
> occurs, and an error is returned *after* the substitution.
> 
> This patch clarifies that.
> 
> Patch against trunk.
> 
> Signed-off-by: Carlos O'Donell <carlos-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
> 
> diff --git a/man3/wordexp.3 b/man3/wordexp.3
> index c695c08..e457943 100644
> --- a/man3/wordexp.3
> +++ b/man3/wordexp.3
> @@ -169,7 +169,7 @@ flag
>  told us to consider this an error.
>  .TP
>  .B WRDE_CMDSUB
> -Command substitution occurred, and the
> +Command substitution requested, but the
>  .B WRDE_NOCMD
>  flag told us to consider this an error.
>  .TP

Thanks!

Applied.

Cheers,

Michael



-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-12-26  6:22 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-19 19:45 [PATCH] wordexp.3: Make it clear that WRDE_NOCMD prevents command substitution Carlos O'Donell
     [not found] ` <546CF348.5080400-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2014-12-26  6:22   ` Michael Kerrisk (man-pages)

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).