Re: [PATCH 02/10] merge_config.sh: print warnings on stderr

From: Darren Hart <dvhart@infradead.org>
To: Olof Johansson <olof@lixom.net>,
	Bruce Ashfield <bruce.ashfield@windriver.com>
Cc: Michal Marek <mmarek@suse.com>,
	linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org,
	dvhart@linux.intel.com
Subject: Re: [PATCH 02/10] merge_config.sh: print warnings on stderr
Date: Wed, 28 Oct 2015 14:56:16 +0900	[thread overview]
Message-ID: <20151028055616.GD1854@malice.jf.intel.com> (raw)
In-Reply-To: <1445992931-28107-3-git-send-email-olof@lixom.net>

On Wed, Oct 28, 2015 at 09:42:03AM +0900, Olof Johansson wrote:
> Makes it easier to grab just errors/warnings in a build log.
> 
> Signed-off-by: Olof Johansson <olof@lixom.net>

I think this is the right change, I just want to make sure Bruce (Cc'd) sees it
for linux-yocto.

Reviewed-by: Darren Hart <dvhart@linux.intel.com>

> ---
>  scripts/kconfig/merge_config.sh | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/scripts/kconfig/merge_config.sh b/scripts/kconfig/merge_config.sh
> index d8cd913..0e98619 100755
> --- a/scripts/kconfig/merge_config.sh
> +++ b/scripts/kconfig/merge_config.sh
> @@ -131,7 +131,7 @@ for MERGE_FILE in $MERGE_LIST ; do
>  			echo Value of $CFG is redundant by fragment $MERGE_FILE:
>  		fi
>  		sed -i "/$CFG[ =]/d" $TMP_FILE
> -	done
> +	done >&2
>  	cat $MERGE_FILE >> $TMP_FILE
>  done
>  
> @@ -168,7 +168,7 @@ for CFG in $(sed -n "$SED_CONFIG_EXP" $TMP_FILE); do
>  		echo "Requested value:  $REQUESTED_VAL"
>  		echo "Actual value:     $ACTUAL_VAL"
>  		echo ""
> -	fi
> +	fi >&2
>  done
>  
>  clean_up
> -- 
> 2.1.4
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
> 

-- 
Darren Hart
Intel Open Source Technology Center