[PATCH] wordexp.3: Make it clear that WRDE_NOCMD prevents command substitution.

[PATCH] wordexp.3: Make it clear that WRDE_NOCMD prevents command substitution.

[Carlos O'Donell]

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

Re: [PATCH] wordexp.3: Make it clear that WRDE_NOCMD prevents command substitution.

[Michael Kerrisk (man-pages)]

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