[Buildroot] [PATCH] utils/test-pkg: log the output of merge-config

[Buildroot] [PATCH] utils/test-pkg: log the output of merge-config

[Yann E. MORIN]

Currently, the stdout is consigned to oblivion, while the stderr is not
redirected at all. So, when the configuration under test redefines a
symbol, like is the case when testing busybox for example, there is an
ugly warning.

So, just redirect both stdout and stderr to the logfile.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 utils/test-pkg | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/utils/test-pkg b/utils/test-pkg
index 54951276fa..aa91ee02cf 100755
--- a/utils/test-pkg
+++ b/utils/test-pkg
@@ -131,7 +131,7 @@ build_one() {
 
     support/kconfig/merge_config.sh -O "${dir}" \
         "${toolchainconfig}" "support/config-fragments/minimal.config" "${cfg}" \
-        > /dev/null
+        >> "${dir}/logfile" 2>&1
     # We want all the options from the snippet to be present as-is (set
     # or not set) in the actual .config; if one of them is not, it means
     # some dependency from the toolchain or arch is not available, in
-- 
2.14.1

[Buildroot] [PATCH] utils/test-pkg: log the output of merge-config

[Matthew Weber]

Yann,

On Sun, Jul 8, 2018 at 5:26 AM, Yann E. MORIN <yann.morin.1998@free.fr> wrote:
> Currently, the stdout is consigned to oblivion, while the stderr is not
> redirected at all. So, when the configuration under test redefines a
> symbol, like is the case when testing busybox for example, there is an
> ugly warning.
>
> So, just redirect both stdout and stderr to the logfile.
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Tested-by: Matt Weber <matthew.weber@rockwellcollins.com>

[Buildroot] [PATCH] utils/test-pkg: log the output of merge-config

[Arnout Vandecappelle]

On 08-07-18 12:26, Yann E. MORIN wrote:
> Currently, the stdout is consigned to oblivion, while the stderr is not
> redirected at all. So, when the configuration under test redefines a
> symbol, like is the case when testing busybox for example, there is an
> ugly warning.
> 
> So, just redirect both stdout and stderr to the logfile.
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

 Applied to master, thanks.

 Regards,
 Arnout

> ---
>  utils/test-pkg | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/utils/test-pkg b/utils/test-pkg
> index 54951276fa..aa91ee02cf 100755
> --- a/utils/test-pkg
> +++ b/utils/test-pkg
> @@ -131,7 +131,7 @@ build_one() {
>  
>      support/kconfig/merge_config.sh -O "${dir}" \
>          "${toolchainconfig}" "support/config-fragments/minimal.config" "${cfg}" \
> -        > /dev/null
> +        >> "${dir}/logfile" 2>&1
>      # We want all the options from the snippet to be present as-is (set
>      # or not set) in the actual .config; if one of them is not, it means
>      # some dependency from the toolchain or arch is not available, in
> 

-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

[Buildroot] [PATCH] utils/test-pkg: log the output of merge-config

[Peter Korsgaard]

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 > Currently, the stdout is consigned to oblivion, while the stderr is not
 > redirected at all. So, when the configuration under test redefines a
 > symbol, like is the case when testing busybox for example, there is an
 > ugly warning.

 > So, just redirect both stdout and stderr to the logfile.

 > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Committed to 2018.02.x and 2018.05.x, thanks.

-- 
Bye, Peter Korsgaard