Re: [PATCH 1/7] Redirect grep's stderr top null too.

From: Đoàn Trần Công Danh <congdanhqx@gmail.com>
Date: May 18, 2020, 14:13 UTC
Message-ID: <20200518141311.GC1980@danh.dev>
In-Reply-To: <20200518100356.29292-1-dtucker@dtucker.net>

Hi Darren,

On 2020-05-18 20:03:50+1000, Darren Tucker <dtucker@dtucker.net> wrote:

> Prevents pollution of configure output on platforms that don't have
> grep -a.

From your other's patch, I think you're in HP-UX, would you mind also run the test.

Since t5703 also uses "grep -a"

Show 22 quoted lines

> 
> Signed-off-by: Darren Tucker <dtucker@dtucker.net>
> ---
>  configure.ac | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 66aedb9288..4effc82b76 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -526,7 +526,7 @@ if test -n "$ASCIIDOC"; then
>  	esac
>  fi
>  
> -if grep -a ascii configure.ac >/dev/null; then
> +if grep -a ascii configure.ac >/dev/null 2>&1; then
>    AC_MSG_RESULT([Using 'grep -a' for sane_grep])
>    SANE_TEXT_GREP=-a
>  else
> -- 
> 2.21.3
>

-- 
Danh