Gentoo Archives: gentoo-dev

From: Matt Turner <mattst88@g.o>
To: gentoo-dev@l.g.o
Cc: "Michał Górny" <mgorny@g.o>
Subject: Re: [gentoo-dev] [PATCH multilib-build] Tee the build logs to ABI-specific files.
Date: Thu, 21 Feb 2013 16:26:36
Message-Id: CAEdQ38EWP0Y0ozTKqj86FQWKwnuu-o5OOukO-itmpu4ZZ9dMqA@mail.gmail.com
In Reply to: [gentoo-dev] [PATCH multilib-build] Tee the build logs to ABI-specific files. by "Michał Górny"
1 On Thu, Feb 21, 2013 at 3:27 AM, Michał Górny <mgorny@g.o> wrote:
2 > This makes reading them a bit easier, especially with phases run
3 > in parallel.
4 > ---
5 > gx86/eclass/multilib-build.eclass | 7 ++++---
6 > 1 file changed, 4 insertions(+), 3 deletions(-)
7 >
8 > diff --git a/gx86/eclass/multilib-build.eclass b/gx86/eclass/multilib-build.eclass
9 > index feac748..93c4335 100644
10 > --- a/gx86/eclass/multilib-build.eclass
11 > +++ b/gx86/eclass/multilib-build.eclass
12 > @@ -100,7 +100,8 @@ multilib_foreach_abi() {
13 > local ABI
14 > for ABI in $(multilib_get_enabled_abis); do
15 > multilib_toolchain_setup "${ABI}"
16 > - BUILD_DIR=${initial_dir%%/}-${ABI} "${@}"
17 > + local BUILD_DIR=${initial_dir%%/}-${ABI}
18 > + "${@}" | tee -a "${T}/build-${ABI}.log"
19 > done
20 > }
21 >
22 > @@ -127,8 +128,8 @@ multilib_parallel_foreach_abi() {
23 > multijob_child_init
24 >
25 > multilib_toolchain_setup "${ABI}"
26 > - BUILD_DIR=${initial_dir%%/}-${ABI}
27 > - "${@}"
28 > + local BUILD_DIR=${initial_dir%%/}-${ABI}
29 > + "${@}" 2>&1 | tee -a "${T}/build-${ABI}.log"
30 > ) &
31 >
32 > multijob_post_fork
33 > --
34 > 1.8.1.2
35 >
36 >
37
38 Definitely seems like a good idea.